Add /var/lib/containers/storage/ to illegal mount validation. #27218
Conversation
Due to the virtiofs filesystem limitations, it cannot deal with multiple uids/gids that are required for the container storage. Fixes: containers#27183 Signed-off-by: Lili Nie <[email protected]>
openshift-ci
bot
added
the
do-not-merge/release-note-label-needed
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: skycastlelily The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
release-note
and removed
do-not-merge/release-note-label-needed
| - The `--mount type=artifact` option to `podman create`, `podman run`, and `podman pod create` now mounts artifacts containing a only a single blob as a file at the given destination path if the path does not exist in the image. | ||
| - The `podman volume export` command now refuses to export to `STDOUT` if it is a TTY ([#26506](https://github.com/containers/podman/issues/26506)). | ||
| - When generating Quadlet units with options known to be problematic when used with Podman, such as `User=`, `Group=`, and `DynamicUser=` in the `[Service]` section of a unit, Quadlet will now warn the user of the potential incompatibility ([#26543](https://github.com/containers/podman/issues/26543)). | ||
| - The `podman machine init` command now throws out errors if users try to mount `/var/lib/containers/storage/`, which podman is not able to support, due to virtiofs filesystem limitations with multiple UIDs/GIDs ([#27183](https://github.com/containers/podman/issues/27183)). |
There was a problem hiding this comment.
No need to add a release note here, just in the PR comment is fine.
| "/tmp": {}, | ||
| "/usr": {}, | ||
| "/var": {}, | ||
| "/var/lib/containers/storage/": {}, |
There was a problem hiding this comment.
Looks like our linter is complaining, could you run gofmt and fix the lint issues?
|
I would think some mention in docs (man-pages) would be a nth. If nothing else, something that says certain paths mounted into machines can be like this ... and maybe, for example in MacOs ... Thanks for the PR! |
Due to the virtiofs filesystem limitations, it cannot deal with multiple uids/gids that are required for the container storage.
Fixes: #27183