-
Notifications
You must be signed in to change notification settings - Fork 339
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
Check for CAP_SYS_ADMIN instead of root #3588
base: main
Are you sure you want to change the base?
Conversation
/cc @davide125 |
Even if we're running as root, we might not have CAP_SYS_ADMIN, so let's always check for CAP_SYS_ADMIN.
@DaanDeMeyer fyi this by itself doesn't seem to be enough to run
(same result without the |
@davide125 Can you check whether the additional commandline invoations from containers/podman#7209 do the trick? |
The |
@davide125 I can't reproduce the devpts failure unfortunately. Any chance there's something special about your environment? Do you get permission denied with podman build -t banana . with selinux off? |
No, you're right, it works fine with selinux disabled with just |
Making a directory on overlayfs the upperdir of another overlayfs is rejected by the kernel. Let's try to find a directory that's not on overlayfs. The /dev/shm fallback is because on a default podman container even /tmp is on overlayfs.
Even if we're running as root, we might not have CAP_SYS_ADMIN, so let's always check for CAP_SYS_ADMIN.