Skip to content

Commit a69eb2c

Browse files
authored
[jaeger] Fix backend service name in jaeger-ing.yaml (#718)
#### What this PR does It fixes the relationship between the ingress and the service, allowing access to the Query UI as well. It includes short instructions on how to enable the query UI. #### Which issue this PR fixes fixes #719 #### Checklist - [x] [DCO](https://github.com/jaegertracing/helm-charts/blob/main/CONTRIBUTING.md#sign-off-your-work) signed - [x] 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]`) - [x] README.md has been updated to match version/contain new values --------- Signed-off-by: Raphael Pinto <raphaelcpinto@gmail.com>
1 parent a27013b commit a69eb2c

3 files changed

Lines changed: 14 additions & 2 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.3.1
6+
version: 4.3.2
77
# CronJobs require v1.21
88
kubeVersion: ">= 1.21-0"
99
keywords:

charts/jaeger/README.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,18 @@ spark:
124124

125125
For a full list of supported environment variables, see the [Spark Dependencies README](https://github.com/jaegertracing/spark-dependencies#readme).
126126

127+
### 5. Query UI
128+
129+
To enable the query ui, you need to enable the ingress and fill at least one host:
130+
131+
```yaml
132+
jaeger:
133+
ingress:
134+
enabled: true
135+
hosts:
136+
- <fill a host here>
137+
```
138+
127139
## Configuring the Collector
128140

129141
The Jaeger v2 configuration is defined in `config` using OpenTelemetry Collector syntax. You can override pipelines, receivers, and processors there.

charts/jaeger/templates/jaeger-ing.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ spec:
2121
paths:
2222
- path: /
2323
pathType: {{ default "ImplementationSpecific" $.Values.jaeger.ingress.pathType }}
24-
backend: {{- include "common.ingress.backend" (dict "serviceName" (printf "%s-query" (include "jaeger.fullname" $)) "servicePort" 16686 "context" $) | nindent 14 }}
24+
backend: {{- include "common.ingress.backend" (dict "serviceName" (include "jaeger.fullname" $) "servicePort" 16686 "context" $) | nindent 14 }}
2525
{{- end -}}
2626
{{- if .Values.jaeger.ingress.tls }}
2727
tls:

0 commit comments

Comments
 (0)