Skip to content

Commit

Permalink
fix: prevent recurring helm install/upgrade creating a loop deleting …
Browse files Browse the repository at this point in the history
…caBundle, when certManager is enabled (#309)

Signed-off-by: Rafael da Fonseca <[email protected]>
  • Loading branch information
rsafonseca authored Jan 25, 2024
1 parent c255b9d commit f1c5d77
Showing 1 changed file with 8 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,9 @@ webhooks:
name: {{ template "vault-secrets-webhook.fullname" . }}
path: /pods
{{- end }}
{{- if not .Values.certificate.useCertManager }}
caBundle: {{ $caCrt }}
{{- end }}
rules:
- operations:
- CREATE
Expand Down Expand Up @@ -188,7 +190,9 @@ webhooks:
name: {{ template "vault-secrets-webhook.fullname" . }}
path: /secrets
{{- end }}
{{- if not .Values.certificate.useCertManager }}
caBundle: {{ $caCrt }}
{{- end }}
rules:
- operations:
- CREATE
Expand Down Expand Up @@ -256,7 +260,9 @@ webhooks:
name: {{ template "vault-secrets-webhook.fullname" . }}
path: /configmaps
{{- end }}
{{- if not .Values.certificate.useCertManager }}
caBundle: {{ $caCrt }}
{{- end }}
rules:
- operations:
- CREATE
Expand Down Expand Up @@ -321,7 +327,9 @@ webhooks:
name: {{ template "vault-secrets-webhook.fullname" . }}
path: /objects
{{- end }}
{{- if not .Values.certificate.useCertManager }}
caBundle: {{ $caCrt }}
{{- end }}
rules:
- operations:
- CREATE
Expand Down

0 comments on commit f1c5d77

Please sign in to comment.