Remove all mention of RAID from docs #21163
Conversation
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
|
once the v25.4 update is reviewed I will backport the change to all supported versions v23.2+ |
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
jhlodin
left a comment
jhlodin
left a comment
There was a problem hiding this comment.
Left a suggestion across two files.
Do we need to explicitly say that RAID is not supported? For the paragraph in recommended-production-settings.md in particular I could see a customer reading "this is recommended if you are using local disks... because they have a greater risk of failure" and going "oh well I'll just set them up in a RAID array for redundancy." Unless nobody's doing RAID for this type of infrastructure anymore - I'd ask support or a PM.
src/current/_includes/v25.4/prod-deployment/topology-recommendations.md
Outdated
Show resolved
Hide resolved
Thanks, updated!
For this PR I'd rather get this change in sooner and do a couple fast-follow PRs to address the language around that
any objections to getting this one in ASAP and doing the above followup work in other PRs? |
jhlodin
left a comment
jhlodin
left a comment
There was a problem hiding this comment.
LGTM. Would like to see what product recommendations are for "don't recommend RAID" vs "explicitly discourage RAID as unsupported" but this works for now.
Fixes DOC-15399 Summary of changes: - Remove all mentions of RAID from various places - Also update the "don't use LVM" guidance to point users to multi-store instead NB. These changes are backported to all supported versions v23.2+
rmloveland
force-pushed
the
20251117-DOC-15399
branch
from
f98ea71 to
241b8d6
Compare
3 participants
Fixes DOC-15399
Summary of changes:
Remove all mentions of RAID from various places
Also update the "don't use LVM" guidance to point users to multi-store instead