Use CRD v1 form

kustomize also requires that we specify a namespace here, or else it won't add one

Author: james-callahan

config/webhook/ingressclassparams_patch.yaml

@@ -7,10 +7,13 @@ metadata:
 spec:
   conversion:
     strategy: Webhook
-    webhookClientConfig:
-      # this is "\n" used as a placeholder, otherwise it will be rejected by the apiserver for being blank,
-      # but we're going to set it later using the cert-manager (or potentially a patch if not using cert-manager)
-      caBundle: Cg==
-      service:
-        name: webhook-service
-        path: /convert
+    webhook:
+      clientConfig:
+        # this is "\n" used as a placeholder, otherwise it will be rejected by the apiserver for being blank,
+        # but we're going to set it later using the cert-manager (or potentially a patch if not using cert-manager)
+        caBundle: Cg==
+        service:
+          namespace: default
+          name: webhook-service
+          path: /convert
+      conversionReviewVersions: ["v1", "v1beta1"]

config/webhook/kustomizeconfig.yaml

@@ -10,6 +10,9 @@ nameReference:
       - kind: ValidatingWebhookConfiguration
         group: admissionregistration.k8s.io
         path: webhooks/clientConfig/service/name
+      - group: apiextensions.k8s.io
+        kind: CustomResourceDefinition
+        path: spec/conversion/webhook/clientConfig/service/name
 
 namespace:
   - kind: MutatingWebhookConfiguration

config/webhook/targetgroupbindings_patch.yaml

@@ -7,10 +7,13 @@ metadata:
 spec:
   conversion:
     strategy: Webhook
-    webhookClientConfig:
-      # this is "\n" used as a placeholder, otherwise it will be rejected by the apiserver for being blank,
-      # but we're going to set it later using the cert-manager (or potentially a patch if not using cert-manager)
-      caBundle: Cg==
-      service:
-        name: webhook-service
-        path: /convert
+    webhook:
+      clientConfig:
+        # this is "\n" used as a placeholder, otherwise it will be rejected by the apiserver for being blank,
+        # but we're going to set it later using the cert-manager (or potentially a patch if not using cert-manager)
+        caBundle: Cg==
+        service:
+          namespace: default
+          name: webhook-service
+          path: /convert
+      conversionReviewVersions: ["v1", "v1beta1"]