Open davidpechcz opened 10 months ago
I think we should remove the webhook conversion block from the helm CRD. would you like to open a PR ? We have move that to the docs here : https://github.com/OT-CONTAINER-KIT/helm-charts/blob/main/charts/redis-operator/readme.md
I'd like to help, but can you explain please the reason for the conversion webhook? I mean it is safe to remove it without any other changes, or did you mean that is should be removed conditionally on some new helm chart variable?
The reason we added the conversion webhook in the 1st place was to support the v1beta1
and v1beta2
. If you use the v1beta2
version it's safe to remove that.
I found out that conversion part is still in the helm chart we should remove that.
same questions, can you tell me how to remove it?
What version of redis operator are you using? HELM chart: 0.15.9
operator logs:
redis-operator version: image: 'quay.io/opstree/redis-operator:v0.15.1'
Does this issue reproduce with the latest release?
Yes
What did you do?
We upgraded to 0.15.9 from older version. We use beta1 CRDs. We replaced to the latest CRD definitions. We tried to experiment with switch beta1 to beta2 etc. On 'kubectl apply' CRD started to show:
We are not able to do anything with the CRDs as the conversion webhook fails and denies all operations.
What did you expect to see?
I suggest 3 options:
What did you see instead?
Workaround for us - remove the CRDs from helm installation, patched them (removed converion part) and applied them separatedly.
Please note - as we use GitOps, the patching the CRD as suggested in the manual with TLS cert info is not viable, so without patching we get this error (of course):