feat: configurable job names (#224)

* feat: configurable job names

set jobName for manual job naming

* docs: configurable job names

erpnext/README.md

@@ -87,6 +87,7 @@ Kubernetes Helm Chart for the latest stable ERPNext branch
 | jobs.backup.affinity | object | `{}` |  |
 | jobs.backup.backoffLimit | int | `0` |  |
 | jobs.backup.enabled | bool | `false` |  |
+| jobs.backup.jobName | string | `""` |  |
 | jobs.backup.nodeSelector | object | `{}` |  |
 | jobs.backup.resources | object | `{}` |  |
 | jobs.backup.siteName | string | `"erp.cluster.local"` |  |
@@ -99,6 +100,7 @@ Kubernetes Helm Chart for the latest stable ERPNext branch
 | jobs.configure.enabled | bool | `true` |  |
 | jobs.configure.envVars | list | `[]` |  |
 | jobs.configure.fixVolume | bool | `true` |  |
+| jobs.configure.jobName | string | `""` |  |
 | jobs.configure.nodeSelector | object | `{}` |  |
 | jobs.configure.resources | object | `{}` |  |
 | jobs.configure.tolerations | list | `[]` |  |
@@ -109,6 +111,7 @@ Kubernetes Helm Chart for the latest stable ERPNext branch
 | jobs.createSite.enabled | bool | `false` |  |
 | jobs.createSite.forceCreate | bool | `false` |  |
 | jobs.createSite.installApps[0] | string | `"erpnext"` |  |
+| jobs.createSite.jobName | string | `""` |  |
 | jobs.createSite.nodeSelector | object | `{}` |  |
 | jobs.createSite.resources | object | `{}` |  |
 | jobs.createSite.siteName | string | `"erp.cluster.local"` |  |
@@ -128,13 +131,15 @@ Kubernetes Helm Chart for the latest stable ERPNext branch
 | jobs.dropSite.backoffLimit | int | `0` |  |
 | jobs.dropSite.enabled | bool | `false` |  |
 | jobs.dropSite.forced | bool | `false` |  |
+| jobs.dropSite.jobName | string | `""` |  |
 | jobs.dropSite.nodeSelector | object | `{}` |  |
 | jobs.dropSite.resources | object | `{}` |  |
 | jobs.dropSite.siteName | string | `"erp.cluster.local"` |  |
 | jobs.dropSite.tolerations | list | `[]` |  |
 | jobs.migrate.affinity | object | `{}` |  |
 | jobs.migrate.backoffLimit | int | `0` |  |
 | jobs.migrate.enabled | bool | `false` |  |
+| jobs.migrate.jobName | string | `""` |  |
 | jobs.migrate.nodeSelector | object | `{}` |  |
 | jobs.migrate.resources | object | `{}` |  |
 | jobs.migrate.siteName | string | `"erp.cluster.local"` |  |
@@ -143,6 +148,7 @@ Kubernetes Helm Chart for the latest stable ERPNext branch
 | jobs.volumePermissions.affinity | object | `{}` |  |
 | jobs.volumePermissions.backoffLimit | int | `0` |  |
 | jobs.volumePermissions.enabled | bool | `false` |  |
+| jobs.volumePermissions.jobName | string | `""` |  |
 | jobs.volumePermissions.nodeSelector | object | `{}` |  |
 | jobs.volumePermissions.resources | object | `{}` |  |
 | jobs.volumePermissions.tolerations | list | `[]` |  |

erpnext/templates/job-backup.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.backup.jobName }}
+  name: {{ .Values.jobs.backup.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-backup-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
 spec:

erpnext/templates/job-configure-bench.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.configure.jobName }}
+  name: {{ .Values.jobs.configure.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-conf-bench-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
 spec:

erpnext/templates/job-create-site.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.createSite.jobName }}
+  name: {{ .Values.jobs.createSite.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-new-site-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
   annotations:
@@ -114,7 +118,7 @@ spec:
                     exit $bench_exit_status;
                 fi
             fi
- 
+
             set -e;
 
             rm -f currentsite.txt

erpnext/templates/job-drop-site.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.dropSite.jobName }}
+  name: {{ .Values.jobs.dropSite.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-drop-site-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
 spec:

erpnext/templates/job-fix-volume-permission.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.volumePermissions.jobName }}
+  name: {{ .Values.jobs.volumePermissions.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-vol-fix-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
 spec:

erpnext/templates/job-migrate-site.yaml

@@ -2,7 +2,11 @@
 apiVersion: batch/v1
 kind: Job
 metadata:
+  {{- if .Values.jobs.migrate.jobName }}
+  name: {{ .Values.jobs.migrate.jobName }}
+  {{- else }}
   name: {{ template "erpnext.fullname" . }}-migrate-{{ now | date "20060102150405" }}
+  {{- end }}
   labels:
     {{- include "erpnext.labels" . | nindent 4 }}
 spec:

erpnext/values.yaml

@@ -288,6 +288,7 @@ ingress:
 
 jobs:
   volumePermissions:
+    # jobName: ""
     enabled: false
     backoffLimit: 0
     resources: {}
@@ -296,6 +297,7 @@ jobs:
     affinity: {}
 
   configure:
+    # jobName: ""
     enabled: true
     fixVolume: true
     backoffLimit: 0
@@ -308,6 +310,7 @@ jobs:
     args: []
 
   createSite:
+    # jobName: ""
     enabled: false
     forceCreate: false
     siteName: "erp.cluster.local"
@@ -322,6 +325,7 @@ jobs:
     affinity: {}
 
   dropSite:
+    # jobName: ""
     enabled: false
     forced: false
     siteName: "erp.cluster.local"
@@ -332,6 +336,7 @@ jobs:
     affinity: {}
 
   backup:
+    # jobName: ""
     enabled: false
     siteName: "erp.cluster.local"
     withFiles: true
@@ -342,6 +347,7 @@ jobs:
     affinity: {}
 
   migrate:
+    # jobName: ""
     enabled: false
     siteName: "erp.cluster.local"
     skipFailing: false