fix: use toYaml instead of 'range' for environment variables, and ali…

…gnment between 'charts' and 'helm' folders Signed-off-by: Hervé Le Meur <[email protected]>
jenkins-x · Nov 21, 2020 · bf2155d · bf2155d
1 parent b758af2
commit bf2155d
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 21 deletions.
diff --git a/charts/charts/templates/deployment.yaml b/charts/charts/templates/deployment.yaml
@@ -39,10 +39,7 @@ spec:
         env:
         - name: VERSION
           value: {{ .Chart.Version }}
-{{- range $pkey, $pval := .Values.env }}
-        - name: {{ $pkey }}
-          value: {{ quote $pval }}
-{{- end }}
+{{ toYaml .Values.env | indent 10 }}
         envFrom:
 {{ toYaml .Values.envFrom | indent 10 }}
         ports:

diff --git a/charts/charts/templates/ksvc.yaml b/charts/charts/templates/ksvc.yaml
@@ -21,10 +21,9 @@ spec:
           env:
           - name: VERSION
             value: {{ .Chart.Version }}
-{{- range $pkey, $pval := .Values.env }}
-          - name: {{ $pkey }}
-            value: {{ quote $pval }}
-{{- end }}
+{{ toYaml .Values.env | indent 12 }}
+          envFrom:
+{{ toYaml .Values.envFrom | indent 12 }}
           livenessProbe:
             httpGet:
               path: {{ .Values.livenessProbe.probePath | default .Values.probePath }}

diff --git a/helm/charts/templates/deployment.yaml b/helm/charts/templates/deployment.yaml
@@ -30,25 +30,24 @@ spec:
         image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
         imagePullPolicy: {{ .Values.image.pullPolicy }}
         env:
-{{- range $pkey, $pval := .Values.env }}
-        - name: {{ $pkey }}
-          value: {{ quote $pval }}
-{{- end }}
+        - name: VERSION
+          value: {{ .Chart.Version }}
+{{ toYaml .Values.env | indent 8 }}
         envFrom:
-{{ toYaml .Values.envFrom | indent 10 }}
+{{ toYaml .Values.envFrom | indent 8 }}
         ports:
         - containerPort: {{ .Values.service.internalPort }}
         livenessProbe:
           httpGet:
-            path: {{ .Values.probePath }}
+            path: {{ .Values.livenessProbe.probePath | default .Values.probePath }}
             port: {{ .Values.service.internalPort }}
           initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
           periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
           successThreshold: {{ .Values.livenessProbe.successThreshold }}
           timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}
         readinessProbe:
           httpGet:
-            path: {{ .Values.probePath }}
+            path: {{ .Values.readinessProbe.probePath | default .Values.probePath }}
             port: {{ .Values.service.internalPort }}
           periodSeconds: {{ .Values.readinessProbe.periodSeconds }}
           successThreshold: {{ .Values.readinessProbe.successThreshold }}

diff --git a/helm/charts/templates/ksvc.yaml b/helm/charts/templates/ksvc.yaml
@@ -18,10 +18,11 @@ spec:
             image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
             imagePullPolicy: {{ .Values.image.pullPolicy }}
             env:
-{{- range $pkey, $pval := .Values.env }}
-            - name: {{ $pkey }}
-              value: {{ quote $pval }}
-{{- end }}
+            - name: VERSION
+              value: {{ .Chart.Version }}
+{{ toYaml .Values.env | indent 12 }}
+            envFrom:
+{{ toYaml .Values.envFrom | indent 12 }}
             livenessProbe:
               httpGet:
                 path: {{ .Values.probePath }}

diff --git a/helm/charts/values.yaml b/helm/charts/values.yaml
@@ -65,12 +65,15 @@ livenessProbe:
   periodSeconds: 10
   successThreshold: 1
   timeoutSeconds: 1
+  # Optional distinct liveness probe path, if empty the probePath is used
+  probePath: ""
 readinessProbe:
   failureThreshold: 1
   periodSeconds: 10
   successThreshold: 1
   timeoutSeconds: 1
-
+  # Optional distinct liveness probe path, if empty the probePath is used
+  probePath: ""
 
 # custom ingress annotations on this service
 ingress:

diff --git a/helm/preview/helmfile.yaml b/helm/preview/helmfile.yaml
@@ -1,12 +1,17 @@
+environments:
+  default:
+    values:
+    - jx-values.yaml
 releases:
 - chart: '../{{ requiredEnv "APP_NAME" }}'
   name: preview
   wait: true
   createNamespace: true
   namespace: '{{ requiredEnv "PREVIEW_NAMESPACE" }}'
   values:
-  - values.yaml.gotmpl
   - jx-values.yaml
+  - imagePullSecrets.yaml
+  - values.yaml.gotmpl
   hooks:
   - events: ["presync"]
     showlogs: true