-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Add the fact that you'll need to find te snapshot before trying to re… #92941
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
base: 8.1
Are you sure you want to change the base?
Conversation
…store ir And it's document above.
@lhirlimann please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation. |
I'm failing to follow the doc :( |
Pinging @elastic/es-distributed (Team:Distributed) |
Thanks for the suggestion @lhirlimann! However, I don't think we should make this change. I would have expected that it is already clear enough that you do indeed need to choose a snapshot to restore as part of this process. Your PR suggests that this wasn't the case for you, in which case I think this needs to be called out much earlier on this page. The section about listing the available snapshots looks like a much better location for this info. |
Don't worry about it, it's not a big deal 🙂 |
I was following the numbers to restore a cluster - in I'm in a hurry mode , cluster is down, and I felt It would be nice to also state that there's a step to look for the proper snapshot to be searched and found. |
…store ir
And it's document above.
gradle check
?