Skip to content

[iot] flake: cloudiot_mqtt_example_test.test_gateway_trigger_error_topic failed #3323

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
tmatsuo opened this issue Apr 10, 2020 · 0 comments · Fixed by #3428
Closed

[iot] flake: cloudiot_mqtt_example_test.test_gateway_trigger_error_topic failed #3323

tmatsuo opened this issue Apr 10, 2020 · 0 comments · Fixed by #3428
Assignees
@tmatsuo
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. testing type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 10, 2020

Build: https://source.cloud.google.com/results/invocations/7386e971-7d94-4b85-8102-98bd5cba10c6/targets

Test cloudiot_mqtt_example_test.test_gateway_trigger_error_topic

google.api_core.exceptions.FailedPrecondition: 400 The registry with the name 'projects/python-docs-samples-tests/locations/us-central1/registries/test-registry-671aa840-7ac0-11ea-b982-42010a8000b3-1586477737' can't be deleted because it's not empty.
@tmatsuo tmatsuo added testing priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Apr 10, 2020
@tmatsuo tmatsuo self-assigned this Apr 10, 2020
@tmatsuo tmatsuo mentioned this issue Apr 10, 2020
Closed
1 task
@tmatsuo tmatsuo mentioned this issue Apr 17, 2020
Merged
tmatsuo pushed a commit to tmatsuo/python-docs-samples that referenced this issue Apr 19, 2020
[iot] fix: use fixtures for resource setup/teardown
3567574 
fixes GoogleCloudPlatform#2861
fixes GoogleCloudPlatform#3323
tmatsuo pushed a commit that referenced this issue Apr 21, 2020
[iot] fix: use fixtures for resource setup/teardown (#3428)
e2cf90d 
* [iot] fix: use fixtures for resource setup/teardown

fixes #2861
fixes #3323
fixes #2859
fixes #2860

* avoid topic name conflict
* refactored out the fixtures
* share the fixtures within two tests
* correct dependencies between fixtures
* use threading for cordinating between background and foreground
  threads.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmatsuo tmatsuo

Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. testing type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[iot] fix: use fixtures for resource setup/teardown tmatsuo/python-docs-samples
1 participant
@tmatsuo