Skip to content

A108: OpenTelemetry Custom Per-Call Metric Label#525

Merged
ejona86 merged 4 commits intogrpc:masterfrom
ejona86:custom-per-call-label
Mar 17, 2026
Merged

A108: OpenTelemetry Custom Per-Call Metric Label#525
ejona86 merged 4 commits intogrpc:masterfrom
ejona86:custom-per-call-label

Conversation

@ejona86
Copy link
Copy Markdown
Member

@ejona86 ejona86 commented Dec 5, 2025

No description provided.


### Go

gRPC Go will add a new API to set and get the custom label value in
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@easwars, @dfawley, if you know the spellings of these APIs I can put them in here.

Comment thread A108-otel-custom-per-call-label.md Outdated
@ejona86
Copy link
Copy Markdown
Member Author

ejona86 commented Dec 16, 2025

CC @ctiller

Copy link
Copy Markdown
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

Overall, this looks good! I just want to chat with @ctiller before approving, just to make sure we've dotted our I's and crossed our T's.

* `grpc.client.call.retry_delay`
* Other per-call instruments, e.g., those added by an LB policy, are encouraged
to support this label
* RLS is the only such case today, but is not defined in a gRFC
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd suggest just removing this bullet, since it isn't actually a public API. If we want this label on these metrics, we can figure that out internally.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unless there is a strong objection, I'm going to leave it, because "figure it out internally" has not been going well, and I basically need to define nothing here. And the code is OSS and I made all the changes to Java in the same commit.

Without this bullet, I'd also need to say "there are no such cases today" to be clear I'm giving the exhaustive list, but that comment would be incorrect because RLS does exist.

Comment thread A108-otel-custom-per-call-label.md Outdated
Comment thread A108-otel-custom-per-call-label.md
@ejona86
Copy link
Copy Markdown
Member Author

ejona86 commented Feb 10, 2026

@markdroth, @easwars, @dfawley, friendly ping for review.

@ejona86 ejona86 requested a review from markdroth March 16, 2026 20:03
@ejona86 ejona86 merged commit 1b4d14d into grpc:master Mar 17, 2026
1 check passed
@ejona86 ejona86 deleted the custom-per-call-label branch March 17, 2026 23:17
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