Skip to content

chore(vision): remove flaky ITSystemTest.detectFacesGcsTest - #14837

Merged
jinseopkim0 merged 1 commit into
mainfrom
disable-flaky-vision-detect-faces-gcs
Nov 11, 2026
Merged

jinseopkim0 merged 1 commit into
mainfrom
disable-flaky-vision-detect-faces-gcs

Conversation

@jinseopkim0

@jinseopkim0 jinseopkim0 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Remove flaky test com.google.cloud.vision.it.ITSystemTest.detectFacesGcsTest.

b/579436201

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request ignores the flaky integration test detectFacesGcsTest in ITSystemTest.java by adding the @Ignore annotation with a reference to the tracking issue. There are no review comments, and I have no feedback to provide.

@jinseopkim0
jinseopkim0 requested a review from blakeli0 October 29, 2026 17:53
@jinseopkim0
jinseopkim0 marked this pull request as ready for review October 29, 2026 17:53
@jinseopkim0
jinseopkim0 requested a review from a team as a code owner October 29, 2026 17:53
@jinseopkim0 jinseopkim0 changed the title chore(vision): disable flaky ITSystemTest.detectFacesGcsTest chore(vision): remove flaky ITSystemTest.detectFacesGcsTest Oct 29, 2026
@jinseopkim0
jinseopkim0 enabled auto-merge (squash) October 29, 2026 22:35
@jinseopkim0
jinseopkim0 requested a review from blakeli0 October 29, 2026 22:35
@jinseopkim0
jinseopkim0 force-pushed the disable-flaky-vision-detect-faces-gcs branch from 0dc9468 to 1231456 Compare November 11, 2026 16:11
@jinseopkim0
jinseopkim0 merged commit 2052ae5 into main Nov 11, 2026
195 of 196 checks passed
@jinseopkim0
jinseopkim0 deleted the disable-flaky-vision-detect-faces-gcs branch November 11, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants