Skip to content

Commit 0d01611

Browse files
[jaeger ]Use subPath for mounting user and ui configs (#726)
#### What this PR does - Fixes jaeger-deploy.yaml template to use subPath for mounting user-config and ui-config files. Currently files are mounted in user-config and ui-config directories respectively which brakes config load command. #### Checklist - [x] [DCO](https://github.com/jaegertracing/helm-charts/blob/main/CONTRIBUTING.md#sign-off-your-work) signed - [ ] Commits are [GPG signed](https://docs.github.com/en/github/authenticating-to-github/about-commit-signature-verification) - [x] Chart Version bumped - [x] Title of the PR starts with chart name (`[jaeger]` or `[jaeger-operator]`) - [ ] README.md has been updated to match version/contain new values Signed-off-by: Lukas Stankovičius <lukas.stankovicius@gmail.com>
1 parent ca467f4 commit 0d01611

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

charts/jaeger/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ appVersion: 2.14.1
33
description: A Jaeger Helm chart for Kubernetes
44
name: jaeger
55
type: application
6-
version: 4.4.0
6+
version: 4.4.1
77
# CronJobs require v1.21
88
kubeVersion: ">= 1.21-0"
99
keywords:

charts/jaeger/templates/jaeger/jaeger-deploy.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,13 @@ spec:
9595
volumeMounts:
9696
{{- if .Values.userconfig }}
9797
- name: user-config
98-
mountPath: /etc/jaeger/user-config
98+
mountPath: /etc/jaeger/user-config.yaml
99+
subPath: user-config.yaml
99100
{{- end }}
100101
{{- if .Values.uiconfig }}
101102
- name: ui-config
102-
mountPath: /etc/jaeger/ui-config
103+
mountPath: /etc/jaeger/ui-config.json
104+
subPath: ui-config.json
103105
{{- end }}
104106
securityContext:
105107
{{- toYaml .Values.jaeger.podSecurityContext | nindent 8 }}

0 commit comments

Comments
 (0)