Skip to content

Commit a58b049

Browse files
authored
fix(chart): Environment variable name spelling error (radondb#514)
- - name: ENABLED_WEBHOOKS + - name: ENABLE_WEBHOOKS
1 parent 43ab2cb commit a58b049

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

charts/mysql-operator/templates/deployment.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ spec:
2121
spec:
2222
securityContext:
2323
runAsNonRoot: true
24+
{{- if .Values.manager.enableWebhooks }}
2425
volumes:
2526
- name: cert
2627
secret:
@@ -54,6 +55,7 @@ spec:
5455
- containerPort: 9443
5556
name: webhook-server
5657
protocol: TCP
58+
{{- if .Values.manager.enableWebhooks }}
5759
volumeMounts:
5860
- name: cert
5961
mountPath: /tmp/k8s-webhook-server/serving-certs/
@@ -75,8 +77,8 @@ spec:
7577
env:
7678
- name: IMAGE_PREFIX
7779
value: {{ .Values.imagePrefix }}
78-
- name: ENABLED_WEBHOOKS
79-
value: {{ .Values.manager.enabledWebhooks | quote }}
80+
- name: ENABLE_WEBHOOKS
81+
value: {{ .Values.manager.enableWebhooks | quote }}
8082
securityContext:
8183
allowPrivilegeEscalation: false
8284
livenessProbe:

charts/mysql-operator/templates/webhook_configuration.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
{{- if .Values.manager.enableWebhooks }}
12
{{- $certManagerEnabled := .Values.webhook.certManager.enabled -}}
23
{{- $caCertPEM := include "webhook.caBundleCertPEM" . -}}
34
{{- $tlsCertPEM := include "webhook.certPEM" . -}}
@@ -71,3 +72,4 @@ data:
7172
tls.crt: {{ b64enc $tlsCertPEM }}
7273
tls.key: {{ b64enc $tlsKeyPEM }}
7374
{{- end }}
75+
{{- end }}

charts/mysql-operator/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ tolerationSeconds: 30
2222
manager:
2323
image: radondb/mysql-operator
2424
tag: v2.2.0
25-
enabledWebhooks: true
25+
enableWebhooks: true
2626
resources: {}
2727
# We usually recommend not to specify default resources and to leave this as a conscious
2828
# choice for the user. This also increases chances charts run on environments with little

0 commit comments

Comments
 (0)