Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only monotonic sum are counters (with _total) #2644

Merged
merged 2 commits into from
Jul 8, 2022

Conversation

gouthamve
Copy link
Member

Gauges shouldn't have a _total attached to them according to OpenMetrics.

Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
@gouthamve gouthamve requested review from a team July 5, 2022 08:23
@pirgeo
Copy link
Member

pirgeo commented Jul 5, 2022

EDIT: Sorry, I was too quick here. I see what you mean now. If a non-monotonic sum is converted to a Prometheus Gauge, then it should not have _total, I agree.

@arminru arminru added spec:metrics Related to the specification/metrics directory area:data-model For issues related to data model labels Jul 5, 2022
@gouthamve
Copy link
Member Author

Yes, a non-monotonic sum is coverted to a Gauge :)

@jmacd jmacd merged commit 7933549 into open-telemetry:main Jul 8, 2022
beeme1mr pushed a commit to beeme1mr/opentelemetry-specification that referenced this pull request Aug 31, 2022
Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>

Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>

Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:data-model For issues related to data model spec:metrics Related to the specification/metrics directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants