fix: don't use generated name for hook

parcellab/common/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: common
 description: A Helm chart library for parcelLab charts
 type: library
-version: 1.2.0
+version: 1.2.2
 maintainers:
   - name: parcelLab
     email: engineering@parcellab.com

parcellab/common/templates/_job.tpl

@@ -6,13 +6,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  name: {{ $name }}
   {{- if $job.hook }}
-  generateName: {{ $name }}-
   annotations:
-    argocd.argoproj.io/hook: {{ default "Skip" $job.hook }}
-    argocd.argoproj.io/hook-delete-policy: {{ default "HookSucceeded" $job.hookDeletePolicy }}
-  {{- else }}
-  name: {{ $name }}
+    argocd.argoproj.io/hook: {{ (default "Skip" $job.hook) | quote }}
+    argocd.argoproj.io/hook-delete-policy: {{ (default "BeforeHookCreation" $job.hookDeletePolicy) | quote }}
   {{- end }}
   labels:
     {{- include "common.labels" $componentValues | nindent 4 }}

parcellab/cronjob/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v2
 name: cronjob
 description: Single cron job
-version: 0.4.0
+version: 0.4.1
 dependencies:
   - name: common
     version: "*"

parcellab/microservice/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v2
 name: microservice
 description: Simple microservice
-version: 0.4.0
+version: 0.4.1
 dependencies:
   - name: common
     version: "*"

parcellab/monolith/values.yaml

@@ -124,7 +124,7 @@ job:
 jobs: []
 #  - name: migrate
 #    hook: PreSync
-#    hookDeletePolicy: HookSucceeded
+#    hookDeletePolicy: HookFailed
 #    command:
 #      - /bin/sh
 #      - -c

parcellab/worker-group/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v2
 name: worker-group
 description: Set of workers that do not expose a service
-version: 0.3.0
+version: 0.3.1
 dependencies:
   - name: common
     version: "*"