add defaultTagPrefix per service (#295)

* add defautlTagPrefix per service

* add unit test and changelog  entry

* fix test

* fix test

Author: filiphaftek

charts/sourcegraph/CHANGELOG.md

@@ -9,6 +9,8 @@ Use `**BREAKING**:` to denote a breaking change
 
 ## Unreleased
 
+- Add defaultTagPrefix per service
+
 ## 5.0.5
 
 - Add support for PodDisruptionBudget in frontend

charts/sourcegraph/templates/_helpers.tpl

@@ -100,9 +100,10 @@ useGlobalTagAsDefault configuration
 {{- $service := index . 1 }}
 {{- $imageName := (index $top.Values $service "image" "name")}}
 {{- $defaultTag := (index $top.Values $service "image" "defaultTag")}}
+{{- $defaultTagPrefix := (index $top.Values $service "image" "defaultTagPrefix")}}
 {{- if $top.Values.sourcegraph.image.useGlobalTagAsDefault }}{{ $defaultTag = (tpl $top.Values.sourcegraph.image.defaultTag $top) }}{{ end }}
 
-{{- $top.Values.sourcegraph.image.repository }}/{{ $imageName }}:{{ default $defaultTag (index $top.Values $service "image" "tag") }}
+{{- $top.Values.sourcegraph.image.repository }}/{{ $imageName }}:{{ $defaultTagPrefix }}{{ default $defaultTag (index $top.Values $service "image" "tag") }}
 {{- end }}
 
 {{- define "sourcegraph.nodeSelector" -}}

charts/sourcegraph/tests/defaultTagPrefix_test.yaml

@@ -0,0 +1,26 @@
+suite: defaultTagPrefix
+release:
+  name: sourcegraph
+  namespace: sourcegraph
+tests:
+- it: should not add defaultTagPrefix
+  template: frontend/sourcegraph-frontend.Deployment.yaml
+  set:
+    frontend:
+      image:
+        defaultTag: test
+  asserts:
+  - equal:
+      path: spec.template.spec.containers[0].image
+      value: "index.docker.io/sourcegraph/frontend:test"
+- it: should add defaultTagPrefix
+  template: frontend/sourcegraph-frontend.Deployment.yaml
+  set:
+    frontend:
+      image:
+        defaultTag: test
+        defaultTagPrefix: "prefix-"
+  asserts:
+  - equal:
+      path: spec.template.spec.containers[0].image
+      value: "index.docker.io/sourcegraph/frontend:prefix-test"