-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
histograms: Implement final OM text format #11265
Comments
For reference: https://docs.google.com/document/d/1w6GhLmDYKkkNLsyPWkC3TGhW3Rs1G73j7E_nG7bhlw0/edit# enumerates ideas how an OM text format could look like |
Are there any decisions made on the OM text format? I would interested in implementing the text format for the OpenTelemetry Prometheus exporter in |
No decisions made yet. The next step would be to create a prototype in any of the instrumentation libraries based on the brainstorming doc linked above. https://github.com/prometheus/client_python comes to mind, and @csmarchbanks seemed interesting in exploring the topic. Not sure how soon he'll have time (or if he is ready to receive PRs from others). |
Unfortunately, Prometheus doesn't propagate native histograms in text format - see prometheus/prometheus#11265, only in Protobuff format, and this is not easily scrapeable.
Hello from the bug scrub. Design doc: prometheus/proposals#32 Being worked on by @csmarchbanks @vesari |
Once OM has decided how the text format representation of Native Histograms should look like (likely in OM 2.0, cf. prometheus/OpenMetrics#237 for tracking), we need to implement it here in prometheus/prometheus.
This is needed to declare Native Histograms a stable feature.
See also #11172, which will inform how to design the text format.
The text was updated successfully, but these errors were encountered: