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
Operator: do not patch the Subscription CRDs if the conversion webhooks are disabled #7154
Conversation
…ks are disabled When running the operator with `ENABLE_WEBHOOKS=false`, which disables listening for the conversion webhooks, the operator should not be patching the CRDs Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
Do we have a test to cover this case? |
We do not, and we do not have the infrastructure for creating tests for this (we do not have the ability to test different Helm options in our CI, for example) |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #7154 +/- ##
==========================================
- Coverage 65.07% 64.95% -0.13%
==========================================
Files 221 221
Lines 20798 20803 +5
==========================================
- Hits 13534 13512 -22
- Misses 6125 6147 +22
- Partials 1139 1144 +5
☔ View full report in Codecov by Sentry. |
When running the operator with
ENABLE_WEBHOOKS=false
, which disables listening for the conversion webhooks, the operator should not be patching the CRDs