-
Notifications
You must be signed in to change notification settings - Fork 21
pass flag required with podman and selinux #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you explain the rationale behind the change in the commit message?
I've seen this :Z
in the past and I think it was a workaround of an issue which was not the best solution and for which we found a better solution afterwards, but my memory might fail me.
CC @ydirson
ff8b3a0
to
5f045d1
Compare
with a default install of fedora 41 workstation, the container wasn't able to access the local directory mounted in the container. references: |
That does not sound unlike the discussion we had when I added podman support, but that was likely about another mount modifier.
Curiously only Docker seem to mention capital "Z"
That SE post makes me wonder why we would want that "private" behavior, we likely don't want any "last caller wins" situation. Lowercase "z" ought to be sufficient, right? |
|
The script can be run concurrently, to build several packages and/or an interactive sessions at the same time, and we don't want a new run to interfere with those previously launched but not terminated. |
--security-opt label=disable is more performant that using :Z at the end of the mount option. Signed-off-by: Gaëtan Lehmann <[email protected]> Co-authored-by: Gael Duperrey <[email protected]> Co-authored-by: Thierry Escande <[email protected]>
Signed-off-by: Gaëtan Lehmann [email protected]