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.

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

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_receive_command failed #2859

Closed
flaky-bot bot opened this issue Feb 11, 2020 · 8 comments · Fixed by #3428
Closed

iot.api-client.mqtt_example.cloudiot_mqtt_example_test: test_receive_command failed #2859

flaky-bot bot opened this issue Feb 11, 2020 · 8 comments · Fixed by #3428
Assignees
@gguuss
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_receive_command 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
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 12, 2020

Test passed in build fbcdef1 (https://source.cloud.google.com/results/invocations/04397697-dd91-4cfb-a092-94cdeb1e87b2)! Closing this issue.

@flaky-bot flaky-bot bot closed this as completed Feb 12, 2020
@flaky-bot flaky-bot bot reopened this Feb 26, 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_receive_command failed
buildID: ff9682e
buildURL: Build Status, Sponge
status: failed

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

gguuss commented Feb 26, 2020

Yes, this is a 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_receive_command 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
@tmatsuo tmatsuo mentioned this issue Apr 20, 2020
Merged
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

@gguuss gguuss

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
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
6 participants
@dmahugh @gguuss @leahecole @tbpg @kurtisvg @yoshi-automation