[ECH] Add data loss warning admonition for found-snapshots repository#7251
Open
kunisen wants to merge 5 commits into
Open
[ECH] Add data loss warning admonition for found-snapshots repository#7251kunisen wants to merge 5 commits into
kunisen wants to merge 5 commits into
Conversation
Contributor
Elastic Docs AI PR menuCheck the box to run an AI review for this pull request.
Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team. |
Contributor
🔍 Preview links for changed docs |
Contributor
There was a problem hiding this comment.
Docs review summary
Focus areas
- Style and clarity: One minor preposition error in the new admonition ("in this page" → "on this page") — see inline comment. The rest of the added prose is clear, direct, and technically specific.
- Jargon: No unexplained jargon introduced.
found-snapshotsis quoted in monospace as expected. - Frontmatter and applies_to: No
descriptionfield in frontmatter — pre-existing gap, not introduced by this PR. - Content type fit: The
importantadmonition is well-placed and appropriate for a high-severity data-loss warning. - Parent issue satisfaction: Satisfied. The admonition prominently surfaces the no-backup, permanent-deletion, and searchable-snapshot caveat, and points users to the custom-repository path for redundancy.
Nits
- Vale flags several pre-existing issues in untouched lines: multiple uses of
may(should becanormight),just(line 17),older versions→earlier versions(line 45),disable(line 85), and wordiness in lines 133–144. These are all in content not touched by this PR.
Notes
mapped_pageswarning from Vale is expected for this file and does not require action.
Generated by Docs review agent for issue #7251 · 30.7 AIC · ⌖ 10.8 AIC · ⊞ 10.7K
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…-content into kuni-docpr-stl-1841
Contributor
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
andrew-moldovan
approved these changes
Jul 7, 2026
yetanothertw
reviewed
Jul 7, 2026
| From within {{ech}}, you can restore a snapshot from a different deployment in the same region. | ||
|
|
||
| ::::{important} | ||
| The underlying `found-snapshots` repository managed by {{ech}} has no backup. Elastic does not enable soft deletion at the cloud provider storage level. Any snapshot deletions — including [searchable snapshots](./searchable-snapshots.md#back-up-restore-searchable-snapshots) — are permanent and cannot be recovered by Elastic. |
Member
There was a problem hiding this comment.
I know this suggestion is longer but I think it adds a bit more context around why this is important. Please feel free to tweak it if anything is not accurate:
Suggested change
| The underlying `found-snapshots` repository managed by {{ech}} has no backup. Elastic does not enable soft deletion at the cloud provider storage level. Any snapshot deletions — including [searchable snapshots](./searchable-snapshots.md#back-up-restore-searchable-snapshots) — are permanent and cannot be recovered by Elastic. | |
| {{ech}} registers the built-in `found-snapshots` repository on every deployment and uses it along with the `cloud-snapshot-policy` SLM policy for automated snapshots. This repository is your backup on {{ech}}. Snapshot data exists only in this repository and {{ech}} does not mirror it to separate storage or retain deleted snapshots through storage-level versioning or soft delete. | |
| When you delete a snapshot from `found-snapshots`, that data is removed from the repository and cannot be recovered. This also applies to [searchable snapshots](searchable-snapshots.md#back-up-restore-searchable-snapshots) stored in `found-snapshots`. |
| ::::{important} | ||
| The underlying `found-snapshots` repository managed by {{ech}} has no backup. Elastic does not enable soft deletion at the cloud provider storage level. Any snapshot deletions — including [searchable snapshots](./searchable-snapshots.md#back-up-restore-searchable-snapshots) — are permanent and cannot be recovered by Elastic. | ||
|
|
||
| If you require additional redundancy, follow the guidance on this page to configure your own snapshot repository and take backups. |
Member
There was a problem hiding this comment.
A little more clarification + link:
Suggested change
| If you require additional redundancy, follow the guidance on this page to configure your own snapshot repository and take backups. | |
| If you need additional redundancy beyond the default repository, [configure a custom snapshot repository](#ess-repo-types) and take additional snapshots to storage you control. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes https://github.com/elastic/support-tech-lead/issues/1841
Adds a prominent
importantadmonition near the top of the ECH snapshotrepositories page to make clear that
found-snapshotshas no backup,deletions are permanent (including searchable snapshots), and the path to
redundancy is to configure a custom repository.
Note: coordinates with #7212
(Stef Nestor). If both PRs merge, the bullet added by #7212 under
Considerations could be removed to avoid duplication.
Generative AI disclosure
cc @stefnestor @jakommo