Skip to content

fix(generator): add otel address#1986

Open
L2D2Grafana wants to merge 1 commit into
mainfrom
l2d2/fix-generator-tracing
Open

fix(generator): add otel address#1986
L2D2Grafana wants to merge 1 commit into
mainfrom
l2d2/fix-generator-tracing

Conversation

@L2D2Grafana

Copy link
Copy Markdown
Collaborator

Summary 📝

  • Fix the local dev trace pipeline: Tempo's OTLP receiver in config/tempo.yaml had no explicit endpoint, so with the newer OTel Collector bundled in Tempo 2.10.x it defaulted to localhost — unreachable from the generator container. Traces never reached Tempo and Traces Drilldown showed "No data". Bind OTLP gRPC/HTTP explicitly to 0.0.0.0:4317 / 0.0.0.0:4318.
  • Remove the deprecated opencensus receiver from the Tempo distributor config.

Test 🧪

  • Restarted Tempo and confirmed the OTLP gRPC server now listens on [::]:4317 (previously 127.0.0.1:4317).
  • tempo_distributor_spans_received_total went from 0 to >27k after the change — Tempo is ingesting spans again.
  • generator logs no longer emit dial tcp …:4317: connect: connection refused; traces render in Traces Drilldown.

Copilot AI review requested due to automatic review settings July 8, 2026 16:51
@L2D2Grafana L2D2Grafana requested a review from a team as a code owner July 8, 2026 16:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the local development tracing pipeline by explicitly binding Tempo’s OTLP receiver endpoints to listen on all interfaces, ensuring traces sent from other containers (e.g., generator) can reach Tempo instead of being blocked by a localhost-only default.

Changes:

  • Set explicit OTLP HTTP and gRPC endpoint bindings to 0.0.0.0:4318 and 0.0.0.0:4317 in Tempo’s distributor receiver config.
  • Remove the deprecated opencensus receiver from the Tempo distributor configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants