weaveworks / flux2-openshift

OperatorHub submission repo for Flux2
Apache License 2.0
4 stars 4 forks source link

Changes on Flux v0.26.0 #10

Closed pjbgf closed 2 years ago

pjbgf commented 2 years ago

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

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

chanwit commented 2 years ago

Thank you so much, @pjbgf

stefanprodan commented 2 years ago

@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 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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

chanwit commented 2 years ago

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 commented 2 years ago

Documented by: https://github.com/fluxcd/website/pull/786

pjbgf commented 2 years ago

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