search

redhat-cop / resource-locker-operator

Apache License 2.0
30 stars 14 forks source link

Operator broke on OKD and Kubernetes #57

Open aelbarkani opened 3 years ago

aelbarkani commented 3 years ago

Hi,

There was a change in the image which breaks the operator for non Redhat users: https://github.com/redhat-cop/resource-locker-operator/blob/6e63ea9c0273858d04121d32c4e76ef1030dab81/config/default/manager_auth_proxy_patch.yaml#L13

MajorFlamingo commented 2 years ago

Is there any way I can support a fix of the recent change?

raffaelespazzoli commented 2 years ago

please see the discussion here: https://github.com/redhat-cop/namespace-configuration-operator/issues/118

MajorFlamingo commented 2 years ago

It looks like the discussion points towards a quick-fix to un-break the operators, likewise on https://github.com/redhat-cop/cert-utils-operator/issues/102.

We could change the kube-rbac-proxy image to quay.io/openshift/origin-kube-rbac-proxy as mentioned in redhat-cop/namespace-configuration-operator#118, and then automatically get multiarch support once it's implemented in OKD4.

Additionally, we could undo #56 - and redo it once the multiarch image has been released?

I'd happily submit a PR, but like to align with the maintainers first.

MajorFlamingo commented 2 years ago

59 fixes this issue, I can confirm this. Thank you @raffaelespazzoli.