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

Extend metrics in the code base #684

Closed otherview closed 5 months ago

otherview commented 6 months ago

Description

Adds metrics to other parts of the code. Based on https://github.com/vechain/protocol-board-repo/issues/119

Type of change

How Has This Been Tested?

No changes to current running code.

Checklist:

codecov-commenter commented 6 months ago

Codecov Report

Attention: Patch coverage is 72.52747% with 75 lines in your changes are missing coverage. Please review.

Project coverage is 57.55%. Comparing base (e42227b) to head (6dd8cdf). Report is 17 commits behind head on pedro/add_metrics.

:exclamation: Current head 6dd8cdf differs from pull request most recent head 9735a3d. Consider uploading reports for the commit 9735a3d to get more accurate results

Files Patch % Lines
telemetry/prometheus.go 83.43% 18 Missing and 9 partials :warning:
cmd/thor/node/node.go 0.00% 18 Missing :warning:
cmd/thor/node/packer_loop.go 0.00% 18 Missing :warning:
api/events/events.go 0.00% 3 Missing :warning:
api/transfers/transfers.go 0.00% 3 Missing :warning:
p2psrv/nodes.go 0.00% 2 Missing :warning:
p2psrv/server.go 0.00% 2 Missing :warning:
telemetry/telemetry.go 86.66% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## pedro/add_metrics #684 +/- ## ===================================================== + Coverage 57.27% 57.55% +0.27% ===================================================== Files 156 159 +3 Lines 17218 17460 +242 ===================================================== + Hits 9862 10049 +187 - Misses 6392 6438 +46 - Partials 964 973 +9 ```

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