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

Clarify restricted.containers.privilege config option #12604

Merged
merged 2 commits into from
Jan 8, 2024

Conversation

roosterfish
Copy link
Contributor

@roosterfish roosterfish commented Dec 4, 2023

Fixes #12591

The fact that it automatically assumes security.idmap.isolated=true if restricted.containers.privilege is set to isoldated seems a bit odd since it is not the default value. However saying that this is the default behavior should explain the config option more thoroughly.

@github-actions github-actions bot added the Documentation Documentation needs updating label Dec 4, 2023
@simondeziel
Copy link
Member

@roosterfish looks like you edited the generated file rather than the canonical reference ;)

@roosterfish
Copy link
Contributor Author

@simondeziel ah yes my bad, thanks!

@roosterfish roosterfish changed the title doc: Clarify restricted.containers.privilege config option lxd: Clarify restricted.containers.privilege config option Dec 4, 2023
@roosterfish roosterfish changed the title lxd: Clarify restricted.containers.privilege config option Clarify restricted.containers.privilege config option Dec 4, 2023
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
@tomponline tomponline merged commit 97792f7 into canonical:main Jan 8, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc: mistake in restricted.containers.privilege documentation
4 participants