-
Notifications
You must be signed in to change notification settings - Fork 246
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
Add node metrics with exporter #258
Comments
What sort of metrics do you have in mind? |
Geth metrics: https://geth.ethereum.org/docs/interface/metrics |
This is something that has been requested a few times now, and its definitely something we want to support. That said, it will be a medium term objective and not the short term priority. |
Sorry for the late reply, for some reasons I didn't see the previous response. What geth does is good enough, even better would be using a standard like opentelemetry for instrumentation and tracing. |
Thanks -- will have a look into the standards. |
Are you open to external contributions for this? I'm running pathfinder in production and I need this and #194. I can write a more thorough proposal before I start working on this so that we can agree on how it will work. |
We would really appreciate a contribution here -- our bandwidth is currently quite limited.
I think this is a great starting point! And additional note - we will be unavailable next week (16-20 May), so communication will be sparse then. |
Since #545 and #620 we've got prometheus metrics support as described in the readme file. @fracek are these sufficient for you? |
Sorry, I was about to just comment, not closing yet. |
Yep, looks good to me. Thank you! |
When running the node in production I'm interested in monitoring the overall node's health. It would be sweet to have the node collect several metrics and export them to InfluxDB and/or Prometheus. Go Ethereum offers this feature and I think it's a good reference.
The text was updated successfully, but these errors were encountered: