Skip to content

Add dual-semconv support to RPC metrics#16297

Closed
zeitlinger wants to merge 1 commit into
open-telemetry:mainfrom
zeitlinger:rpc-metrics-dual-semconv
Closed

Add dual-semconv support to RPC metrics#16297
zeitlinger wants to merge 1 commit into
open-telemetry:mainfrom
zeitlinger:rpc-metrics-dual-semconv

Conversation

@zeitlinger

Copy link
Copy Markdown
Member

Part of #15932 — split into smaller reviewable PRs. This is PR 4 of 5 (depends on #16130).

  • Add stable metric rpc.client.call.duration (seconds) alongside old rpc.client.duration (milliseconds)
  • Add stable metric rpc.server.call.duration (seconds) alongside old rpc.server.duration (milliseconds)
  • Update RpcMetricsAdvice with separate old/stable attribute key lists (rpc.system.name, rpc.response.status_code for stable)
  • Add OLD_RPC_METHOD_CONTEXT_KEY infrastructure for correct rpc.method handling in dup mode
  • Size metrics (rpc.*.request.size, rpc.*.response.size) remain old-only
  • Tests validate all three modes: old-only, stable-only, and dup

#15871

@zeitlinger zeitlinger requested a review from a team as a code owner February 26, 2026 11:09
@zeitlinger zeitlinger closed this Feb 26, 2026
@zeitlinger zeitlinger deleted the rpc-metrics-dual-semconv branch February 26, 2026 11:15
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.

1 participant