-
Notifications
You must be signed in to change notification settings - Fork 162
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(backup): remote backup data might be deleted #1004
Conversation
✅ Deploy Preview for longhornio ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (5)
content/docs/1.6.4/deploy/important-notes/index.md (2)
21-21
: Approved: Improved clarity on backup data risksThe changes in this section significantly improve the documentation by clearly outlining the potential risks associated with backup operations. The new title and content accurately reflect the issue mentioned in the PR objectives (longhorn/longhorn#9530).
To further enhance clarity, consider adding a brief explanation of what a "race condition" is, as some users might not be familiar with this technical term.
Also applies to: 227-235
227-235
: Approved: Clearer explanation of XFS filesystem size restrictionsThe updates to this section provide more precise information about the minimum XFS filesystem size restrictions. The addition of specific scenarios (CSI flow and UI actions) greatly improves clarity for users.
To make the information even more comprehensive, consider adding a brief explanation of why this restriction exists (e.g., due to recent versions of
xfsprogs
). This context could help users understand the technical reason behind the limitation.content/docs/1.7.2/important-notes/_index.md (1)
211-220
: Approve changes with a suggestion for clarityThe updates to the backup section provide crucial information about potential data loss scenarios and how Longhorn v1.7.2 handles them. This is valuable for users to understand the risks and behaviors associated with backups.
Consider adding a brief explanation of what "BackupVolume, BackupBackingImage, SystemBackup, and Backup" custom resources are, as this might not be immediately clear to all users. For example:
-Longhorn will attempt to clean up the backup-related custom resources (BackupVolume, BackupBackingImage, SystemBackup, and Backup) in the following scenarios: +Longhorn will attempt to clean up the backup-related custom resources (such as BackupVolume, BackupBackingImage, SystemBackup, and Backup, which store metadata about backups) in the following scenarios:content/docs/1.8.0/important-notes/_index.md (2)
215-224
: Improved clarity on custom resource and backup data deletion scenarios.The updated section provides more detailed information about the potential deletion of custom resources and backup data. This is valuable for users to understand the risks and behaviors.
Consider adding a brief explanation of what a "race condition" is for users who might not be familiar with the term. For example:
- There is a race condition that could potentially delete the remote backup volume and its corresponding backups when resetting the backup target within a short period. + There is a race condition (a timing-sensitive issue) that could potentially delete the remote backup volume and its corresponding backups when resetting the backup target within a short period.
217-217
: New feature announcement for disaster recovery volumes.The addition of disaster recovery volumes support is an important note for users. However, consider expanding this section to provide more context and value to users.
Consider adding more details, such as:
- A brief explanation of what disaster recovery volumes are.
- The benefits of this feature.
- A link to more comprehensive documentation about this feature, if available.
For example:
### Disaster Recovery Volumes Disaster recovery volumes are supported from Longhorn v1.8.0. These volumes allow you to create standby copies of your data in a separate cluster, enhancing your ability to recover from major outages or disasters. For more information on setting up and using disaster recovery volumes, see [Disaster Recovery Volumes Documentation](link-to-detailed-docs).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🧰 Additional context used
🔇 Additional comments (3)
content/docs/1.6.4/deploy/important-notes/index.md (1)
Line range hint
1-235
: Overall: Excellent improvements to documentation clarity and accuracyThe changes in this file significantly enhance the documentation for Longhorn v1.6.4. The updates to the "Backup Related Custom Resources" and "Minimum XFS Filesystem Size" sections provide more detailed and accurate information, addressing potential risks and limitations.
These improvements align well with the PR objectives and the AI-generated summary, focusing on clarifying potential data loss scenarios and updating information on data integrity and reliability. The changes will help users better understand the implications of certain operations and the limitations of the system.
content/docs/1.8.0/important-notes/_index.md (2)
32-32
: Table of contents update looks good.The new entry "Related Custom Resources In The Cluster And Backup Data On The Remote Backup Server Might Be Deleted" accurately reflects the updated section title and is correctly placed in the document structure.
Line range hint
1-224
: Overall improvements to the Important Notes document.The changes in this file enhance the clarity and completeness of the Longhorn v1.8.0 documentation. The additions and modifications provide users with valuable information about potential data deletion scenarios and new features. The document structure remains consistent, and the new content integrates well with the existing information.
f387f02
to
0277a41
Compare
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
content/docs/1.7.2/important-notes/_index.md (1)
211-220
: Expanded information on potential data loss scenarios and improved behaviorThe updated section provides crucial details about scenarios where backup-related custom resources might be cleaned up, including a newly mentioned race condition. The addition of information about the improved behavior in Longhorn v1.7.2, where remote backup data will not be deleted and information will be resynchronized, is a significant improvement.
Consider adding a brief explanation of what "resynchronized during the next polling interval" means for users who might not be familiar with the polling mechanism. This could help clarify the expected behavior and timing of the resynchronization process.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- content/docs/1.6.4/deploy/important-notes/index.md
- content/docs/1.8.0/important-notes/_index.md
🧰 Additional context used
🔇 Additional comments (2)
content/docs/1.7.2/important-notes/_index.md (2)
31-32
: Improved section title clarity in table of contentsThe updated title "Related Custom Resources In The Cluster And Backup Data On The Remote Backup Server Might Be Deleted" provides a more comprehensive and accurate description of the section's content. This change enhances the document's navigability and sets clearer expectations for readers.
Line range hint
1-220
: Summary: Improved documentation on backup data handlingThe changes in this file effectively address the PR objective related to remote backup data potentially being deleted. The updates provide more detailed and accurate information about potential data loss scenarios and the improved behavior in Longhorn v1.7.2. These changes enhance the overall clarity and completeness of the documentation, offering valuable insights to users about the handling of backup-related custom resources and remote backup data.
0277a41
to
f882016
Compare
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
content/docs/1.7.2/important-notes/_index.md (1)
31-31
: Improved clarity on backup data deletion scenarios. Consider adding a warning box.The changes in this section significantly improve the documentation by providing more specific information about potential backup data deletion scenarios. The new title is more accurate and attention-grabbing, which is crucial for user awareness.
Consider wrapping the entire section in a warning box to make it even more noticeable. For example:
> **Warning** > ### Backup Data On The Remote Backup Server Might Be Deleted > > [Rest of the content...]This will help ensure that users don't miss this critical information.
Also applies to: 211-220
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- content/docs/1.6.4/deploy/important-notes/index.md
- content/docs/1.8.0/important-notes/_index.md
🧰 Additional context used
🔇 Additional comments (1)
content/docs/1.7.2/important-notes/_index.md (1)
Line range hint
1-220
: Comprehensive update with valuable new information. Well done!The document has been significantly improved with the addition of several new sections covering important features and improvements in Longhorn v1.7.2. The new content is well-organized and provides clear explanations of the latest functionalities.
These additions will greatly help users understand the new capabilities and potential considerations when using Longhorn v1.7.2. Great job on keeping the documentation up-to-date and informative!
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.
Review done
f882016
to
f5691f5
Compare
WalkthroughThe pull request involves updates to documentation for Longhorn across versions 1.6.0, 1.7.2, and 1.8.0. Key changes include renaming sections for clarity, enhancing descriptions of potential data loss, and specifying restrictions related to XFS filesystem sizes. New details have been added regarding Kubernetes compatibility, backup reliability, resource management settings, and networking limitations. Overall, the modifications improve user understanding of Longhorn's features and behaviors. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Longhorn
participant BackupServer
User->>Longhorn: Request volume creation
Longhorn->>User: Check XFS filesystem size
alt Size < 300 MiB
Longhorn-->>User: Error: Minimum size requirement not met
else Size >= 300 MiB
Longhorn->>BackupServer: Create backup volume
Longhorn->>User: Volume created successfully
end
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (2)
🧰 Additional context used🔇 Additional comments (4)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (4)
content/docs/1.7.2/important-notes/_index.md (4)
211-221
: Improved clarity on potential backup data lossThe renaming of this section and the added content significantly improve the documentation by:
- Providing a more accurate and attention-grabbing title.
- Detailing specific scenarios where backup data might be unintentionally deleted.
- Explaining Longhorn's behavior in v{{< current-version >}} to mitigate these issues.
These changes will help users better understand and prepare for potential risks to their backup data.
Consider adding a brief explanation of what "backup-related custom resources" are for users who might not be familiar with the term. For example:
"Longhorn may unintentionally delete backup-related custom resources (such as BackupVolume, BackupBackingImage, SystemBackup, and Backup, which are Kubernetes objects used to manage and track backups) and backup data on the remote backup server in the following scenarios:"
223-223
: Added reference to GitHub issueThe addition of the GitHub issue reference is helpful for users who want to dive deeper into the technical details of this problem.
Consider adding a brief description of what users can find in the GitHub issue. For example:
"For more information, including technical details and ongoing discussions about this issue, see #9530."
Line range hint
225-265
: Comprehensive updates to V2 Data Engine sectionThe additions to the V2 Data Engine section greatly enhance the documentation by covering various important aspects:
- Upgrade instructions and prerequisites
- Kernel module requirements
- New features and improvements
- Known issues and limitations
These updates will help users better understand and utilize the V2 Data Engine.
Consider the following improvements:
In the "Longhorn System Upgrade" section, add a note about the potential downtime implications of detaching all V2 volumes before upgrading.
In the "Block-type Disk Supports SPDK AIO, NVMe and VirtIO Bdev Drivers" section, provide a brief explanation of the performance benefits of using NVMe or VirtIO bdev drivers over AIO.
In the "Filesystem Trim" section, the reference "(ref)" seems to be incomplete. Please add the actual reference or remove it if not needed.
In the "Linux Kernel on Longhorn Nodes" section, consider adding a brief explanation of why kernel version 6.7 or later is recommended for improved system stability.
For the last two sections about snapshot-related issues, consider adding a note about any planned fixes or workarounds for future versions.
Line range hint
1-265
: Comprehensive and well-structured important notesThe updated document provides a wealth of valuable information for Longhorn v1.7.2 users. The structure is logical, and the content covers a wide range of important topics, including:
- Deprecations
- General notes and requirements
- Resilience improvements
- Data integrity and reliability enhancements
- Scheduling features
- Networking changes
- Backup considerations
- V2 Data Engine details and known issues
This comprehensive update will greatly assist users in understanding and effectively using Longhorn v1.7.2.
For consistency, consider adding brief introductory sentences at the beginning of each main section (e.g., "General", "Resilience", "Data Integrity and Reliability", etc.) to provide context for the subsections that follow.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- content/docs/1.6.4/deploy/important-notes/index.md
- content/docs/1.8.0/important-notes/_index.md
🧰 Additional context used
🔇 Additional comments (1)
content/docs/1.7.2/important-notes/_index.md (1)
31-31
: Important addition to the table of contentsThe new section "Backup Data On The Remote Backup Server Might Be Deleted" is a crucial addition to the important notes. This highlights a potential data loss scenario that users should be aware of.
f5691f5
to
b3475c5
Compare
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (5)
content/docs/1.6.4/deploy/important-notes/index.md (1)
227-238
: LGTM: Clear explanation of backup data deletion risk and improvementsThis new section effectively addresses the concerns about backup data deletion and clearly explains the improvements made in v1.6.4. It provides valuable information for users about the potential risks in previous versions and the safeguards implemented in the current version.
Consider adding a comma in line 236 for improved readability:
- Longhorn only deletes the backup-related custom resources in the cluster and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server. + Longhorn only deletes the backup-related custom resources in the cluster, and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server.🧰 Tools
🪛 LanguageTool
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.7.2/important-notes/_index.md (1)
211-222
: Approved: Clear and informative update on backup data managementThe changes in this section provide valuable information about potential backup data deletion scenarios and improvements in Longhorn v1.7.2. The new title accurately reflects the content, and the added details help users understand the risks and new behaviors.
Consider adding a brief explanation of how users can manually trigger the resynchronization of backup-related custom resources if needed, rather than waiting for the next polling interval.
content/docs/1.8.0/important-notes/_index.md (3)
215-217
: LGTM: Improved section title and descriptionThe updated section title and expanded description provide more clarity on the potential issue with backup data deletion. This change effectively communicates the scope of the problem to users.
Consider adding "unintentionally" to the section title for even more clarity:
-### Backup Data On The Remote Backup Server Might Be Deleted +### Backup Data On The Remote Backup Server Might Be Unintentionally Deleted
222-226
: LGTM: Clear explanation of new backup data handlingThe added information clearly explains the changes in Longhorn v1.8.0 regarding backup data handling. It provides important details about data safety, manual deletion options, and the resynchronization process.
Consider adding a comma in line 224 for improved readability:
-Longhorn only deletes the backup-related custom resources in the cluster and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server. +Longhorn only deletes the backup-related custom resources in the cluster, and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server.🧰 Tools
🪛 LanguageTool
[uncategorized] ~224-~224: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
228-228
: LGTM: Added support for disaster recovery volumesThe addition of information about disaster recovery volumes support is valuable. However, it could benefit from more context.
Consider expanding this note to provide more context about disaster recovery volumes and their significance. For example:
### Disaster Recovery Volumes Disaster recovery volumes, which allow for data replication and quick recovery in case of catastrophic failures, are supported from Longhorn v1.8.0. This feature enhances Longhorn's ability to provide robust data protection and business continuity solutions.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🧰 Additional context used
🪛 LanguageTool
content/docs/1.6.4/deploy/important-notes/index.md
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.8.0/important-notes/_index.md
[uncategorized] ~224-~224: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
🔇 Additional comments (7)
content/docs/1.6.4/deploy/important-notes/index.md (3)
21-21
: LGTM: Updated table of contentsThe addition of "Backup Data On The Remote Backup Server Might Be Deleted" to the table of contents accurately reflects the new section and aligns with the PR objectives.
229-232
: LGTM: Additional context and informationThe addition of "before Longhorn v{{< current-version >}}" in line 229 clarifies that the issue was present in previous versions. The new bullet point about the race condition (line 232) provides valuable information about a specific scenario that could lead to data loss.
These additions enhance the overall clarity and completeness of the documentation.
Line range hint
21-238
: Overall: Excellent improvements to documentationThe changes made in this file significantly enhance the documentation by:
- Adding a clear and informative section on the risk of backup data deletion.
- Explaining the improvements made in v1.6.4 to prevent unintentional data loss.
- Providing specific scenarios that could lead to data loss in previous versions.
- Clarifying the behavior of backup-related custom resources after the fix.
These updates align well with the PR objectives and address the concerns raised in past review comments. The documentation now provides users with valuable information about potential risks and the safeguards implemented in the current version.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.7.2/important-notes/_index.md (2)
223-223
: Approved: Good practice to include GitHub issue referenceIncluding the reference to GitHub issue #9530 is helpful for users who want to dive deeper into the background of these changes.
211-223
: Overall: Excellent update to the backup documentationThe changes in this section significantly improve the documentation by:
- Providing a more accurate and specific section title.
- Clearly explaining potential data deletion scenarios.
- Detailing the improvements in Longhorn v1.7.2 for handling backup-related issues.
- Offering guidance on manual deletion of remote backup data.
- Explaining the resynchronization process for backup-related custom resources.
These updates will help users better understand and manage their Longhorn backups.
content/docs/1.8.0/important-notes/_index.md (2)
32-32
: LGTM: New section added to table of contentsThe addition of "Backup Data On The Remote Backup Server Might Be Deleted" to the table of contents is appropriate and consistent with the new content in the document.
220-221
: LGTM: Added important scenario for potential data lossThe addition of the race condition scenario during backup target reset is valuable information for users. It helps them understand specific circumstances that could lead to unintended data deletion.
b3475c5
to
af1e53d
Compare
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (4)
content/docs/1.6.4/deploy/important-notes/index.md (1)
229-237
: LGTM: Clear explanation of backup data deletion issue and improvementsThis new section effectively addresses the PR objectives by explaining the potential issues with backup data deletion and the improvements made in Longhorn v1.6.4. The content provides valuable information for users about the behavior changes and safeguards implemented.
Consider adding a comma in line 236 for improved clarity:
- Longhorn only deletes the backup-related custom resources in the cluster and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server. + Longhorn only deletes the backup-related custom resources in the cluster, and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server.🧰 Tools
🪛 LanguageTool
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.7.2/important-notes/_index.md (1)
211-221
: Improved clarity on potential backup data deletion issues.The updated section provides clear and important information about scenarios where backup data might be unintentionally deleted and the improvements made in v1.7.2 to address these issues. This is crucial information for users and administrators.
Consider adding a comma in line 220 for improved readability:
- Longhorn only deletes the backup-related custom resources in the cluster and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server. + Longhorn only deletes the backup-related custom resources in the cluster, and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server.🧰 Tools
🪛 LanguageTool
[uncategorized] ~220-~220: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.8.0/important-notes/_index.md (2)
222-225
: Important clarification on improved backup handlingThis addition provides crucial information about Longhorn's improved behavior in the current version:
- Longhorn will not delete data on the remote backup server in case of discrepancies between the cluster and the remote backup server.
- This change enhances data safety and reduces the risk of unintended data loss.
These improvements in backup handling are significant and should be highlighted to users.
Consider adding a comma in the following sentence for improved readability:
- Longhorn only deletes the backup-related custom resources in the cluster and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server. + Longhorn only deletes the backup-related custom resources in the cluster, and does not delete anything on the remote backup server if there is a discrepancy between the backup information in the cluster and that on the remote backup server.🧰 Tools
🪛 LanguageTool
[uncategorized] ~224-~224: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
227-227
: New feature announcement: Disaster recovery volumesThe addition of support for disaster recovery volumes in Longhorn v1.8.0 is a significant feature that users should be aware of.
Consider expanding this note slightly to provide more context:
- Disaster recovery volumes are supported from Longhorn v1.8.0. + Disaster recovery volumes are a new feature supported from Longhorn v1.8.0. This enhancement improves the system's ability to recover from catastrophic failures.This expansion would give users a better understanding of the feature's importance and its role in the system.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (3)
- content/docs/1.6.4/deploy/important-notes/index.md (2 hunks)
- content/docs/1.7.2/important-notes/_index.md (2 hunks)
- content/docs/1.8.0/important-notes/_index.md (2 hunks)
🧰 Additional context used
🪛 LanguageTool
content/docs/1.6.4/deploy/important-notes/index.md
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.7.2/important-notes/_index.md
[uncategorized] ~220-~220: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.8.0/important-notes/_index.md
[uncategorized] ~224-~224: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
🔇 Additional comments (8)
content/docs/1.6.4/deploy/important-notes/index.md (4)
21-21
: LGTM: New section added to table of contentsThe addition of "Backup Data On The Remote Backup Server Might Be Deleted" to the table of contents is appropriate and aligns with the PR objectives. This change effectively highlights the important issue addressed in this update.
227-228
: LGTM: Clarified XFS filesystem size limitationsThe update provides clear and important information about Longhorn's restrictions on XFS filesystem sizes. This change enhances user understanding of potential limitations when working with Longhorn volumes.
239-239
: LGTM: Added reference to GitHub issueThe inclusion of the link to the GitHub issue (#9530) provides a valuable reference for users seeking more detailed information about the backup data deletion issue. This addition enhances the documentation's completeness.
Line range hint
21-239
: Overall: Excellent documentation updateThe changes in this file effectively address the PR objectives by providing clear and detailed information about the potential deletion of remote backup data. The new section "Backup Data On The Remote Backup Server Might Be Deleted" is particularly valuable, explaining both the issue and the improvements made in Longhorn v1.6.4.
The updates to the "Minimum XFS Filesystem Size" section and the addition of the GitHub issue reference further enhance the documentation's completeness and usefulness for Longhorn users.
These changes significantly improve the documentation and will help users better understand and manage their Longhorn deployments.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~236-~236: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.7.2/important-notes/_index.md (1)
Line range hint
1-279
: Comprehensive and informative updates to the important notes.The changes made to this document significantly improve its clarity and provide crucial information for Longhorn v1.7.2 users. The updates cover a wide range of topics, including backup data management, networking, scheduling, and data integrity. These changes will help users better understand and utilize Longhorn's features and be aware of potential issues.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~220-~220: Possible missing comma found.
Context: ... backup-related custom resources in the cluster and does not delete anything on the rem...(AI_HYDRA_LEO_MISSING_COMMA)
content/docs/1.8.0/important-notes/_index.md (3)
32-32
: Important addition to the table of contentsThe new section "Backup Data On The Remote Backup Server Might Be Deleted" is a crucial addition to the table of contents. It highlights an important issue that users should be aware of regarding potential data loss scenarios.
215-217
: Improved section title and descriptionThe changes in this section significantly improve clarity and accuracy:
- The new title "Backup Data On The Remote Backup Server Might Be Deleted" more accurately reflects the content of the section.
- The updated description provides more specific information about the potential risks, including the types of custom resources that might be affected (BackupVolume, BackupBackingImage, SystemBackup, and Backup).
These changes will help users better understand the potential risks associated with backup data.
220-221
: Critical information about potential data loss scenarioThis new bullet point provides crucial information about a specific scenario that could lead to data loss:
- A race condition during backup target reset could result in the deletion of the remote backup volume and its corresponding backups.
This addition is essential for users to understand the potential risks and take necessary precautions when managing their backup targets.
af1e53d
to
0604ce3
Compare
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.
Second review done
ref: longhorn/longhorn 9530 Signed-off-by: James Lu <[email protected]>
0604ce3
to
e0bd571
Compare
Which issue(s) this PR fixes:
Issue # longhorn/longhorn#9530
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context
Summary by CodeRabbit
New Features
Documentation