vechain / thor

A general purpose blockchain highly compatible with Ethereum's ecosystem
GNU Lesser General Public License v3.0
796 stars 247 forks source link

chore: add additional metrics (websockets / node stats) #778

Closed darrenvechain closed 2 months ago

darrenvechain commented 3 months ago

Description

Adding metrics around subscriptions and node stats

Please delete options that are not relevant.

How Has This Been Tested?

TODO

Checklist:

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 62.06897% with 11 lines in your changes missing coverage. Please review.

Project coverage is 62.13%. Comparing base (6c62498) to head (e003a86).

Files Patch % Lines
metrics/prometheus.go 20.00% 4 Missing :warning:
cmd/thor/node/node.go 0.00% 3 Missing :warning:
cmd/thor/node/packer_loop.go 0.00% 2 Missing :warning:
metrics/noop.go 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #778 +/- ## ========================================== - Coverage 62.14% 62.13% -0.01% ========================================== Files 203 203 Lines 18688 18702 +14 ========================================== + Hits 11613 11620 +7 - Misses 5975 5980 +5 - Partials 1100 1102 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

darrenvechain commented 2 months ago

@otherview screenshots of the affected metrics:

image image