-
Notifications
You must be signed in to change notification settings - Fork 933
Expose Prometheus metrics #355
Comments
@guidoiaquinti are you suggesting to add promotheus metrics? Otherwise how should this be done? |
My suggestion is to add an HTTP route (e.g. Here's a list of some third-party software natively exposing metrics in the Prometheus format. |
@guidoiaquinti thanks :) I meant to ask whether you intend to implements us or want us to. I'm not familiar with promotheus and this isn't on my immediate roadmap; given that I'll need to invest learning promotheus/metrics, I'm unlikely to do this in the near future. |
Sorry for the misunderstanding @shlomi-noach 😄 I've opened this issue only to keep track of the enhancement. I don't think I'll have time in the near future to implement it and it's currently not a high-pri for us. |
Cool. Thank you! |
PR for review #1414 |
It would be good to integrate the Prometheus client library to expose Orchestrator metrics like we already do for graphite.
I know that we expose the same metrics in expvar-like format via
/debug/metrics
but that's not exactly Prometheus compatible (see related discussion in thegh-ost
repo).I'm opening this issue to keep track of this enhancement.
Thank you!
The text was updated successfully, but these errors were encountered: