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

Changes on Flux v0.26.0 #10

Closed
pjbgf opened this issue Jan 18, 2022 · 8 comments
Closed

Changes on Flux v0.26.0 #10

pjbgf opened this issue Jan 18, 2022 · 8 comments

Comments

@pjbgf
Copy link

pjbgf commented Jan 18, 2022

We are rolling out a few security related changes in flux2 that may impact flux2-openshift.
Here's a summary of them:

  • Enable Seccomp by default using new API (requires Kubernetes 1.19).
  • Hard-code userId on container images and enable securityContext.runAsNonRoot.
  • Drop all capabilities that are not being used.

I will link the PRs here to keep track of progress.

@chanwit
Copy link
Member

chanwit commented Jan 18, 2022

Thank you so much, @pjbgf

@stefanprodan
Copy link
Contributor

@chanwit any news on this, we got OpenShift users that are reporting that Flux 0.26 doesn't work due to the seccompProfile. Fails like this:

PodSecurityPolicy: unable to admit pod: [pod.metadata.annotations[container.seccomp.security.alpha.kubernetes.io/manager]: Forbidden: seccomp may not be set]

@chanwit
Copy link
Member

chanwit commented Feb 18, 2022

Still working on it. Best bet at the moment is either:

  1. Use the privileged SCC, but it's going to allow us to set some privileged settings too,

or

  1. Drop seccomp profile from the YAML.

@stefanprodan
Copy link
Contributor

I guess the scope of this repo is to allow "one click" install for Flux on OpenShift, so I would drop everything that prevents that.

@chanwit
Copy link
Member

chanwit commented Feb 18, 2022

BTW, the document updated last 2 weeks does not work for the user?
https://fluxcd.io/docs/use-cases/openshift/#security-context-constraints

@chanwit
Copy link
Member

chanwit commented Feb 18, 2022

I'll work with @pjbgf on Monday to check the details in each SCC per his request. Will get back to you with some solutions.

@chanwit
Copy link
Member

chanwit commented Feb 21, 2022

Documented by: fluxcd/website#786

@pjbgf
Copy link
Author

pjbgf commented Feb 21, 2022

I will close this issue based on the documentation updates.
Thank you @chanwit for testing and updating the documentation.

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

No branches or pull requests

3 participants