Skip to content

video.cloud-client.analyze.video_detect_logo_test: test_detect_logo failed #3275

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 Apr 3, 2020 · 4 comments · Fixed by #3446
Closed

video.cloud-client.analyze.video_detect_logo_test: test_detect_logo failed #3275

flaky-bot bot opened this issue Apr 3, 2020 · 4 comments · Fixed by #3446
Assignees
@tmatsuo
Labels
api: videointelligence Issues related to the Video Intelligence API API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. 🚨 This issue needs some love. 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 Apr 3, 2020

This test failed!

To configure my behavior, see the Build Cop Bot documentation.

If I'm commenting on this issue too often, add the buildcop: quiet label and
I will stop commenting.

commit: ff791ec
buildURL: Build Status, Sponge
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 Apr 3, 2020
@flaky-bot
Copy link
Author

flaky-bot bot commented Apr 3, 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: ff791ec
buildURL: Build Status, Sponge
status: passed

@JustinBeckwith JustinBeckwith added the api: videointelligence Issues related to the Video Intelligence API API. label Apr 7, 2020
@tmatsuo tmatsuo assigned tmatsuo and unassigned crwilcox Apr 7, 2020
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Apr 17, 2020
@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

The error: google.api_core.exceptions.InvalidArgument: 400 Request contains an invalid argument.

I'll try to reproduce this.

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

The build log above says that the failed tests are:

  • beta_snippets_test.test_track_objects
  • video_detect_logo_beta_test.test_sample_annotate_video
  • video_detect_logo_gcs_beta_test.test_sample_annotate_video

The error for the test video.cloud-client.analyze.video_detect_logo_test: test_detect_logo was google.api_core.exceptions.ServiceUnavailable: 503 502:Bad Gateway.

@tmatsuo
Copy link
Contributor

tmatsuo commented Apr 20, 2020

The tests are somewhat stable when I tried locally. I think just marking tests with pytest.mark.flaky will mitigate the issue.

tmatsuo pushed a commit to tmatsuo/python-docs-samples that referenced this issue Apr 20, 2020
[video] testing: mark some tests witih flaky
cb73881 
fixes GoogleCloudPlatform#3275
@tmatsuo tmatsuo mentioned this issue Apr 20, 2020
Merged
tmatsuo pushed a commit that referenced this issue Apr 21, 2020
[video] testing: mark some tests witih flaky (#3446)
38af56e 
fixes #3275
dizcology pushed a commit that referenced this issue Sep 11, 2023
@danoscarmike
[video] testing: mark some tests witih flaky [(#3446)](#3446)
ada70ec 
fixes #3275
leahecole pushed a commit that referenced this issue Sep 15, 2023
@leahecole
[video] testing: mark some tests witih flaky [(#3446)](#3446)
f62af27 
fixes #3275
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmatsuo tmatsuo

Labels
api: videointelligence Issues related to the Video Intelligence API API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. 🚨 This issue needs some love. 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.

[video] testing: mark some tests witih flaky tmatsuo/python-docs-samples
4 participants
@tmatsuo @JustinBeckwith @crwilcox @yoshi-automation