Skip to content

[prometheus]: Stabilize OTLP -> Prometheus Summaries#5090

Closed
johannaojeling wants to merge 2 commits into
open-telemetry:mainfrom
johannaojeling:otlp-prometheus-stabilize-summaries
Closed

[prometheus]: Stabilize OTLP -> Prometheus Summaries#5090
johannaojeling wants to merge 2 commits into
open-telemetry:mainfrom
johannaojeling:otlp-prometheus-stabilize-summaries

Conversation

@johannaojeling

Copy link
Copy Markdown
Member

Fixes #4923

Changes

Stabilizes the conversion of OTLP Summaries into Prometheus Summaries.

@johannaojeling
johannaojeling requested review from a team as code owners May 12, 2026 10:44
Comment thread specification/compatibility/prometheus_and_openmetrics.md Outdated
Co-authored-by: David Ashpole <dashpole@google.com>
@ArthurSens

ArthurSens commented May 13, 2026

Copy link
Copy Markdown
Member

@johannaojeling, I believe David's suggestion was not so literal that we can just copy and paste it into the spec.

Now we have duplicated information, for example, at the top we say:

- A single `{name}_count` metric denoting the count field of the summary.

And at the bottom, we repeat the same information

 The count is converted to the Summary's count.

My interpretation is that the previous way we documented A single {name}_count... is overly focused on the text exposition format and doesn't apply to other Prometheus protocols, such as Remote Write or future improvements planned on the Prometheus side (prometheus/prometheus#16949).

Could you reword the section to be less specific about text format?

@dashpole

Copy link
Copy Markdown
Contributor

Ah, sorry. I meant to replace the paragraph above, but seem to have messed up the comment anchor.

@johannaojeling

Copy link
Copy Markdown
Member Author

Closing this as it is being addressed in #5107

@johannaojeling
johannaojeling deleted the otlp-prometheus-stabilize-summaries branch May 25, 2026 07:52
pull Bot pushed a commit to CodeWeaver13/opentelemetry-specification that referenced this pull request Jun 2, 2026
Fixes open-telemetry#4923

## Changes

Stabilizes the transformation of OTLP Summaries to Prometheus summaries.

This PR superseeds open-telemetry#5090, since @johannaojeling is out on PTO for a
couple of weeks

For non-trivial changes, follow the [change proposal
process](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CONTRIBUTING.md#proposing-a-change).

* [X] Related issues open-telemetry#4923 
* [ ] Related [OTEP(s)](https://github.com/open-telemetry/oteps) #
* [ ] Links to the prototypes (when adding or changing features)
* [X]
[`CHANGELOG.md`](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CHANGELOG.md)
file updated for non-trivial changes
* For trivial changes, include `[chore]` in the PR title to skip the
changelog check
* [ ] [Spec compliance
matrix](https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix/template.yaml)
updated if necessary
* [ ] [Declarative config data
model](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/data-model.md#overview)
is updated if SDK config surface is changed

---------

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Co-authored-by: Jack Berg <34418638+jack-berg@users.noreply.github.com>
dashpole pushed a commit to dashpole/opentelemetry-specification that referenced this pull request Jun 8, 2026
Fixes open-telemetry#4923

Stabilizes the transformation of OTLP Summaries to Prometheus summaries.

This PR superseeds open-telemetry#5090, since @johannaojeling is out on PTO for a
couple of weeks

For non-trivial changes, follow the [change proposal
process](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CONTRIBUTING.md#proposing-a-change).

* [X] Related issues open-telemetry#4923
* [ ] Related [OTEP(s)](https://github.com/open-telemetry/oteps) #
* [ ] Links to the prototypes (when adding or changing features)
* [X]
[`CHANGELOG.md`](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CHANGELOG.md)
file updated for non-trivial changes
* For trivial changes, include `[chore]` in the PR title to skip the
changelog check
* [ ] [Spec compliance
matrix](https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix/template.yaml)
updated if necessary
* [ ] [Declarative config data
model](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/data-model.md#overview)
is updated if SDK config surface is changed

---------

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
Co-authored-by: Jack Berg <34418638+jack-berg@users.noreply.github.com>
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.

[prometheus] Stabilize OTLP->Prometheus: Summaries

3 participants