Skip to content

Commit

Permalink
fix(chart): Environment variable name spelling error
Browse files Browse the repository at this point in the history
- - name: ENABLED_WEBHOOKS
+ - name: ENABLE_WEBHOOKS
  • Loading branch information
runkecheng committed Jun 8, 2022
1 parent 5e0d3c2 commit f645027
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
8 changes: 6 additions & 2 deletions charts/mysql-operator/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,13 @@ spec:
spec:
securityContext:
runAsNonRoot: true
{{- if .Values.manager.enableWebhooks }}
volumes:
- name: cert
secret:
defaultMode: 420
secretName: "{{ template "webhook.name" . }}-certs"
{{- end }}
containers:
{{- if .Values.rbacProxy.create }}
- name: kube-rbac-proxy
Expand All @@ -48,10 +50,12 @@ spec:
- containerPort: 9443
name: webhook-server
protocol: TCP
{{- if .Values.manager.enableWebhooks }}
volumeMounts:
- name: cert
mountPath: /tmp/k8s-webhook-server/serving-certs/
readOnly: true
{{- end }}
command:
- /manager
args:
Expand All @@ -67,8 +71,8 @@ spec:
env:
- name: IMAGE_PREFIX
value: {{ .Values.imagePrefix }}
- name: ENABLED_WEBHOOKS
value: {{ .Values.manager.enabledWebhooks | quote }}
- name: ENABLE_WEBHOOKS
value: {{ .Values.manager.enableWebhooks | quote }}
securityContext:
allowPrivilegeEscalation: false
livenessProbe:
Expand Down
2 changes: 2 additions & 0 deletions charts/mysql-operator/templates/webhook_configuration.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{{- if .Values.manager.enableWebhooks }}
{{- $certManagerEnabled := .Values.webhook.certManager.enabled -}}
{{- $caCertPEM := include "webhook.caBundleCertPEM" . -}}
{{- $tlsCertPEM := include "webhook.certPEM" . -}}
Expand Down Expand Up @@ -71,3 +72,4 @@ data:
tls.crt: {{ b64enc $tlsCertPEM }}
tls.key: {{ b64enc $tlsKeyPEM }}
{{- end }}
{{- end }}
2 changes: 1 addition & 1 deletion charts/mysql-operator/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ tolerationSeconds: 30
manager:
image: radondb/mysql-operator
tag: v2.2.0
enabledWebhooks: true
enableWebhooks: true
resources: {}
# We usually recommend not to specify default resources and to leave this as a conscious
# choice for the user. This also increases chances charts run on environments with little
Expand Down

0 comments on commit f645027

Please sign in to comment.