-
Notifications
You must be signed in to change notification settings - Fork 451
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 utility function to refresh shoot webhook netpols #2663
Add utility function to refresh shoot webhook netpols #2663
Conversation
extensions/pkg/controller/controlplane/genericactuator/actuator.go
Outdated
Show resolved
Hide resolved
/status author-action |
@rfranzke The pull request was assigned to you under |
10b59e5
to
020a8b2
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
extensions/pkg/controller/controlplane/genericactuator/actuator.go
Outdated
Show resolved
Hide resolved
020a8b2
to
5ad64cd
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
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.
Just a small question from my side.
/lgtm |
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
How to categorize this PR?
/area networking
/kind bug
/priority normal
What this PR does / why we need it:
A utility function that reconciles the shoot webhook configurations has been added to the generic
ControlPlane
actuator package. It is recommended for all extensions that have shoot webhooks to call this function before starting the control loops to ensure that the webhook configurations are updated in case the ports change.Special notes for your reviewer:
/cc @timebertt @stoyanr
Release note: