Skip to content

Commit

Permalink
update _helpers.tpl consul.ValidateDatadogConfiguration func to accou…
Browse files Browse the repository at this point in the history
…nt for using 'https' as protocol => should fail
  • Loading branch information
natemollica-nm committed Feb 9, 2024
1 parent 307991b commit a139223
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 .Values.global.metrics.datadog.otlp.enabled (or (eq (.Values.global.metrics.datadog.otlp.protocol | trimAll "\"" | quote) "http") (eq (.Values.global.metrics.datadog.otlp.protocol | trimAll "\"" | quote) "grpc")) }}
{{- if and (ne .Values.global.metrics.datadog.otlp.protocol "http") (ne .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 a139223

Please sign in to comment.