Skip to content

Commit

Permalink
[jaeger] Add securityContext for spark cron job (#587)
Browse files Browse the repository at this point in the history
Signed-off-by: Andreas Seelinger <[email protected]>
Signed-off-by: Pavel Nikolov <[email protected]>
Co-authored-by: Pavel Nikolov <[email protected]>
  • Loading branch information
brainstorm82 and pavelnikolov authored Aug 28, 2024
1 parent 26e6439 commit 3060749
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/jaeger/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ appVersion: 1.53.0
description: A Jaeger Helm chart for Kubernetes
name: jaeger
type: application
version: 3.1.2
version: 3.1.3
# CronJobs require v1.21
kubeVersion: ">= 1.21-0"
keywords:
Expand Down
4 changes: 4 additions & 0 deletions charts/jaeger/templates/spark-cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ spec:
spec:
serviceAccountName: {{ template "jaeger.spark.serviceAccountName" . }}
{{- include "spark.imagePullSecrets" . | nindent 10 }}
securityContext:
{{- toYaml .Values.spark.podSecurityContext | nindent 12 }}
containers:
- name: {{ include "jaeger.fullname" . }}-spark
image: {{ include "spark.image" . }}
Expand Down Expand Up @@ -76,6 +78,8 @@ spec:
subPath: {{ .subPath }}
readOnly: {{ .readOnly }}
{{- end }}
securityContext:
{{- toYaml .Values.spark.securityContext | nindent 14 }}
restartPolicy: OnFailure
volumes:
{{- range .Values.spark.extraConfigmapMounts }}
Expand Down
2 changes: 2 additions & 0 deletions charts/jaeger/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -790,6 +790,8 @@ query:

spark:
enabled: false
securityContext: {}
podSecurityContext: {}
annotations: {}
image:
registry: ""
Expand Down

0 comments on commit 3060749

Please sign in to comment.