iqlusioninc / tmkms

Tendermint KMS: Key Management System for Tendermint Validators
https://tendermint.com/
Apache License 2.0
326 stars 117 forks source link

Prometheus metrics #692

Open freak12techno opened 1 year ago

freak12techno commented 1 year ago

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?

tony-iqlusion commented 1 year ago

See #667

freak12techno commented 1 year ago

That would solve it for me. Any estimation on when this can be merged?

tony-iqlusion commented 1 year ago

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.

zheli commented 11 months ago

+1

tony-iqlusion commented 9 months ago

My feedback on #667 (which is now stale) was that I would prefer to use https://autometrics.dev/ instead

freak12techno commented 4 days ago

@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.