Skip to content

Commit

Permalink
Merge pull request #12 from apollo13/patch-2
Browse files Browse the repository at this point in the history
Properly describe bind_propagation_non_recursive
  • Loading branch information
eveld committed Oct 31, 2023
2 parents e4bc05a + 83dfd63 commit 31c7512
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/pages/docs/resources/shared/volume.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@ the file declaring the container.

For more information please see the Docker documentation [https://docs.docker.com/storage/bind-mounts/#configure-bind-propagation](https://docs.docker.com/storage/bind-mounts/#configure-bind-propagation)
</Property>
<Property name="bind_propagation_non_recursive" type="bool" required="false" value="false">
We are going to be 100% honest, we have no idea what this option does, but it is a thing so we made it configuable.
Would love a PR if you actually know how this is supposed to work.

For more information please see the Docker documentation [https://docs.docker.com/storage/bind-mounts/#configure-bind-propagation](https://docs.docker.com/storage/bind-mounts/#configure-bind-propagation)
<Property name="bind_propagation_non_recursive" type="boolean" required="false" value="false">
Configures recursiveness of the bind mount. By default Docker mounts with the equivalent of `mount --rbind` meaning that mounts below the the source directory are visible in the container.
For instance running `docker run --rm --mount type=bind,src=/,target=/host,readonly busybox` will make `/run` of the host available as `/host/run` in the container -- and to make matters
even worse it will be writable (since only the toplevel bind is set readonly, not the children). If `bind_propagation_non_recursive` is set to `true` then the container will only see an
empty `/host/run`, meaning the `tmpfs` which is typically mounted to `/run` on the host is not propagated into the container.
</Property>
</Properties>
</Properties>

0 comments on commit 31c7512

Please sign in to comment.