[incubator/sparkoperator] fixes ISSUE-23875 (#23876)

[] fixes missing colon

bumps version

adds to README

fixes lint error

Signed-off-by: Daniel Moore <moore@knights.ucf.edu>

incubator/sparkoperator/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 name: sparkoperator
 description: A Helm chart for Spark on Kubernetes operator
-version: 0.8.2
+version: 0.8.3
 appVersion: v1beta2-1.2.0-3.0.0
 keywords:
   - spark

incubator/sparkoperator/README.md

@@ -70,6 +70,7 @@ The following table lists the configurable parameters of the Spark operator char
 | `leaderElection.lockName` | Lock name to use for leader election                         | `spark-operator-lock`                  |
 | `leaderElection.lockNamespace` | Namespace to use for leader election                    | (namespace of release)                 |
 | `securityContext` | Defines security context for operator container. | `{}` |
+| `istio.enabled`  			| Whether Jobs will run in service mesh						   | false									| 
 
 Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`.
 

incubator/sparkoperator/templates/crd-cleanup-job.yaml

@@ -14,6 +14,12 @@ metadata:
     app.kubernetes.io/managed-by: {{ .Release.Service }}
 spec:
   template:
+    metadata:
+      name: {{ include "sparkoperator.fullname" . }}-crd-cleanup
+      {{- if .Values.istio.enabled }}
+      annotations:
+        "sidecar.istio.io/inject": "false"
+      {{- end }}
     spec:
       serviceAccountName: {{ include "sparkoperator.serviceAccountName" . }}
       restartPolicy: OnFailure

incubator/sparkoperator/templates/webhook-cleanup-job.yaml

@@ -14,6 +14,12 @@ metadata:
     app.kubernetes.io/managed-by: {{ .Release.Service }}
 spec:
   template:
+    metadata:
+      name: {{ include "sparkoperator.fullname" . }}-webhook-cleanup
+      {{- if .Values.istio.enabled }}
+      annotations:
+        "sidecar.istio.io/inject": "false"
+      {{- end }}
     spec:
       serviceAccountName: {{ include "sparkoperator.serviceAccountName" . }}
       restartPolicy: OnFailure

incubator/sparkoperator/templates/webhook-init-job.yaml

@@ -10,6 +10,12 @@ metadata:
     app.kubernetes.io/instance: {{ .Release.Name }}
     app.kubernetes.io/managed-by: {{ .Release.Service }}
 spec:
+  metadata:
+    name: {{ include "sparkoperator.fullname" . }}-webhook-init
+    {{- if .Values.istio.enabled }}
+    annotations:
+      "sidecar.istio.io/inject": "false"
+    {{- end }}
   template:
     spec:
       serviceAccountName: {{ include "sparkoperator.serviceAccountName" . }}

incubator/sparkoperator/values.yaml

@@ -76,3 +76,7 @@ leaderElection:
   ## Optionally store the lock in another namespace. Defaults to operator's namespace
   # lockNamespace: ""
 replicas: 1
+
+## When using istio Jobs need to run without a sidecar to properly terminate
+istio:
+  enabled: false