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

"_total" suffix missing from counter metrics with older Prometheus clients #53

Open
eero-t opened this issue Jun 12, 2023 · 1 comment

Comments

@eero-t
Copy link

eero-t commented Jun 12, 2023

When using python3-prometheus-client in OpenSuse 15.4 (that corresponds to SLES 15.4), XPUM counter metrics are missing _total suffix required by the OpenMetrics spec: https://github.com/OpenObservability/OpenMetrics/blob/main/specification/OpenMetrics.md

XPUM should not rely on (newer) prometheus-client adding the required suffix, but directly use correct metric names.

PS. While OpenSuse prometheus client is very old v0.0.20 (from 2017), _total suffix enforcement is more recent feature. I.e. this bug will manifest also with never client versions than that.

@eero-t
Copy link
Author

eero-t commented Jun 12, 2023

I.e. this bug will manifest also with never client versions than that.

Reading Prometheus Python client release history, "_total" suffix enforcement seems to have come already in 0.4.0 (released in 2018).

And OpenSuse 15.4 has also v0.7.1 (released in 2019) of that package, named as python3-prometheus_client.

Using that package (instead python3-prometheus-client), adds the required _total suffixes, so this bug can be considered very low priority.

@eero-t eero-t changed the title "_counter" suffix missing from counter metrics "_total" suffix missing from counter metrics with older Prometheus clients Jun 12, 2023
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

No branches or pull requests

1 participant