Skip to content

Commit

Permalink
fixed issuer indent (#1316)
Browse files Browse the repository at this point in the history
  • Loading branch information
AndrewChubatiuk authored Aug 25, 2024
1 parent 5e25f6a commit 4bd005c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
3 changes: 2 additions & 1 deletion charts/victoria-metrics-operator/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

## Next release

- TODO
- set `admissionWebhooks.keepTLSSecret` to `true` by default
- fixed indent, for Issuer crd, when `cert-manager.enabled: true`

## 0.34.1

Expand Down
5 changes: 3 additions & 2 deletions charts/victoria-metrics-operator/templates/webhook.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
{{- if .Values.admissionWebhooks.enabled }}
{{- $tls := fromYaml (include "vm-operator.certs" .) }}
---
apiVersion: admissionregistration.k8s.io/v1
kind: ValidatingWebhookConfiguration
metadata:
Expand Down Expand Up @@ -41,11 +42,11 @@ webhooks:
- UPDATE
resources:
- {{ $name }}{{ ternary "" "s" (hasSuffix "s" $name) }}
---
{{- end }}
{{- end }}
{{- if .Values.admissionWebhooks.certManager.enabled }}
{{- if not .Values.admissionWebhooks.certManager.issuer -}}
{{- if not .Values.admissionWebhooks.certManager.issuer }}
---
apiVersion: cert-manager.io/v1
kind: Issuer
metadata:
Expand Down
2 changes: 1 addition & 1 deletion charts/victoria-metrics-operator/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ admissionWebhooks:
enabled: false
# --If needed, provide own issuer. Operator will create self-signed if empty.
issuer: {}
keepTLSSecret: false
keepTLSSecret: true
# tls specifies TLS cert/key for the webhook
tls:
caCert:
Expand Down

0 comments on commit 4bd005c

Please sign in to comment.