-
Notifications
You must be signed in to change notification settings - Fork 101
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
Docs: Enhance immutable snapshots guide with detailed instructions #811
Docs: Enhance immutable snapshots guide with detailed instructions #811
Conversation
Revise 'enabling_immutable_snapshots.md' to provide a comprehensive guide on enabling immutable snapshots in `etcd-backup-restore`. - Add a Terminology section for clarity and consistent use of terms. - Include detailed instructions for configuring bucket-level immutability on Google Cloud Storage (GCS) and Azure Blob Storage (ABS). - Elaborate on best practices. - Improve grammar and readability throughout the document.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much for the documentation @seshachalam-yv!
- Changed 'Retention Period' to 'Immutability Period'. - Removed reference to future AWS S3 support.
Thanks @renormalize for reviewing. I've addressed all your review comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @seshachalam-yv!
I've pointed out one comment again which you'd missed. Other than that, the doc looks great.
What this PR does / why we need it:
This PR enhances the documentation for enabling immutable snapshots in
etcd-backup-restore
. It provides a comprehensive guide with detailed instructions for configuring bucket-level immutability on Google Cloud Storage (GCS) and Azure Blob Storage (ABS). The updated guide includes:Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
docs/usage/immutable_snapshots.md
has been revised and renamed todocs/usage/enabling_immutable_snapshots.md
to better reflect its content.cc: @renormalize @ishan16696
Release note: