Skip to content

Commit 715b8a5

Browse files
committed
Fix SetupWebhookWithManager
1 parent c5d0843 commit 715b8a5

File tree

4 files changed

+19
-4
lines changed

4 files changed

+19
-4
lines changed

apis/eventing/v1alpha1/cloudeventsource_webhook.go

+2
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,14 @@ var cloudeventsourcelog = logf.Log.WithName("cloudeventsource-validation-webhook
3434

3535
func (ces *CloudEventSource) SetupWebhookWithManager(mgr ctrl.Manager) error {
3636
return ctrl.NewWebhookManagedBy(mgr).
37+
WithValidator(&CloudEventSourceCustomValidator{}).
3738
For(ces).
3839
Complete()
3940
}
4041

4142
func (cces *ClusterCloudEventSource) SetupWebhookWithManager(mgr ctrl.Manager) error {
4243
return ctrl.NewWebhookManagedBy(mgr).
44+
WithValidator(&ClusterCloudEventSourceCustomValidator{}).
4345
For(cces).
4446
Complete()
4547
}

apis/keda/v1alpha1/scaledjob_webhook.go

+1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ var scaledjoblog = logf.Log.WithName("scaledjob-validation-webhook")
3333

3434
func (s *ScaledJob) SetupWebhookWithManager(mgr ctrl.Manager) error {
3535
return ctrl.NewWebhookManagedBy(mgr).
36+
WithValidator(&ScaledJobCustomValidator{}).
3637
For(s).
3738
Complete()
3839
}

apis/keda/v1alpha1/scaledobject_webhook.go

+14-4
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,18 @@ var memoryString = "memory"
5353
var cpuString = "cpu"
5454

5555
func (so *ScaledObject) SetupWebhookWithManager(mgr ctrl.Manager, cacheMissFallback bool) error {
56+
err := setupKubernetesClients(mgr, cacheMissFallback)
57+
if err != nil {
58+
return fmt.Errorf("failed to setup kubernetes clients: %w", err)
59+
}
60+
61+
return ctrl.NewWebhookManagedBy(mgr).
62+
WithValidator(&ScaledObjectCustomValidator{}).
63+
For(so).
64+
Complete()
65+
}
66+
67+
func setupKubernetesClients(mgr ctrl.Manager, cacheMissFallback bool) error {
5668
kc = mgr.GetClient()
5769
restMapper = mgr.GetRESTMapper()
5870
cacheMissToDirectClient = cacheMissFallback
@@ -70,10 +82,8 @@ func (so *ScaledObject) SetupWebhookWithManager(mgr ctrl.Manager, cacheMissFallb
7082
return fmt.Errorf("failed to initialize direct client: %w", err)
7183
}
7284
}
73-
return ctrl.NewWebhookManagedBy(mgr).
74-
WithValidator(&ScaledObjectCustomValidator{}).
75-
For(so).
76-
Complete()
85+
86+
return nil
7787
}
7888

7989
// +kubebuilder:webhook:path=/validate-keda-sh-v1alpha1-scaledobject,mutating=false,failurePolicy=ignore,sideEffects=None,groups=keda.sh,resources=scaledobjects,verbs=create;update,versions=v1alpha1,name=vscaledobject.kb.io,admissionReviewVersions=v1

apis/keda/v1alpha1/triggerauthentication_webhook.go

+2
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,14 @@ var triggerauthenticationlog = logf.Log.WithName("triggerauthentication-validati
3838

3939
func (ta *TriggerAuthentication) SetupWebhookWithManager(mgr ctrl.Manager) error {
4040
return ctrl.NewWebhookManagedBy(mgr).
41+
WithValidator(&TriggerAuthenticationCustomValidator{}).
4142
For(ta).
4243
Complete()
4344
}
4445

4546
func (cta *ClusterTriggerAuthentication) SetupWebhookWithManager(mgr ctrl.Manager) error {
4647
return ctrl.NewWebhookManagedBy(mgr).
48+
WithValidator(&ClusterTriggerAuthenticationCustomValidator{}).
4749
For(cta).
4850
Complete()
4951
}

0 commit comments

Comments
 (0)