-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
add: enable webhook patching with flag #6396
Conversation
bc4db15
to
5877362
Compare
Nice addition! I think that we should document it also as part of docs (maybe here) and support it in our helm chart (disabling the patching when the webhooks aren't deployed) |
893fb26
to
ea6aea9
Compare
can you please also add a changelog entry? |
ea6aea9
to
8216cf8
Compare
7bb8cec
to
4218d66
Compare
Signed-off-by: krishna sindhur <[email protected]>
Signed-off-by: krishna sindhur <[email protected]>
Signed-off-by: Jan Wozniak <[email protected]>
4218d66
to
bcbcb6e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thank you!
/run-e2e internal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good,
Could you please update docs and helm as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome! Could you open a PR updating docs as well?
here it is @JorTurFer |
Includes a flag in the operator to the webhook patching
Checklist
Fixes #6184