Skip to content

docs: correct read-only volume wording - #25614

Merged
dvdksn merged 1 commit into
docker:mainfrom
dvdksn:codex/fix-issue-11159-volume-wording
Jul 21, 2026
Merged

dvdksn merged 1 commit into
docker:mainfrom
dvdksn:codex/fix-issue-11159-volume-wording

Conversation

@dvdksn

@dvdksn dvdksn commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Correct the read-only volume section so it refers to writing to a volume rather than a bind mount. Remove the unrelated claim that changes propagate to the Docker host.

@netlify /engine/storage/volumes/

Preview the updated Volumes page

Closes #11159

Generated by Codex

The read-only volume section incorrectly described the mount as a bind mount.

Refer to the mounted data as a volume and remove the host propagation claim.

Co-Authored-By: Codex <noreply@openai.com>
@netlify

netlify Bot commented Jul 21, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit ae96cdf
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a5f4a17be76c000089f4ef0
😎 Deploy Preview https://deploy-preview-25614--docsdocker.netlify.app/engine/storage/volumes/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn dvdksn added the status/review Pull requests that are ready for review label Jul 21, 2026
@github-actions github-actions Bot added area/engine Issue affects Docker engine/daemon area/storage Relates to storage, volumes labels Jul 21, 2026
@dvdksn
dvdksn requested a review from a team July 21, 2026 10:29

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The change correctly fixes the documentation to say that containers cannot write to the volume when it is mounted read-only, and removes the inaccurate claim about changes propagating back to the Docker host. The edit is accurate, well-scoped, and free of style-guide violations.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

The change correctly replaces the inaccurate "bind mount" reference with "volume" and removes the unrelated claim that changes propagate back to the Docker host. The two new lines are accurate, concise, and consistent with the Docker documentation style guide — no hedge words, no redundant phrases, no marketing language.

No issues found.

@dvdksn
dvdksn merged commit a314f40 into docker:main Jul 21, 2026
17 checks passed
@dvdksn
dvdksn deleted the codex/fix-issue-11159-volume-wording branch July 21, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/engine Issue affects Docker engine/daemon area/storage Relates to storage, volumes status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix volumes document mention of readonly bind mounts

3 participants