Skip to content

Commit

Permalink
correct otlp protocol helpers.tpl check to lower-case the protocol to…
Browse files Browse the repository at this point in the history
… match the opent-telemetry-deployment.yaml behavior
  • Loading branch information
natemollica-nm committed Feb 9, 2024
1 parent 5559f02 commit 436884b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion charts/consul/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -637,7 +637,7 @@ Usage: {{ template "consul.validateDatadogConfiguration" . }}
{{- if and .Values.global.metrics.datadog.otlp.enabled (not .Values.telemetryCollector.enabled) }}
{{fail "Cannot enable Datadog OTLP metrics collection (global.metrics.datadog.otlp.enabled) without consul-telemetry-collector. Ensure Consul OTLP collection is enabled (telemetryCollector.enabled) and configured." }}
{{- end }}
{{- if and (ne .Values.global.metrics.datadog.otlp.protocol "http") (ne .Values.global.metrics.datadog.otlp.protocol "grpc") }}
{{- if and (ne ( lower .Values.global.metrics.datadog.otlp.protocol) "http") (ne ( lower .Values.global.metrics.datadog.otlp.protocol) "grpc") }}
{{fail "Valid values for global.metrics.datadog.otlp.protocol must be one of either \"http\" or \"grpc\"." }}
{{- end }}
{{- end -}}
Expand Down

0 comments on commit 436884b

Please sign in to comment.