Skip to content

test(bigtable): disable testDeadlockPrevention to fix CI execution timeout - #13902

Merged
lqiu96 merged 2 commits into
mainfrom
fix/disable-bigtable-timeout-tests
Jul 27, 2026
Merged

test(bigtable): disable testDeadlockPrevention to fix CI execution timeout#13902
lqiu96 merged 2 commits into
mainfrom
fix/disable-bigtable-timeout-tests

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 27, 2026

Copy link
Copy Markdown
Member

Disable flaky test (#13903). Bigtable unit tests are run as part of the main units test suite. This is causing some of our required checks to timeout after ~360 minutes. Disable flaky tests to be re-enabled after they get resolved.

@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 disables the testDeadlockPrevention test in ClientConfigurationManagerTest.java by adding the @disabled annotation. The reviewer suggests providing an explanation or tracking issue within the @disabled annotation as a best practice to ensure the test is eventually re-enabled.

@lqiu96
lqiu96 force-pushed the fix/disable-bigtable-timeout-tests branch 2 times, most recently from 4244522 to 0303142 Compare July 27, 2026 17:53
@lqiu96
lqiu96 marked this pull request as ready for review July 27, 2026 18:23
@lqiu96
lqiu96 requested review from a team as code owners July 27, 2026 18:23
@lqiu96
lqiu96 merged commit 6991168 into main Jul 27, 2026
210 of 212 checks passed
@lqiu96
lqiu96 deleted the fix/disable-bigtable-timeout-tests branch July 27, 2026 18:23
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