Skip to content

Commit

Permalink
[jaeger] Add securityContext for the oauth sidecar in query-deploy.ya…
Browse files Browse the repository at this point in the history
…ml (#600)

The sidecar is missing the securityContext entry, and it makes it
difficult to apply specific settings in the context of K8s PSS
(migrating from PSP for example).

Signed-off-by: Luca Toscano <[email protected]>
  • Loading branch information
elukey authored Sep 11, 2024
1 parent 4b3ba57 commit 3f88797
Show file tree
Hide file tree
Showing 2 changed files with 3 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.2.0
version: 3.2.1
# CronJobs require v1.21
kubeVersion: ">= 1.21-0"
keywords:
Expand Down
2 changes: 2 additions & 0 deletions charts/jaeger/templates/query-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,8 @@ spec:
port: admin
{{- if .Values.query.oAuthSidecar.enabled }}
- name: {{ template "jaeger.agent.name" . }}-oauth2-sidecar
securityContext:
{{- toYaml .Values.query.securityContext | nindent 10 }}
image: {{ include "oAuthSidecar.image" . }}
imagePullPolicy: {{ .Values.query.oAuthSidecar.image.pullPolicy }}
args:
Expand Down

0 comments on commit 3f88797

Please sign in to comment.