-
Notifications
You must be signed in to change notification settings - Fork 70
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
Volsync does not start in cluster with PodSecurityAdmission set to restricted #1430
Comments
You may need to set https://github.com/backube/volsync/blob/main/helm/volsync/values.yaml#L66 |
Right... not sure how I missed that... podSecurityContext:
runAsNonRoot: true
seccompProfile:
type: RuntimeDefault Leads to:
So, new thing failed - but runAsRoot needs to be set in order to reach restricted. |
I think it sounds like you also will need to set |
So, this is the conclussion I chose: podSecurityContext:
runAsUser: 1000
runAsGroup: 1000
fsGroup: 1000
fsGroupChangePolicy: "OnRootMismatch"
runAsNonRoot: true
seccompProfile:
type: RuntimeDefault This will allow volsync to work under restricted. In order to do backup/restores: moverSecurityContext:
runAsUser: 1000
runAsGroup: 1000
fsGroup: 1000
fsGroupChangePolicy: "OnRootMismatch"
runAsNonRoot: true
seccompProfile:
type: RuntimeDefault Maybe some of these values should be the defaults? My issue is closed however, I solved it with some configuration, I'll leave it open in-case someone needs/wants to change the default securityContext of volsync. |
I think it would be fine to set this as a default on the helm chart? podSecurityContext:
runAsNonRoot: true
runAsUser: 65534
runAsGroup: 65534
seccompProfile:
type: RuntimeDefault |
Describe the bug
VolSync fails to start in a cluster with Pod Security Configuration enforced to restricted, due to missing SecCompProfile.
Steps to reproduce
Enforce pod security configuration to restricted in your cluster.
Deploy volsync
Expected behavior
Volsync is actually deployed
Actual results
Additional context
#292 is somewhat related, but deals with the implementation in kind I guess.
The text was updated successfully, but these errors were encountered: