caBundle doesn't need to be set

And it causes the resource to show up in e.g. ArgoCD as Out-of-Sync
kubernetes-sigs · Nov 27, 2024 · 71d7b81 · 71d7b81
1 parent f99160b
commit 71d7b81
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/config/webhook/ingressclassparams_patch.yaml b/config/webhook/ingressclassparams_patch.yaml
@@ -9,9 +9,7 @@ spec:
     strategy: Webhook
     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==
+        # `caBundle` should be filled in by cert-manager (or potentially a patch if not using cert-manager)
         service:
           namespace: default
           name: webhook-service

diff --git a/config/webhook/targetgroupbindings_patch.yaml b/config/webhook/targetgroupbindings_patch.yaml
@@ -9,9 +9,7 @@ spec:
     strategy: Webhook
     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==
+        # `caBundle` should be filled in by cert-manager (or potentially a patch if not using cert-manager)
         service:
           namespace: default
           name: webhook-service