Skip to content

Conversation

@ywmei-brt1
Copy link
Collaborator

No description provided.

@ywmei-brt1 ywmei-brt1 force-pushed the break-a-test-case branch 2 times, most recently from 177d68b to 20355c3 Compare August 15, 2025 18:48
@ywmei-brt1 ywmei-brt1 changed the title I think we don't have any test to protect this repo Update bazel verison Aug 15, 2025
@ywmei-brt1 ywmei-brt1 requested a review from gkousik August 15, 2025 19:11
@ywmei-brt1 ywmei-brt1 force-pushed the break-a-test-case branch 2 times, most recently from f64b375 to c7a27e4 Compare August 15, 2025 20:32
To open this project with GoLand + bazel plugin, the min version of bazel we need is 6.0.0.

There is a race condition in the test that `batch_retries_test.go` cannot run together with the rest of the test files.

This CL fix add a lock to fix the race condition. add "exclusive" bazel tag to not run client_test with other bazel test targets, and
add an arg `-test.parallel=1` to make go run all the test case under client_test one by one.

Test: run bazelisk test //... --test_output=streamed --flaky_test_attempts=1 on my cloudtop
Bug: NA
Copy link

@fangism fangism left a comment

Choose a reason for hiding this comment

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

I can confirm, bazel test ... passes with this change.

@fangism fangism mentioned this pull request Aug 15, 2025
@ywmei-brt1 ywmei-brt1 merged commit 5c71954 into bazelbuild:master Aug 18, 2025
6 checks passed
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.

3 participants