Skip to content
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

podman: change permissions on /etc/containers #15673

Merged
merged 1 commit into from
May 22, 2021
Merged

Conversation

flyn-org
Copy link
Contributor

Running podman as users other than root seems to require that those
users can read the configuration files in /etc/containers. This change
sets the permissions of /etc/containers and its contents to match those
used on Fedora.

Signed-off-by: W. Michael Petullo [email protected]

Maintainer: @oskarirauta
Compile tested: x86_64 master
Run tested: x86_64 master

Description:

Copy link
Member

@dangowrt dangowrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use our macros instead of calling chmod directly.

utils/podman/Makefile Outdated Show resolved Hide resolved
utils/podman/Makefile Outdated Show resolved Hide resolved
Running podman as users other than root seems to require that those
users can read the configuration files in /etc/containers. This change
sets the permissions of /etc/containers and its contents to match those
used on Fedora.

Signed-off-by: W. Michael Petullo <[email protected]>
@dangowrt dangowrt merged commit f51ef46 into openwrt:master May 22, 2021
@dangowrt
Copy link
Member

Thanks!

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.

2 participants