Skip to content

Retrieve gRPC server.address/server.port from gRPC target#16161

Draft
trask wants to merge 3 commits intoopen-telemetry:mainfrom
trask:grpc-server-address
Draft

Retrieve gRPC server.address/server.port from gRPC target#16161
trask wants to merge 3 commits intoopen-telemetry:mainfrom
trask:grpc-server-address

Conversation

@trask
Copy link
Copy Markdown
Member

@trask trask commented Feb 12, 2026

Currently built on top of

Prototype for open-telemetry/semantic-conventions#3317

Java agent instrumentation can do this for all gRPC versions.

Library instrumentation can only do this (automatically) using an internal gRPC API available since version 1.64.0: InternalManagedChannelBuilder.

I'm not convinced about the API / dependency on internals, could limit us on marking it stable, but marking ready-for-review to get feedback.

Related to #15871

Copy link
Copy Markdown
Member

@lmolkova lmolkova left a comment

Choose a reason for hiding this comment

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

Looks great (assuming you're comfortable depending on '@internal public' API)

@trask trask force-pushed the grpc-server-address branch from c45ee61 to ce22789 Compare February 13, 2026 04:14
@trask trask marked this pull request as ready for review February 13, 2026 22:54
@trask trask requested a review from a team as a code owner February 13, 2026 22:54
@trask trask force-pushed the grpc-server-address branch from 9a1c3d7 to 2a0e0b9 Compare February 17, 2026 17:14
@trask trask force-pushed the grpc-server-address branch 5 times, most recently from 0f60636 to dd4d89f Compare February 26, 2026 04:09
@trask trask marked this pull request as draft February 26, 2026 04:28
@trask trask force-pushed the grpc-server-address branch 6 times, most recently from d93c7e4 to e19f9d2 Compare March 1, 2026 02:27
@trask trask force-pushed the grpc-server-address branch from e19f9d2 to 9ae10a5 Compare March 2, 2026 21:05
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.

3 participants