Skip to content

Commit

Permalink
caBundle doesn't need to be set
Browse files Browse the repository at this point in the history
And it causes the resource to show up in e.g. ArgoCD as Out-of-Sync
  • Loading branch information
james-callahan committed Dec 23, 2024
1 parent 6919efa commit 0611a16
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 1 addition & 3 deletions config/webhook/ingressclassparams_patch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
4 changes: 1 addition & 3 deletions config/webhook/targetgroupbindings_patch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 0611a16

Please sign in to comment.