Skip to content

cmd/geth, internal/telemetry: wire OpenTelemetry tracing via CLI flags#33484

Merged
fjl merged 19 commits intoethereum:masterfrom
jrhea:rpc-otel-cli
Feb 14, 2026
Merged

cmd/geth, internal/telemetry: wire OpenTelemetry tracing via CLI flags#33484
fjl merged 19 commits intoethereum:masterfrom
jrhea:rpc-otel-cli

Conversation

@jrhea
Copy link
Copy Markdown
Contributor

@jrhea jrhea commented Dec 23, 2025

Summary

This PR adds OpenTelemetry tracing configuration to geth via command-line flags. When enabled, geth initializes the global OpenTelemetry TracerProvider and installs standard trace context propagation. When disabled (the default), tracing remains a no-op and behavior is unchanged.

Comment thread internal/otel/otel.go Outdated
Comment thread cmd/utils/flags.go Outdated
Comment thread cmd/utils/flags.go
Comment thread cmd/utils/rpc_tracing.go Outdated
@jrhea jrhea marked this pull request as draft January 6, 2026 23:44
Comment thread cmd/utils/flags.go Outdated
@jrhea jrhea force-pushed the rpc-otel-cli branch 3 times, most recently from 1509464 to 9e0078d Compare January 12, 2026 18:11
@jrhea jrhea marked this pull request as ready for review January 12, 2026 18:13
@jrhea jrhea changed the title cmd/geth: wire OpenTelemetry tracing via CLI flags cmd/geth, internal/telemetry: wire OpenTelemetry tracing via CLI flags Jan 14, 2026
@jrhea jrhea force-pushed the rpc-otel-cli branch 2 times, most recently from eafcef9 to cb14351 Compare January 14, 2026 21:37
@fjl fjl added this to the 1.17.0 milestone Jan 15, 2026
jrhea and others added 3 commits February 2, 2026 18:26
Changes are made to allows configuring tracing via TOML config file.
The tracing config is now a part of the node.Config.
fjl
fjl previously approved these changes Feb 3, 2026
@fjl fjl requested a review from s1na as a code owner February 3, 2026 14:08
@fjl fjl merged commit c50e5ed into ethereum:master Feb 14, 2026
7 of 8 checks passed
@jrhea jrhea deleted the rpc-otel-cli branch February 14, 2026 01:25
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