From c6372992c609797e0f771e31b32469221df51dda Mon Sep 17 00:00:00 2001
From: Jennifer Richards <jennifer@staff.ietf.org>
Date: Sun, 17 Mar 2024 12:35:16 +1000
Subject: [PATCH] ci: use datatracker image values for celery/beat (#7213)

* ci: use datatracker image values for celery/beat

* ci: refactor exec command lists as JSON

---------

Co-authored-by: Robert Sparks <rjsparks@nostrum.com>
---
 helm/templates/deployments/beat.yaml   |  4 +--
 helm/templates/deployments/celery.yaml |  4 +--
 helm/values.yaml                       | 44 ++++++++------------------
 3 files changed, 18 insertions(+), 34 deletions(-)

diff --git a/helm/templates/deployments/beat.yaml b/helm/templates/deployments/beat.yaml
index bfb3e99b1..6f7409fb9 100644
--- a/helm/templates/deployments/beat.yaml
+++ b/helm/templates/deployments/beat.yaml
@@ -27,8 +27,8 @@ spec:
         - name: {{ .Chart.Name }}
           securityContext:
             {{- toYaml $podValues.securityContext | nindent 12 }}
-          image: "{{ $podValues.image.repository }}:{{ default .Chart.AppVersion (default $.Values.datatracker.image.tag $podValues.image.tag) }}"
-          imagePullPolicy: {{ default "IfNotPresent" $podValues.image.imagePullPolicy }}
+          image: "{{ default $.Values.datatracker.image.repository $podValues.image.repository }}:{{ default .Chart.AppVersion (default $.Values.datatracker.image.tag $podValues.image.tag) }}"
+          imagePullPolicy: {{ default "IfNotPresent" (default $.Values.datatracker.image.imagePullPolicy $podValues.image.imagePullPolicy) }}
           env:
             - name: "CONTAINER_ROLE"
               value: "beat"
diff --git a/helm/templates/deployments/celery.yaml b/helm/templates/deployments/celery.yaml
index 8dc1643ff..f1f043abb 100644
--- a/helm/templates/deployments/celery.yaml
+++ b/helm/templates/deployments/celery.yaml
@@ -27,8 +27,8 @@ spec:
         - name: {{ .Chart.Name }}
           securityContext:
             {{- toYaml $podValues.securityContext | nindent 12 }}
-          image: "{{ $podValues.image.repository }}:{{ default .Chart.AppVersion (default $.Values.datatracker.image.tag $podValues.image.tag) }}"
-          imagePullPolicy: {{ default "IfNotPresent" $podValues.image.imagePullPolicy }}
+          image: "{{ default $.Values.datatracker.image.repository $podValues.image.repository }}:{{ default .Chart.AppVersion (default $.Values.datatracker.image.tag $podValues.image.tag) }}"
+          imagePullPolicy: {{ default "IfNotPresent" (default $.Values.datatracker.image.imagePullPolicy $podValues.image.imagePullPolicy) }}
           env:
             - name: "CONTAINER_ROLE"
               value: "celery"
diff --git a/helm/values.yaml b/helm/values.yaml
index cfdef165a..68b9344c0 100644
--- a/helm/values.yaml
+++ b/helm/values.yaml
@@ -166,9 +166,10 @@ datatracker:
 
 celery:
   name: celery
-  image:
-    repository: "ghcr.io/ietf-tools/datatracker"
-    pullPolicy: IfNotPresent
+  image: {}
+    # defaults to datatracker settings if not specified separately
+    #repository: "ghcr.io/ietf-tools/datatracker"
+    #pullPolicy: IfNotPresent
     # Overrides the image tag whose default is the chart appVersion.
     # tag: "v1.1.0"
 
@@ -178,10 +179,7 @@ celery:
 
   livenessProbe:
     exec:
-      command: 
-        - celery
-        - inspect
-        - ping
+      command: ["celery", "inspect", "ping"] 
     periodSeconds: 30
     timeoutSeconds: 5
 
@@ -235,10 +233,7 @@ celery:
     successThreshold: 1
     failureThreshold: 60
     exec:
-      command:
-        - celery
-        - inspect
-        - ping
+      command: ["celery", "inspect", "ping"]
 
   # Additional volumes on the output Deployment definition.
   volumes:
@@ -282,9 +277,10 @@ celery:
 
 beat:
   name: beat
-  image:
-    repository: "ghcr.io/ietf-tools/datatracker"
-    pullPolicy: IfNotPresent
+  image: {}
+    # defaults to datatracker settings if not specified separately
+    # repository: "ghcr.io/ietf-tools/datatracker"
+    # pullPolicy: IfNotPresent
     # Overrides the image tag whose default is the chart appVersion.
     # tag: "v1.1.0"
 
@@ -294,10 +290,7 @@ beat:
 
   livenessProbe:
     exec:
-      command:
-        - celery
-        - inspect
-        - ping
+      command: ["celery", "inspect", "ping"]
     periodSeconds: 30
     timeoutSeconds: 5
 
@@ -351,10 +344,7 @@ beat:
     successThreshold: 1
     failureThreshold: 60
     exec:
-      command:
-        - celery
-        - inspect
-        - ping
+      command: ["celery", "inspect", "ping"]
 
   # Additional volumes on the output Deployment definition.
   volumes:
@@ -409,10 +399,7 @@ rabbitmq:
 
   livenessProbe:
     exec:
-      command:
-        - rabbitmq-diagnostics
-        - -q
-        - ping
+      command: ["rabbitmq-diagnostics", "-q", "ping"]
     periodSeconds: 30
     timeoutSeconds: 5
 
@@ -466,10 +453,7 @@ rabbitmq:
     successThreshold: 1
     failureThreshold: 60
     exec:
-      command:
-        - rabbitmq-diagnostics
-        - -q
-        - ping
+      command: ["rabbitmq-diagnostics", "-q", "ping"]
 
   # Additional volumes on the output Deployment definition.
   volumes: []