feat: add support for additional env vars (#9)

DEVmachine-fr · Aug 8, 2021 · ff95876 · ff95876
1 parent c753bd6
commit ff95876
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 4 deletions.
diff --git a/charts/alidns-webhook/Chart.yaml b/charts/alidns-webhook/Chart.yaml
@@ -1,5 +1,5 @@
 apiVersion: v1
 appVersion: "0.2.0"
-description: Deploys alidns webhook for cert-manager. 
+description: Deploys alidns webhook for cert-manager.
 name: alidns-webhook
-version: 0.4.0
+version: 0.5.0
diff --git a/charts/alidns-webhook/templates/deployment.yaml b/charts/alidns-webhook/templates/deployment.yaml
@@ -31,6 +31,9 @@ spec:
           env:
             - name: GROUP_NAME
               value: {{ .Values.groupName | quote }}
+          {{- if .Values.env }}
+{{- toYaml .Values.extraEnv | nindent 12 }}
+          {{- end }}
           ports:
             - name: https
               containerPort: {{ .Values.securePort }}

diff --git a/charts/alidns-webhook/templates/pki.yaml b/charts/alidns-webhook/templates/pki.yaml
@@ -29,7 +29,7 @@ metadata:
     heritage: {{ .Release.Service }}
 spec:
   secretName: {{ include "alidns-webhook.rootCACertificate" . }}
-  duration: 43800h # 5y
+  duration: 43800h0m0s # 5y
   issuerRef:
     name: {{ include "alidns-webhook.selfSignedIssuer" . }}
   commonName: "ca.alidns-webhook.cert-manager"
@@ -67,7 +67,7 @@ metadata:
     heritage: {{ .Release.Service }}
 spec:
   secretName: {{ include "alidns-webhook.servingCertificate" . }}
-  duration: 8760h # 1y
+  duration: 8760h0m0s # 1y
   issuerRef:
     name: {{ include "alidns-webhook.rootCAIssuer" . }}
   dnsNames:

diff --git a/charts/alidns-webhook/values.yaml b/charts/alidns-webhook/values.yaml
@@ -47,3 +47,11 @@ nodeSelector: {}
 tolerations: []
 
 affinity: {}
+
+
+# Optional additional envs for the webhook container
+extraEnv: []
+# - name: ALIDNS_WEBHOOK_ENV_VAR_1
+#   value: "env_var_1"
+# - name: ALIDNS_WEBHOOK_ENV_VAR_2
+#   value: "env_var_2"