Update Parity Swap process description (to clarify disk replacement order)#495
Update Parity Swap process description (to clarify disk replacement order)#495DigicoolThings wants to merge 1 commit into
Conversation
Clarify the Parity Swap process description to correctly state that the new parity disk is installed first (previous wording indicated the new parity was installed as the second step).
📝 WalkthroughWalkthroughA single-line documentation update clarifies the sequence of operations in the parity swap procedure: the new larger disk is installed as parity first, then the previous parity disk is moved to a data slot. ChangesParity Swap Procedure Clarification
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@docs/unraid-os/using-unraid-to/manage-storage/array/replacing-disks-in-array.mdx`:
- Line 208: The partial
docs/unraid-os/using-unraid-to/manage-storage/partials/parity-swap-what.mdx
still describes the old sequence for a parity swap (saying it "moves your
current parity disk to the data slot, then installs a new, larger disk as the
new parity disk"); update that sentence to match the corrected sequence used in
replacing-disks-in-array.mdx (describe that the process installs a new larger
disk as the new parity disk first, then installs the previous parity disk into a
data slot), keeping wording consistent with the phrasing in
replacing-disks-in-array.mdx around "installs a new, larger disk as the new
parity disk, then installs your previous parity disk to the data slot."
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 0304e0ec-f1e0-440e-9be2-526e9b78792d
📒 Files selected for processing (1)
docs/unraid-os/using-unraid-to/manage-storage/array/replacing-disks-in-array.mdx
| ## Parity swap | ||
|
|
||
| Parity swap is a special procedure used when you need to replace a data disk with a disk that is larger than your current parity disk. The process moves your current parity disk to the data slot, then installs a new, larger disk as the new parity disk. This ensures your array remains protected and allows for larger data drives in the future. | ||
| Parity swap is a special procedure used when you need to replace a data disk with a disk that is larger than your current parity disk. The process installs a new, larger disk as the new parity disk, then installs your previous parity disk to the data slot. This ensures your array remains protected and allows for larger data drives in the future. |
There was a problem hiding this comment.
Inconsistency with shared partial needs resolution.
The updated sequence description here is correct and aligns with both the procedure steps (lines 254-256) and the example scenario (line 212). However, the shared partial at docs/unraid-os/using-unraid-to/manage-storage/partials/parity-swap-what.mdx still contains the old sequence: "moves your current parity disk to the data slot, then installs a new, larger disk as the new parity disk."
Please update the shared partial to match this corrected sequence to maintain consistency across the documentation.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
`@docs/unraid-os/using-unraid-to/manage-storage/array/replacing-disks-in-array.mdx`
at line 208, The partial
docs/unraid-os/using-unraid-to/manage-storage/partials/parity-swap-what.mdx
still describes the old sequence for a parity swap (saying it "moves your
current parity disk to the data slot, then installs a new, larger disk as the
new parity disk"); update that sentence to match the corrected sequence used in
replacing-disks-in-array.mdx (describe that the process installs a new larger
disk as the new parity disk first, then installs the previous parity disk into a
data slot), keeping wording consistent with the phrasing in
replacing-disks-in-array.mdx around "installs a new, larger disk as the new
parity disk, then installs your previous parity disk to the data slot."
Clarify the Parity Swap process description to correctly state that the new parity disk is installed first (previous wording indicated the new parity was installed as the second step).
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit