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

[7.x] Add snapshot stress tests (#78596) #78611

Merged
merged 3 commits into from
Oct 18, 2021

Conversation

DaveCTurner
Copy link
Contributor

Backports the following commits to 7.x:

@DaveCTurner DaveCTurner added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Oct 4, 2021
@original-brownbear
Copy link
Member

@DaveCTurner I still have some open backports for the snapshot state machine, this one is probably blocked on those. Working through them today. I can merge this one afterwards!

Adds `SnapshotStressTestsIT`, a test that performs a wide variety of
concurrent snapshot-related operations to explore the corners of the
snapshot state machine in a randomized fashion:

- indexing docs, deleting and re-creating the indices
- restarting nodes
- removing and adding repositores
- taking snapshots (sometimes partial), cloning them, and deleting them

It ensures that these operations should succeed via a system of
shared/exclusive locks. None of the operations block. If the necessary
locks aren't all available then the operation just releases the ones it
has acquired and tries again later. The test completes after completing
a certain number of snapshots or after a certain time has elapsed.
@DaveCTurner DaveCTurner force-pushed the backport/7.x/pr-78596 branch from 8552d0e to a832fc1 Compare October 4, 2021 12:37
@DaveCTurner DaveCTurner removed the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 4, 2021
@DaveCTurner
Copy link
Contributor Author

Compile failures are the usual stuff, I force-pushed a fixed backport. It'll likely merge fine but may fail until everything else is backported so I'll leave this for you to merge when ready.

@DaveCTurner
Copy link
Contributor Author

@original-brownbear is this still blocked on anything?

@original-brownbear
Copy link
Member

@DaveCTurner yea but it's on my list for today afternoon :) Sorry, busy times :(

@original-brownbear
Copy link
Member

@elasticmachine update branch

@original-brownbear
Copy link
Member

@DaveCTurner sorry for the delay here, this should be just to merge now. All backported at last :)

@DaveCTurner DaveCTurner merged commit 61b0c3c into elastic:7.x Oct 18, 2021
@DaveCTurner DaveCTurner deleted the backport/7.x/pr-78596 branch October 18, 2021 07:42
@DaveCTurner
Copy link
Contributor Author

Great, I've done that

DaveCTurner added a commit that referenced this pull request Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants