Skip to content
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.

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

Various test improvements #11582

Merged

Conversation

kjnilsson
Copy link
Contributor

various flakes and speed improvements.

@mergify mergify bot added the bazel label Jun 28, 2024
@kjnilsson kjnilsson force-pushed the testfixes-glorious-testfixes-and-other-improvements-hurray branch from 3c9b951 to 2750535 Compare June 28, 2024 10:04
@kjnilsson kjnilsson changed the title Testfixes glorious testfixes and other improvements hurray test improvements Jun 28, 2024
@kjnilsson kjnilsson changed the title test improvements Various test improvements Jun 28, 2024
@kjnilsson kjnilsson force-pushed the testfixes-glorious-testfixes-and-other-improvements-hurray branch 3 times, most recently from 8ccbb51 to ac6ea4b Compare June 28, 2024 14:44
test `confirm_nack` had a race condition that mean either ack or nack
where likely outcomes. By stopping the queue before the publish we
ensure only nack is the valid outcome.
By removing explicit meta data store groups.
@kjnilsson kjnilsson force-pushed the testfixes-glorious-testfixes-and-other-improvements-hurray branch from beaea61 to 2b09891 Compare July 1, 2024 09:27
@kjnilsson kjnilsson marked this pull request as ready for review July 1, 2024 09:58
@kjnilsson kjnilsson merged commit 7273c68 into main Jul 1, 2024
251 checks passed
@kjnilsson kjnilsson deleted the testfixes-glorious-testfixes-and-other-improvements-hurray branch July 1, 2024 15:54
@michaelklishin
Copy link
Member

@Mergifyio backport v3.13.x

Copy link

mergify bot commented Jul 1, 2024

backport v3.13.x

✅ Backports have been created

michaelklishin added a commit that referenced this pull request Jul 1, 2024
michaelklishin added a commit that referenced this pull request Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants