Skip to content

codelabs.flex_and_vision.main_test: test_upload_photo failed #2925

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
flaky-bot bot opened this issue Feb 18, 2020 · 5 comments · Fixed by #3489
Closed

codelabs.flex_and_vision.main_test: test_upload_photo failed #2925

flaky-bot bot opened this issue Feb 18, 2020 · 5 comments · Fixed by #3489
Assignees
@tmatsuo
Labels
api: vision Issues related to the Cloud Vision 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 18, 2020

codelabs.flex_and_vision.main_test: test_upload_photo failed
buildID: 6c0760b
buildURL: https://source.cloud.google.com/results/invocations/89924ede-76d1-4a32-b1a0-d7c2ac34c425
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 18, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 18, 2020

codelabs.flex_and_vision.main_test: test_upload_photo failed
buildID: 6c0760b
buildURL: https://source.cloud.google.com/results/invocations/652e543e-512c-44d9-986d-e6d1ef7ac164
status: failed

@JustinBeckwith JustinBeckwith assigned czahedi and unassigned busunkim96 Feb 19, 2020
@JustinBeckwith JustinBeckwith added the api: vision Issues related to the Cloud Vision API. label Feb 19, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Feb 20, 2020

Test passed in build 3e2ebaf (Build Status, Sponge)! Closing this issue.

@flaky-bot flaky-bot bot closed this as completed Feb 20, 2020
@flaky-bot flaky-bot bot reopened this Apr 8, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Apr 8, 2020

Looks like this issue is flaky. 😟

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

A human should fix and close this.

commit: 442efa4
buildURL: Build Status, Sponge
status: failed

@czahedi czahedi assigned sgreenberg and unassigned czahedi Apr 8, 2020
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Apr 17, 2020
@gguuss gguuss added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Apr 21, 2020
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Apr 21, 2020
@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 23, 2020

@sgreenberg Are you still the right person to look at this? If so, this must be an easy fix (see the comment below). Can you fix this right away? It has been open for more than 2 months now.

The log

Error:
google.api_core.exceptions.TooManyRequests: 429 PATCH https://storage.googleapis.com/storage/v1/b/python-docs-samples-tests/o/flex_and_vision.jpg?projection=full: The rate of change requests to the object python-docs-samples-tests/flex_and_vision.jpg exceeds the rate limit. Please reduce the rate of create, update, and delete requests.

It seems like an easy fix on this line:

python-docs-samples/codelabs/flex_and_vision/main_test.py

Line 44 in 1cc9e70

'file': (six.BytesIO(test_photo_data), 'flex_and_vision.jpg')

just by adding uuid (or any other sorts) in the filename.

@sgreenberg sgreenberg assigned gguuss and telpirion and unassigned sgreenberg Apr 23, 2020
@tmatsuo tmatsuo assigned tmatsuo and unassigned gguuss and telpirion Apr 23, 2020
tmatsuo pushed a commit to tmatsuo/python-docs-samples that referenced this issue Apr 23, 2020
[codelabs] fix: use unique filename for photo upload
8ec97fb 
fixes GoogleCloudPlatform#2925
@tmatsuo tmatsuo mentioned this issue Apr 23, 2020
Merged
tmatsuo pushed a commit that referenced this issue Apr 24, 2020
[codelabs] fix: use unique filename for photo upload (#3489)
3133f4e 
fixes #2925
@telpirion
Copy link
Contributor

Thank you @tmatsuo !

@flaky-bot flaky-bot bot mentioned this issue Dec 14, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmatsuo tmatsuo

Labels
api: vision Issues related to the Cloud Vision 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.

[codelabs] fix: use unique filename for photo upload tmatsuo/python-docs-samples
8 participants
@tmatsuo @JustinBeckwith @gguuss @sgreenberg @telpirion @busunkim96 @czahedi @yoshi-automation