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

MAISTRA-548 Drop CAP_MKNOD from sidecar container #102

Merged
merged 1 commit into from
Jun 27, 2019

Conversation

luksa
Copy link
Contributor

@luksa luksa commented Jun 27, 2019

The anyuid SCC requires the MKNOD kernel capability to be dropped. If
the container's spec doesn't drop it, the SCC admission controller tries
to add it, but isn't allowed to, because it tries to do that during the
validation phase (as the sidecar container wasn't injected prior to the
SCC controller running during the mutation phase). Because it can't
mutate the pod, the SCC admission controller rejects it. By dropping the
capability in the sidecar template, the mutation is not required and
thus the pod is marked as valid by the SCC admission controller.

The anyuid SCC requires the MKNOD kernel capability to be dropped. If
the container's spec doesn't drop it, the SCC admission controller tries
to add it, but isn't allowed to, because it tries to do that during the
validation phase (as the sidecar container wasn't injected prior to the
SCC controller running during the mutation phase). Because it can't
mutate the pod, the SCC admission controller rejects it. By dropping the
capability in the sidecar template, the mutation is not required and
thus the pod is marked as valid by the SCC admission controller.
Copy link
Contributor

@rcernich rcernich left a comment

Choose a reason for hiding this comment

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

lgtm

@rcernich rcernich merged commit 3616126 into maistra:maistra-0.12 Jun 27, 2019
@luksa luksa deleted the drop_cap_mknod branch June 28, 2019 11:00
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.

None yet

2 participants