Skip to content

Commit b497b82

Browse files
committed
chart: fix otel url and allow overriding tag
Signed-off-by: clux <[email protected]>
1 parent 3504368 commit b497b82

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

charts/doc-controller/templates/_helpers.tpl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,9 @@ app: {{ include "controller.name" . }}
1818
{{- end }}
1919

2020
{{- define "controller.tag" -}}
21-
{{- if .Values.tracing.enabled }}
21+
{{- if .Values.image.tag }}
22+
{{- .Values.image.tag }}
23+
{{- else if .Values.tracing.enabled }}
2224
{{- "otel-" }}{{ .Values.version | default .Chart.AppVersion }}
2325
{{- else }}
2426
{{- .Values.version | default .Chart.AppVersion }}

charts/doc-controller/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
value: {{ .Values.logging.env_filter }}
4646
{{- if .Values.tracing.enabled }}
4747
- name: OPENTELEMETRY_ENDPOINT_URL
48-
value: https://{{ .Values.tracing.service }}.{{ .Values.tracing.namespace }}.cluster.local:{{ .Values.tracing.port }}
48+
value: http://{{ .Values.tracing.service }}.{{ .Values.tracing.namespace }}.svc:{{ .Values.tracing.port }}
4949
{{- end }}
5050
{{- with .Values.env }}
5151
{{- toYaml . | nindent 8 }}

0 commit comments

Comments
 (0)