-
Notifications
You must be signed in to change notification settings - Fork 36
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
SDK metrics: Prometheus exporter #751
Comments
I would take it if no one minds. |
Does this cover pushing to a push gateway as well (e.g. for serverless)? Or is that separate |
From my point of view, this issue covers exactly prometheus exporter part from OpenTelemetry spec. And according to spec, prometheus exporter is a pull metric exporter which responds to HTTP requests. That is, this issue doesn't imply the push model. |
Ah yes, it has a Maybe I can add a contrib module after the basic prometheus is in? We have otel collectors at $work but only configured for traces and not metrics, we do have a push gateway though |
I think it makes sense to create a separate issue where this proposal can be discussed :) |
I would be careful keeping non-spec functionality in the main repository. Eventually, the maintenance burden could catch us up. If it's really needed, you can implement it here: https://github.com/typelevel/otel4s-experimental. |
Prometheus exporter works as an HTTP server that exposes metrics in Prometheus-compatible format.
A prototype: iRevive#5.
A few things to consider:
otel4s-sdk-exporter-prometheus
?Examples
Auto-configuration
Env configuration:
Code:
SDK will launch a dedicated HTTP server at
localhost:9464
.Manual configuration
Instead of launching a dedicated HTTP server, a user can merge exporter routes with the app routes.
The text was updated successfully, but these errors were encountered: