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

loqrecovery: deflake TestRetrieveApplyStatus #101295

Merged
merged 1 commit into from
Apr 12, 2023

Conversation

aliher1911
Copy link
Contributor

Test expects fully upreplicated 5 node cluster while TestCluster does not guarantee 5 replica config to propagate and upreplciate before proceeding. This fix adds explicit check during startup to wait appropriate cluster state.

Release note: None

Fixes #98623

Test expects fully upreplicated 5 node cluster while TestCluster does
not guarantee 5 replica config to propagate and upreplciate before
proceeding. This fix adds explicit check during startup to wait
appropriate cluster state.

Release note: None
@aliher1911 aliher1911 requested a review from a team April 12, 2023 10:56
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@aliher1911 aliher1911 added the backport-23.1.x Flags PRs that need to be backported to 23.1 label Apr 12, 2023
@aliher1911 aliher1911 self-assigned this Apr 12, 2023
@aliher1911
Copy link
Contributor Author

bors r=erikgrinaker

@craig
Copy link
Contributor

craig bot commented Apr 12, 2023

Build succeeded:

@craig craig bot merged commit bce125d into cockroachdb:master Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kv/kvserver/loqrecovery: TestRetrieveApplyStatus failed
3 participants