Decouple server job manifests from Values.server.enabled

charts/temporal/templates/server-job.yaml

@@ -1,4 +1,4 @@
-{{- if $.Values.server.enabled }}{{- if or $.Values.schema.createDatabase.enabled $.Values.schema.setup.enabled $.Values.schema.update.enabled }}
+{{- if or $.Values.schema.createDatabase.enabled $.Values.schema.setup.enabled $.Values.schema.update.enabled }}
 apiVersion: batch/v1
 kind: Job
 metadata:
@@ -220,4 +220,4 @@ spec:
       volumes:
         {{- toYaml . | nindent 8 }}
       {{- end }}
-{{- end -}}{{- end -}}
+{{- end -}}

charts/temporal/templates/server-secret.yaml

@@ -1,4 +1,4 @@
-{{- if $.Values.server.enabled }}
+{{- if or $.Values.server.enabled $.Values.schema.createDatabase.enabled $.Values.schema.setup.enabled $.Values.schema.update.enabled }}
   {{- range $store := (list "default" "visibility") }}
   {{- $storeConfig := index $.Values.server.config.persistence $store }}
   {{- $driver := include "temporal.persistence.driver" (list $ $store) -}}

charts/temporal/tests/server_job_test.yaml

@@ -2,6 +2,14 @@ suite: test server job
 templates:
   - server-job.yaml
 tests:
+  - it: is able to create the server job even if Values.server.enabled is false
+    set:
+      server:
+        enabled: false
+    asserts:
+      - containsDocument:
+          kind: Job
+          apiVersion: batch/v1
   - it: includes additional init containers
     set:
       admintools:

charts/temporal/tests/server_secret_test.yaml

@@ -0,0 +1,12 @@
+suite: test server secret
+templates:
+  - server-secret.yaml
+tests:
+  - it: creates the secret when the server job is enabled, even when server.enabled is false
+    set:
+      server:
+        enabled: false
+    asserts:
+      - containsDocument:
+          kind: Secret
+          apiVersion: v1