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

roachtest: deflake replicagc-changed-peers #99020

Merged
merged 1 commit into from
Mar 23, 2023

Conversation

aliher1911
Copy link
Contributor

@aliher1911 aliher1911 commented Mar 20, 2023

Test doesn't give enough time for allocator to see that node is dead and move replicas from killed node. This PR changes retry timeout and interval and adds extra logging for visibility.

Release note: None

Fixes #98966

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@aliher1911 aliher1911 self-assigned this Mar 20, 2023
@aliher1911 aliher1911 requested a review from a team March 20, 2023 13:34
@aliher1911
Copy link
Contributor Author

Let's fix master and backport to branch where actual test failure happened. Change seem applicable to both.

@aliher1911 aliher1911 force-pushed the deflage_replicagc_roachtest branch 2 times, most recently from e646d59 to 73d970d Compare March 20, 2023 18:26
@aliher1911 aliher1911 marked this pull request as ready for review March 20, 2023 18:28
@aliher1911 aliher1911 requested a review from a team as a code owner March 20, 2023 18:28
@aliher1911 aliher1911 requested review from srosenberg, smg260, a team and tbg and removed request for a team March 20, 2023 18:28
@aliher1911 aliher1911 force-pushed the deflage_replicagc_roachtest branch 3 times, most recently from 02a019a to 3a8d684 Compare March 21, 2023 10:49
Test doesn't give enough time for allocator to see that node is dead
and move replicas from killed node. This PR changes retry timeout
and interval and adds extra logging for visibility.

Release note: None
@aliher1911
Copy link
Contributor Author

bors r=tbg

@craig
Copy link
Contributor

craig bot commented Mar 23, 2023

Build succeeded:

@erikgrinaker
Copy link
Contributor

blathers backport 23.1

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.

roachtest: replicagc-changed-peers/restart=false failed
4 participants