Skip to content

roachtest: stabilize splits/largerange/size=32GiB,nodes=6#63359

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
aayushshah15:20210408_stabilizeLargeSplits
Apr 9, 2021
Merged

roachtest: stabilize splits/largerange/size=32GiB,nodes=6#63359
craig[bot] merged 1 commit intocockroachdb:masterfrom
aayushshah15:20210408_stabilizeLargeSplits

Conversation

@aayushshah15
Copy link
Copy Markdown
Contributor

@aayushshah15 aayushshah15 commented Apr 9, 2021

This commit increases the snapshot recovery rate of the test cluster,
which is important in cases where one of the followers of a large range
falls behind the rest of the group and needs a raft snapshot.

Additionally, it also adds a 100 second grace period to when the test
waits for the large range to split into sub-ranges of the expected size.

Fixes #61541

Release note: None

This commit increases the snapshot recovery rate, which is important in
cases where one of the followers of a large range falls behind the rest
of the group and needs a raft snapshot.

Additionally, it also adds a 100 second grace period to when the test
waits for the large range to split into sub-ranges of the expected size.

Fixes cockroachdb#61541

Release note: None
@aayushshah15 aayushshah15 requested a review from nvb April 9, 2021 01:07
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@aayushshah15
Copy link
Copy Markdown
Contributor Author

I do still plan on adding at least one variant that has non-voters but I want to stress it on my end before I add it. I'll get to that soon™.

Copy link
Copy Markdown
Contributor

@nvb nvb left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @aayushshah15)

@aayushshah15
Copy link
Copy Markdown
Contributor Author

TFTR

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 9, 2021

Build succeeded:

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: splits/largerange/size=32GiB,nodes=6 failed [stuck in kv]

3 participants