-
Notifications
You must be signed in to change notification settings - Fork 122
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
Prometheus metrics #692
Comments
See #667 |
That would solve it for me. Any estimation on when this can be merged? |
Whenever I have time to review it. In the meantime you can run that branch and report back on how well it's working, which would be helpful with the review process. |
+1 |
My feedback on #667 (which is now stale) was that I would prefer to use https://autometrics.dev/ instead |
@tony-iqlusion any news on that? I'd say having metrics for the app that's responsible for signing blocks and that's potentially securing millions of dollars (and may cause huge losses if something is not right), this seems like a must. |
I'm using Prometheus a lot of metrics and alerting and it'd be nice if tmkms would expose some of its own metrics in a Prometheus compatible format so I'd have an overview on how good tmkms is performing and can add another alerts set for it. At least the following would be nice to have:
Do you think it's possible or desireable?
The text was updated successfully, but these errors were encountered: