Centralize job hash generation and incorporate chart version in it (#30)

refactor: centralize job hash generation and incorporate chart version in it
stafftastic · May 28, 2024 · ef1bb9b · ef1bb9b
1 parent 58090a8
commit ef1bb9b
Show file tree

Hide file tree

Showing 10 changed files with 22 additions and 15 deletions.
diff --git a/templates/_helpers.tpl b/templates/_helpers.tpl
@@ -50,6 +50,13 @@ app.kubernetes.io/name: {{ include "jitsu.name" . }}
 app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end }}
 
+{{/*
+Hash, used as a unique label for the Chart version + value set
+*/}}
+{{- define "jitsu.hash" -}}
+{{- sha1sum (printf "%s%s" (toJson .Values) (.Chart.Version)) | substr 0 8 -}}
+{{- end }}
+
 {{/*
 Create the name of the service account to use
 */}}

diff --git a/templates/bulker/deployment.yaml b/templates/bulker/deployment.yaml
@@ -44,7 +44,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.clickhouse.enabled }}
         - name: wait-for-clickhouse
@@ -80,7 +80,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.console.enabled }}
         - name: wait-for-console

diff --git a/templates/console/deployment.yaml b/templates/console/deployment.yaml
@@ -44,7 +44,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.clickhouse.enabled }}
         - name: wait-for-clickhouse
@@ -62,7 +62,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ include "jitsu.hash" . }}"]
         {{- end }}
       containers:
         - name: {{ .Chart.Name }}

diff --git a/templates/event-log-init/job.yaml b/templates/event-log-init/job.yaml
@@ -2,7 +2,7 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
-  name: {{ include "jitsu.fullname" . }}-event-log-init-{{ sha1sum (toJson .Values) | substr 0 8 }}
+  name: {{ include "jitsu.fullname" . }}-event-log-init-{{ include "jitsu.hash" . }}
   labels:
     {{- include "jitsu.labels" . | nindent 4 }}
   {{- with .Values.eventLogInit.jobAnnotations }}
@@ -38,7 +38,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
           {{- end }}
         {{- if .Values.console.enabled }}
         - name: wait-for-console

diff --git a/templates/event-log-trim/cronjob.yaml b/templates/event-log-trim/cronjob.yaml
@@ -2,7 +2,7 @@
 apiVersion: batch/v1
 kind: CronJob
 metadata:
-  name: {{ include "jitsu.fullname" . }}-event-log-trim-{{ sha1sum (toJson .Values) | substr 0 8 }}
+  name: {{ include "jitsu.fullname" . }}-event-log-trim
   labels:
     {{- include "jitsu.labels" . | nindent 4 }}
   {{- with .Values.eventLogTrim.jobAnnotations }}
@@ -41,7 +41,7 @@ spec:
                 {{- toYaml . | nindent 16 }}
               {{- end }}
               image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-              args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+              args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
               {{- end }}
             {{- if .Values.console.enabled }}
             - name: wait-for-console

diff --git a/templates/ingest/deployment.yaml b/templates/ingest/deployment.yaml
@@ -40,7 +40,7 @@ spec:
         {{- if .Values.tokenGenerator.enabled }}
         - name: wait-for-tokens
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.clickhouse.enabled }}
         - name: wait-for-clickhouse

diff --git a/templates/migration/job.yaml b/templates/migration/job.yaml
@@ -2,7 +2,7 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
-  name: {{ include "jitsu.fullname" . }}-migration-{{ sha1sum (toJson .Values) | substr 0 8 }}
+  name: {{ include "jitsu.fullname" . }}-migration-{{ include "jitsu.hash" . }}
   labels:
     {{- include "jitsu.labels" . | nindent 4 }}
   {{- with .Values.migration.jobAnnotations }}
@@ -38,7 +38,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
           {{- end }}
         {{- if .Values.postgresql.enabled }}
         - name: wait-for-postgresql

diff --git a/templates/rotor/deployment.yaml b/templates/rotor/deployment.yaml
@@ -44,7 +44,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.clickhouse.enabled }}
         - name: wait-for-clickhouse

diff --git a/templates/syncctl/deployment.yaml b/templates/syncctl/deployment.yaml
@@ -44,7 +44,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}"]
         {{- end }}
         {{- if .Values.migration.enabled }}
         - name: wait-for-migration
@@ -53,7 +53,7 @@ spec:
             {{- toYaml . | nindent 12 }}
           {{- end }}
           image: ghcr.io/groundnuty/k8s-wait-for:v2.0
-          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ sha1sum (toJson .Values) | substr 0 8 }}"]
+          args: ["job-wr", "{{ include "jitsu.fullname" . }}-migration-{{ include "jitsu.hash" . }}"]
         {{- end }}
       containers:
         - name: {{ .Chart.Name }}

diff --git a/templates/token-generator/job.yaml b/templates/token-generator/job.yaml
@@ -2,7 +2,7 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
-  name: {{ include "jitsu.fullname" . }}-token-generator-{{ sha1sum (toJson .Values) | substr 0 8 }}
+  name: {{ include "jitsu.fullname" . }}-token-generator-{{ include "jitsu.hash" . }}
   labels:
     {{- include "jitsu.labels" . | nindent 4 }}
   {{- with .Values.tokenGenerator.jobAnnotations }}