search

jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
258 stars 340 forks source link

[Feature]: It's not possible to use Cassandra without authorization. #460

Open AVBIT opened 1 year ago

AVBIT commented 1 year ago

Requirement

use Cassandra without authorization.

Problem

It's not possible to use Cassandra without authorization.

Proposal

modify file charts/jaeger/templates/_helpers.tpl... append condition .Values.storage.cassandra.usePassword:

{{- if .Values.storage.cassandra.usePassword }}
- name: CASSANDRA_USERNAME
  value: {{ .Values.storage.cassandra.user }}
- name: CASSANDRA_PASSWORD
  valueFrom:
    secretKeyRef:
      name: {{ if .Values.storage.cassandra.existingSecret }}{{ .Values.storage.cassandra.existingSecret }}{{- else }}{{ include "jaeger.fullname" . }}-cassandra{{- end }}
      key: password
{{- end }}

I can't make a pull request due to lack of rights ((

Open questions

No response