You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After an internal discussion, we decided that, for now, if a field is removed we will use mutating webhooks to set it back to the same value as before.
This is to avoid issues like removing a PVC field and losing all the data (if no PVC is defined, the reconcile logic would define an emptyDir).
If we find that this is not the best approach, we will tweak the settings for each field of Pulp CR.
git-hyagi
added a commit
to git-hyagi/pulp-operator
that referenced
this issue
Oct 4, 2022
We did some tests and could make the validation through the admission controller work, but decided to not enable it yet.
To enable the validation webhooks users would need to first configure a certificate that needs to be trusted by api pods.
We checked some ways to achieve this:
through service certificate in OCP environments
by manually installing cert-manager
by manually installing cert-manager-operator in OCP environments
by manually creating and configuring the certificates and CA
Considering that any of the above alternatives would be another burden for the operator adoption, we decided to not merge #658 into main yet.
Our reconciliation logic is not handling a resource removal from CR, it is only checking if what is defined is deployed.
The text was updated successfully, but these errors were encountered: