Merge pull request #1021 from TBeijen/app_image_fix

Chart fix to use AppVersion by default, allow override via values.yaml
kuberhealthy · Feb 24, 2022 · ac0402e · ac0402e
2 parents 0c52160 + 38f5f12
commit ac0402e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/deploy/helm/kuberhealthy/templates/deployment.yaml b/deploy/helm/kuberhealthy/templates/deployment.yaml
@@ -69,9 +69,9 @@ spec:
 {{- end }}
       containers:
       {{- if .Values.imageRegistry }}
-      - image: {{ .Values.imageRegistry }}/{{ .Values.image.repository }}:{{ .Chart.AppVersion }}
+      - image: {{ .Values.imageRegistry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion}}
       {{- else if .Values.image.registry }}
-      - image: {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Chart.AppVersion }}
+      - image: {{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion}}
       {{- end }}
         command: {{ .Values.deployment.command }}
         {{- if .Values.deployment.args }}

diff --git a/deploy/helm/kuberhealthy/values.yaml b/deploy/helm/kuberhealthy/values.yaml
@@ -39,7 +39,8 @@ imageRegistry: {}
 image:
   registry: kuberhealthy
   repository: kuberhealthy
-  tag: v2.4.1
+  # Leave empty to use .Chart.AppVersion
+  tag:
 
 resources:
   requests: