Fix issue #24010: warn about containerd separate storage path#24447
Fix issue #24010: warn about containerd separate storage path#24447
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
This PR adds a valuable warning about containerd using a separate storage path from the Docker data directory. The content addresses a real user issue (filling root partitions when custom data directories are configured).
Issues found:
- 1 style guide violation (bold used for emphasis)
- 1 maintainability concern (external link stability)
|
Thank you for the feedback on both the style violation and link stability concern. You're right on both counts. I've made the following changes:
The warning now follows our documentation standards while maintaining a reliable reference. |
|
I would also suggest linking to: https://docs.docker.com/engine/daemon/#configure-the-data-directory-location |
|
Thanks for the suggestion, @SuperNinja-4965. The bullet list below the callout already links to the internal Docker docs page for configuring Docker's data directory. The callout's external link points specifically to the containerd configuration documentation, which covers configuring containerd's own data directory — a separate step from configuring Docker's data directory. The two links serve different purposes. Generated by Claude Code |
|
You're right, @SuperNinja-4965 — the Docker daemon docs page already covers configuring containerd's data directory. Updated the link to point to the internal docs instead. |
|
Sorry about that, my robot underling is not behaving this monday |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
36bc862 to
bd2e53f
Compare
Adds warning about containerd using a separate storage path from the Docker data directory.
Changes:
This prevents users from unexpectedly filling their root partition when they assumed their custom data directory configuration would apply to containerd.
Fixes #24010