mirrors/kyverno
mirrors/kyverno
1
0
Fork 0
mirror of https://github.com/kyverno/kyverno.git synced 2025-03-28 18:38:40 +00:00
Code Activity

fix: update explicit webhook based on the policy type (#11580)

Browse source 
Signed-off-by: ShutingZhao <shuting@nirmata.com>
Co-authored-by: Jim Bugwadia <jim@nirmata.com>
This commit is contained in:
shuting 2024-11-13 23:39:36 +08:00 • committed by GitHub
parent 734fe164f1
commit 244dbe19cf
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
pkg/controllers/webhook/controller.go
View file

@ -528,7 +528,7 @@ func (c *controller) isGlobalContextEntryReady(name string, gctxentries []*kyver
return false
}
func (c *controller) updatePolicyStatuses(ctx context.Context) error {
func (c *controller) updatePolicyStatuses(ctx context.Context, webhookType string) error {
c.lock.Lock()
defer c.lock.Unlock()
policies, err := c.getAllPolicies()
@ -546,10 +546,9 @@ func (c *controller) updatePolicyStatuses(ctx context.Context) error {
}
ready, message := true, "Ready"
if c.autoUpdateWebhooks {
for _, set := range c.policyState {
if set, ok := c.policyState[webhookType]; ok {
if !set.Has(policyKey) {
ready, message = false, "Not Ready"
break
}
}
}
@ -670,7 +669,7 @@ func (c *controller) reconcile(ctx context.Context, logger logr.Logger, key, nam
if err := c.reconcileResourceMutatingWebhookConfiguration(ctx); err != nil {
return err
}
if err := c.updatePolicyStatuses(ctx); err != nil {
if err := c.updatePolicyStatuses(ctx, config.MutatingWebhookConfigurationName); err != nil {
return err
}
}
@ -681,7 +680,7 @@ func (c *controller) reconcile(ctx context.Context, logger logr.Logger, key, nam
if err := c.reconcileResourceValidatingWebhookConfiguration(ctx); err != nil {
return err
}
if err := c.updatePolicyStatuses(ctx); err != nil {
if err := c.updatePolicyStatuses(ctx, config.ValidatingWebhookConfigurationName); err != nil {
return err
}
}