Skip to content
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.

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

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_gateway_listen_for_bound_device_configs failed #2860

Closed
flaky-bot bot opened this issue Feb 11, 2020 · 9 comments · Fixed by #3428
Assignees
Labels
api: cloudiot Issues related to the IoT Core API. 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.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Feb 11, 2020

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_gateway_listen_for_bound_device_configs failed
buildID: 48f7af7
buildURL: https://source.cloud.google.com/results/invocations/067ead8d-9004-4bef-acf3-6c74265f1e31
status: failed

@flaky-bot flaky-bot bot added buildcop:issue priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Feb 11, 2020
@kurtisvg kurtisvg assigned gguuss and unassigned kurtisvg Feb 11, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 12, 2020

@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 26, 2020

Looks like this issue is flaky. 😟

I'm going to leave this open and stop commenting.

A human should fix and close this.

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_gateway_listen_for_bound_device_configs failed
buildID: ff9682e
buildURL: Build Status, Sponge
status: failed

@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Feb 26, 2020
@gguuss
Copy link
Contributor

gguuss commented Feb 26, 2020

This is a known flaky test.

@gguuss gguuss closed this as completed Feb 26, 2020
@flaky-bot flaky-bot bot reopened this Feb 26, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 26, 2020

Oops! Looks like this issue is still flaky. 😬

I reopened the issue, but a human will need to close it again.

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_gateway_listen_for_bound_device_configs failed
buildID: 25107b8
buildURL: Build Status, Sponge
status: failed

@tbpg tbpg added api: cloudiot Issues related to the IoT Core API. and removed api: iot labels Feb 27, 2020
@gguuss
Copy link
Contributor

gguuss commented Feb 27, 2020

This is a known flaky test.

@gguuss gguuss closed this as completed Feb 27, 2020
@flaky-bot flaky-bot bot reopened this Feb 29, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 29, 2020

Oops! Looks like this issue is still flaky. 😬

I reopened the issue, but a human will need to close it again.

buildID: a669fae
buildURL: Build Status, Sponge
status: failed

@leahecole leahecole added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed 🚨 This issue needs some love. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Mar 9, 2020
@gguuss
Copy link
Contributor

gguuss commented Mar 11, 2020

Potentially fixed in #3068

@gguuss gguuss closed this as completed Mar 11, 2020
@flaky-bot flaky-bot bot reopened this Mar 20, 2020
@flaky-bot flaky-bot bot added the priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. label Mar 20, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Mar 20, 2020

Oops! Looks like this issue is still flaky. 😬

I reopened the issue, but a human will need to close it again.


commit: 1037f47
buildURL: Build Status, Sponge
status: failed

@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Mar 20, 2020
@kurtisvg kurtisvg removed the priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. label Apr 1, 2020
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Apr 1, 2020
@gguuss
Copy link
Contributor

gguuss commented Apr 1, 2020

I may need to use the "safe clear" delete for instances where the tests are interrupted.

tmatsuo pushed a commit that referenced this issue Apr 21, 2020
* [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
Labels
api: cloudiot Issues related to the IoT Core API. 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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants