Skip to content

Deprecate rpc.*.requests_per_rpc and rpc.*.responses_per_rpc metrics#2846

Merged
joaopgrassi merged 2 commits intoopen-telemetry:mainfrom
lmolkova:deprecate-per-rpc-metrics
Oct 7, 2025
Merged

Deprecate rpc.*.requests_per_rpc and rpc.*.responses_per_rpc metrics#2846
joaopgrassi merged 2 commits intoopen-telemetry:mainfrom
lmolkova:deprecate-per-rpc-metrics

Conversation

@lmolkova
Copy link
Copy Markdown
Member

@lmolkova lmolkova commented Oct 1, 2025

Fixes #2784

It might be interesting to know number of messages per RPC for streaming cases, but current metrics have confusing names (requests != messages).

Deprecating them now and we can always come back with better conventions later when we work on streaming support.

@lmolkova lmolkova moved this from Untriaged to Needs More Approval in Semantic Conventions Triage Oct 1, 2025
Comment thread model/rpc/deprecated/metrics-deprecated.yaml Outdated
@joaopgrassi joaopgrassi moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Oct 7, 2025
@joaopgrassi joaopgrassi enabled auto-merge October 7, 2025 11:43
@joaopgrassi joaopgrassi added this pull request to the merge queue Oct 7, 2025
Merged via the queue into open-telemetry:main with commit 9f21316 Oct 7, 2025
16 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in RPC Semantic Conventions Stability Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

Deprecate rpc.(client|server).(requests|responses)_per_rpc metrics

3 participants