Skip to content

Pass supplemental groups when Podman is using crun #1647

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mwinters0
Copy link

@mwinters0 mwinters0 commented May 15, 2025

This solves the common issue where /dev/vboxusb/* is owned by group vboxusers, causing volume mounts to fail when running toolbox enter. Same is true for any other mapped dirs with supplemental group owners. This is a common issue, frequently reported by its rather cryptic symptoms.

Tested on Fedora Workstation 42.

Fixes #1589
Probably #1640

And maybe:
#1310
#1085
#569


Note:
Podman option keep-groups was created in containers/podman@e356160 and first released in v3.2.0 (2021-06-03).

@mwinters0 mwinters0 requested a review from debarshiray as a code owner May 15, 2025 19:03
This solves the common issue where /dev/vboxusb/* is owned by group
"vboxusers", causing volume mounts to fail when running `toolbox enter`.
Same is true for any other mapped dirs with supplemental group owners.

Signed-off-by: Michael Winters <[email protected]>
@mwinters0
Copy link
Author

mwinters0 commented May 15, 2025

I see that some of the Zuul tests are failing because they now see additional groups inside the container:

# -- line differs --
# index    : 0
# expected : uid=1000(zuul-worker) gid=1000(zuul-worker) groups=1000(zuul-worker)
# actual   : uid=1000(zuul-worker) gid=1000(zuul-worker) groups=1000(zuul-worker),27(sudo)
# --

I'm not able to address this because I don't really know what I'm doing with BATS:

% bats ./test/system
1..1
not ok 1 bats-gather-tests
# bats_load_safe: Could not find '/home/mwinters/projects/forks/containers_toolbox/test/system/libs/bats-support/load'[.bash]

Signed-off-by: Michael Winters <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fails to start if VirtualBox is installed
1 participant