Skip to content

Commit

Permalink
Change OTLP receiver port to 4317
Browse files Browse the repository at this point in the history
Port 55680 was deprecated
  • Loading branch information
dmitryax committed Apr 16, 2021
1 parent 38ff878 commit 10c8c05
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ exporters:
{{- if .Values.otelCollector.enabled }}
# If collector is enabled, metrics and traces will be sent to collector
otlp:
endpoint: {{ include "splunk-otel-collector.fullname" . }}:55680
endpoint: {{ include "splunk-otel-collector.fullname" . }}:4317
insecure: true
{{- else }}
# If collector is disabled, metrics and traces will be set to to SignalFx backend
Expand Down
8 changes: 4 additions & 4 deletions helm-charts/splunk-otel-collector/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -107,8 +107,8 @@ otelAgent:
# For example to disable zipkin ports set `otelAgent.ports.zipkin: null`.
ports:
otlp:
containerPort: 55680
hostPort: 55680
containerPort: 4317
hostPort: 4317
protocol: TCP
zipkin:
containerPort: 9411
Expand Down Expand Up @@ -618,7 +618,7 @@ otelCollector:
# Any changes should be alligned with service.ports configuraition below.
ports:
otlp:
containerPort: 55680
containerPort: 4317
protocol: TCP
jaeger-thrift:
containerPort: 14268
Expand Down Expand Up @@ -681,7 +681,7 @@ service:
# Ports exposed by the opentelemetry collector service. Container named ports used.
ports:
otlp:
containerPort: 55680
containerPort: 4317
targetPort: otlp
protocol: TCP
jaeger-thrift:
Expand Down

0 comments on commit 10c8c05

Please sign in to comment.