Skip to content

Commit

Permalink
[collector] replace Jaeger to OTLP exporter (#435)
Browse files Browse the repository at this point in the history
  • Loading branch information
Juliano Costa authored Oct 13, 2022
1 parent 1539aca commit e41ee24
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 5 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -113,3 +113,5 @@ significant modifications will be credited to OpenTelemetry Authors.
([#339](https://github.com/open-telemetry/opentelemetry-demo/pull/339))
* Added basic metrics support for recommendation service (Python)
([#416](https://github.com/open-telemetry/opentelemetry-demo/pull/416))
* Replaced the Jaeger exporter to the OTLP exporter in the OTel Collector
([#435](https://github.com/open-telemetry/opentelemetry-demo/pull/435))
6 changes: 4 additions & 2 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,16 @@ services:
command:
- "--memory.max-traces"
- "10000"
environment:
- COLLECTOR_OTLP_ENABLED=true
ports:
- "16686:16686" # Jaeger UI
- "14250" # Jaeger model.proto endpoint
- "4317" # OTLP gRPC default port
logging: *logging

# Collector
otelcol:
image: otel/opentelemetry-collector-contrib:0.56.0
image: otel/opentelemetry-collector-contrib:0.61.0
container_name: otel-col
command: [ "--config=/etc/otelcol-config.yml", "--config=/etc/otelcol-config-extras.yml" ]
volumes:
Expand Down
6 changes: 3 additions & 3 deletions src/otelcollector/otelcol-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ receivers:
endpoint: "localhost:65535"

exporters:
jaeger:
endpoint: "jaeger:14250"
otlp:
endpoint: "jaeger:4317"
tls:
insecure: true
logging:
Expand All @@ -31,7 +31,7 @@ service:
traces:
receivers: [otlp]
processors: [spanmetrics, batch]
exporters: [logging, jaeger]
exporters: [logging, otlp]
metrics:
receivers: [otlp]
processors: [batch]
Expand Down

0 comments on commit e41ee24

Please sign in to comment.