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.

Sign up for GitHub

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

Jump to bottom

Emphasize the importance of having snapshot #8051

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
+2 −0
Open

Emphasize the importance of having snapshot #8051

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2233b96
Emphasize the importance of having snapshot
kunisen Sep 13, 2024
202d9ce
Update snapshots.asciidoc
kunisen Sep 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions docs/orchestrating-elastic-stack-applications/elasticsearch/snapshots.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ endif::[]
[id="{p}-{page_id}"]
= Create automated snapshots

NOTE: Snapshots are important to ensure that your Elasticsearch indices can be recovered in the event of an accidental deletion, or to migrate data across clusters.
Copy link
Contributor

Choose a reason for hiding this comment

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

It's up to you to decide, but here's another suggestion that's a little more direct and concise.

Suggested change
NOTE: Snapshots are important to ensure that your Elasticsearch indices can be recovered in the event of an accidental deletion, or to migrate data across clusters.
NOTE: Snapshots are essential for recovering Elasticsearch indices in case of accidental deletion or for migrating data between clusters.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have this note as a header of the "Run Elasticsearch on ECK" section here. I'm asking because my feeling is that if someone is looking at the snapshot documentation then it may mean that this person is already aware that it's an important topic (or maybe because it's too late 🙃 )


To set up automated snapshots for Elasticsearch on Kubernetes you have to:

. Register the snapshot repository with the Elasticsearch API.
Expand Down