Define images repository and tag as seperate keys, fallback to old st…

…yle (#16)

and switch to new upstream registry

charts/etcd/templates/_image.tpl

@@ -0,0 +1,15 @@
+{{- define "image-etcd" -}}
+{{- if ((.Values.images).etcd) -}}
+{{ .Values.images.etcd }}
+{{- else -}}
+{{ (index .Values "etcd").image.repository }}:{{ (index .Values "etcd").image.tag }}
+{{- end -}}
+{{- end -}}
+
+{{- define "image-etcd-backup-restore" -}}
+{{- if (and (hasKey .Values "images") (index .Values.images "etcd-backup-restore")) -}}
+{{ index .Values.images "etcd-backup-restore" }}
+{{- else -}}
+{{ (index .Values "etcd-backup-restore").image.repository }}:{{ (index .Values "etcd-backup-restore").image.tag }}
+{{- end -}}
+{{- end -}}

charts/etcd/templates/statefulset-etcd.yaml

@@ -47,7 +47,7 @@ spec:
     spec:
       containers:
       - name: etcd
-        image: {{ index .Values.images "etcd" }}
+        image: {{ template "image-etcd" . }}
         imagePullPolicy: IfNotPresent
         command:
         - /bootstrap/bootstrap.sh
@@ -123,7 +123,7 @@ spec:
         - --service-endpoints=https://{{ .Values.name }}-0:2379
         - --etcd-connection-timeout=300s
         - --delta-snapshot-period=300s
-        image: {{ index .Values.images "etcd-backup-restore" }}
+        image: {{ template "image-etcd-backup-restore" . }}
         imagePullPolicy: IfNotPresent
         ports:
         - containerPort: 8080

charts/etcd/tests/statefulset-etcd_test.yaml

@@ -0,0 +1,46 @@
+templates:
+  - statefulset-etcd.yaml
+tests:
+  - it: should use the default values images
+    values:
+      - ../../../test-values.yaml
+    asserts:
+      - equal:
+          path: spec.template.spec.containers[0].image
+          value: europe-docker.pkg.dev/gardener-project/public/gardener/etcd:v3.4.26-3
+      - equal:
+          path: spec.template.spec.containers[1].image
+          value: europe-docker.pkg.dev/gardener-project/public/gardener/etcdbrctl:v0.24.8
+  - it: should allow overwriting the images
+    values:
+      - ../../../test-values.yaml
+    set:
+      etcd:
+        image:
+          repository: foo/bar
+          tag: baz
+      etcd-backup-restore: 
+        image:
+          repository: baz/foo
+          tag: bar
+    asserts:
+      - equal:
+          path: spec.template.spec.containers[0].image
+          value: foo/bar:baz
+      - equal:
+          path: spec.template.spec.containers[1].image
+          value: baz/foo:bar
+  - it: should include the old style image repo:tag when provided
+    values:
+      - ../../../test-values.yaml
+    set:
+      images:
+        etcd: foo/bar:baz
+        etcd-backup-restore: baz/foo:bar
+    asserts:
+      - equal:
+          path: spec.template.spec.containers[0].image
+          value: foo/bar:baz
+      - equal:
+          path: spec.template.spec.containers[1].image
+          value: baz/foo:bar

charts/etcd/values.yaml

@@ -14,9 +14,15 @@
 
 name: etcd
 
-images:
-  etcd: eu.gcr.io/gardener-project/gardener/etcd:v3.4.26-3
-  etcd-backup-restore: eu.gcr.io/gardener-project/gardener/etcdbrctl:v0.24.7
+etcd:
+  image:
+    repository: europe-docker.pkg.dev/gardener-project/public/gardener/etcd
+    tag: v3.4.26-3
+
+etcd-backup-restore:
+  image:
+    repository: europe-docker.pkg.dev/gardener-project/public/gardener/etcdbrctl
+    tag: v0.24.8
 
 backup:
   schedule: "0 */24 * * *" # cron standard schedule

test-values.yaml

@@ -0,0 +1,10 @@
+tls:
+  ca:
+    crt: dGVzdA==
+    key: dGVzdA==
+  server:
+    crt: dGVzdA==
+    key: dGVzdA==
+  client:
+    crt: dGVzdA==
+    key: dGVzdA==