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
Generate webhooks with kubebuilder #745
Conversation
✅ Deploy Preview for elated-stonebraker-105904 canceled.
|
#@ load("@ytt:overlay", "overlay") | ||
|
||
#@overlay/match by=overlay.subset({"kind": "ValidatingWebhookConfiguration"}) | ||
|
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.
oh interesting, nice that we can do all the changes we need right here with a single overlay 👍
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.
overall, lgtm! didn't get to run locally, but looks good!
for anyone looking at this PR - note that we end up with one less
vs
but, in the end, we're registering validation for the same gvrs as before (just in a single object instead). |
* Register cluster run template webhook Co-authored-by: Marty Spiewak <mspiewak@vmware.com>
3c71ccd
to
baaaaed
Compare
Co-authored-by: Marty Spiewak <mspiewak@vmware.com>
Changes proposed by this PR
closes #
Release Note
PR Checklist
Note: Please do not remove items. Mark items as done
[x]
or usestrikethroughif you believe they are not relevantFixes #123
orUpdates #123
wip
commits