Skip to content

Commit 9ff9110

Browse files
committed
[jaeger] Re-add anonymous Elasticsearch access support
The `storage.elasticsearch.anonymous` option was added in PR #442 (chart v0.67.1) but was lost when the chart was rewritten for Jaeger v2. The elasticsearch.env helper unconditionally injects ES_USERNAME and ES_PASSWORD with defaults (elastic/changeme), which breaks AWS OpenSearch deployments that use VPC-level access without basic auth. AWS rejects the basic auth Authorization header with a 403 because it expects SigV4 format. Setting `storage.elasticsearch.anonymous: true` now skips injecting ES_USERNAME and ES_PASSWORD, restoring the behavior from #442. Signed-off-by: domolitom <domolitom@gmail.com>
1 parent 1e11ae9 commit 9ff9110

3 files changed

Lines changed: 4 additions & 1 deletion

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.15.1
33
description: A Jaeger Helm chart for Kubernetes
44
name: jaeger
55
type: application
6-
version: 4.5.0
6+
version: 4.5.1
77
# Artifact Hub annotations
88
# The jaeger image is whitelisted from security scanning because the reported
99
# CVEs are in the upstream Alpine base image (OpenSSL libcrypto3/libssl3) and

charts/jaeger/templates/_helpers.tpl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,10 +189,12 @@ Elasticsearch related environment variables
189189
value: {{ $url | quote }}
190190
- name: ES_NODES
191191
value: {{ $url | quote }}
192+
{{- if not $es.anonymous }}
192193
- name: ES_USERNAME
193194
value: {{ $user | quote }}
194195
- name: ES_PASSWORD
195196
value: {{ $password | quote }}
197+
{{- end }}
196198
{{- /* Handle TLS insecurity */ -}}
197199
{{- if and (($es).tls).enabled (($es).tls).insecure }}
198200
- name: ES_TLS_SKIP_HOST_VERIFY

charts/jaeger/values.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -268,6 +268,7 @@ storage:
268268
mountPath: ""
269269
subPath: ""
270270
url: http://elasticsearch-master:9200
271+
anonymous: false
271272
# user: elastic
272273
# password: changeme
273274

0 commit comments

Comments
 (0)