Skip to content

Commit 72babc6

Browse files
author
Nont
committed
Compact the webhook rules
Signed-off-by: Nont <nont@duck.com>
1 parent e0f7d89 commit 72babc6

File tree

1 file changed

+8
-52
lines changed

1 file changed

+8
-52
lines changed

pkg/webhook/webhook.go

Lines changed: 8 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -413,64 +413,20 @@ func (w *Config) buildFleetValidatingWebhooks() []admv1.ValidatingWebhook {
413413
AdmissionReviewVersions: admissionReviewVersions,
414414
Rules: []admv1.RuleWithOperations{
415415
{
416-
Operations: []admv1.OperationType{
417-
admv1.Create, admv1.Update, admv1.Delete,
418-
},
419-
Rule: createRule(
420-
[]string{
421-
placementv1beta1.GroupVersion.Group,
422-
},
423-
[]string{
424-
placementv1beta1.GroupVersion.Version,
425-
},
426-
[]string{
427-
placementv1beta1.ClusterResourcePlacementResource,
428-
}, &clusterScope),
416+
Operations: []admv1.OperationType{admv1.Create, admv1.Update, admv1.Delete},
417+
Rule: createRule([]string{placementv1beta1.GroupVersion.Group}, []string{placementv1beta1.GroupVersion.Version}, []string{placementv1beta1.ClusterResourcePlacementResource}, &clusterScope),
429418
},
430419
{
431-
Operations: []admv1.OperationType{
432-
admv1.Create, admv1.Update, admv1.Delete,
433-
},
434-
Rule: createRule(
435-
[]string{
436-
corev1.SchemeGroupVersion.Group,
437-
},
438-
[]string{
439-
corev1.SchemeGroupVersion.Version,
440-
},
441-
[]string{
442-
namespaceResourceName,
443-
}, &clusterScope),
420+
Operations: []admv1.OperationType{admv1.Create, admv1.Update, admv1.Delete},
421+
Rule: createRule([]string{corev1.SchemeGroupVersion.Group}, []string{corev1.SchemeGroupVersion.Version}, []string{namespaceResourceName}, &clusterScope),
444422
},
445423
{
446-
Operations: []admv1.OperationType{
447-
admv1.Create, admv1.Update, admv1.Delete,
448-
},
449-
Rule: createRule(
450-
[]string{
451-
corev1.SchemeGroupVersion.Group,
452-
},
453-
[]string{
454-
corev1.SchemeGroupVersion.Version,
455-
},
456-
[]string{
457-
resourceQuotaResourceName,
458-
}, &namespacedScope),
424+
Operations: []admv1.OperationType{admv1.Create, admv1.Update, admv1.Delete},
425+
Rule: createRule([]string{corev1.SchemeGroupVersion.Group}, []string{corev1.SchemeGroupVersion.Version}, []string{resourceQuotaResourceName}, &namespacedScope),
459426
},
460427
{
461-
Operations: []admv1.OperationType{
462-
admv1.Create, admv1.Update, admv1.Delete,
463-
},
464-
Rule: createRule(
465-
[]string{
466-
networkingv1.SchemeGroupVersion.Group,
467-
},
468-
[]string{
469-
networkingv1.SchemeGroupVersion.Version,
470-
},
471-
[]string{
472-
networkPolicyResourceName,
473-
}, &namespacedScope),
428+
Operations: []admv1.OperationType{admv1.Create, admv1.Update, admv1.Delete},
429+
Rule: createRule([]string{networkingv1.SchemeGroupVersion.Group}, []string{networkingv1.SchemeGroupVersion.Version}, []string{networkPolicyResourceName}, &namespacedScope),
474430
},
475431
},
476432
TimeoutSeconds: longWebhookTimeout,

0 commit comments

Comments
 (0)