-
Notifications
You must be signed in to change notification settings - Fork 43
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
Instrument with OpenCensus #8
Comments
@fabxc you already started working on this. |
All parts currently in functionality scope of OC should be migrated to it. Anything further depends on gauge support being added. |
OpenCensus now supports a view type of LastValue which is like a gauge https://godoc.org/go.opencensus.io/stats/view#LastValue |
Thanks, Bill. We are working with OpenCensus to have a first-class gauge in Go soon, like it was added to Java https://github.com/census-instrumentation/opencensus-java/blob/master/api/src/main/java/io/opencensus/metrics/LongGauge.java |
This is a cleanup. We'll take care of this if necessary, but currently it's not a blocker, so I'm closing the issue. |
This is for monitoring the sidecar itself, not how we write the Prometheus data.
This is important so we don't rely on Prometheus to monitor the Prometheus sidecar (although that will be an option with OpenCensus).
https://opencensus.io/go/index.html
The text was updated successfully, but these errors were encountered: