celestiaorg / go-header

Go library with all the services needed to request, sync and store blockchain headers.
Apache License 2.0
19 stars 18 forks source link

feat(p2p): metrics for Subscriber #119

Closed Wondertan closed 11 months ago

Wondertan commented 11 months ago

Adds 4 metrics:

TODO - Graphana Dashboard

Wondertan commented 11 months ago

Btw, @nashqueue, this PR reports sizes for Accepted headers.

codecov-commenter commented 11 months ago

Codecov Report

Merging #119 (300cde6) into main (098c994) will decrease coverage by 1.30%. Report is 2 commits behind head on main. The diff coverage is 15.09%.

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
- Coverage   69.20%   67.91%   -1.30%     
==========================================
  Files          37       38       +1     
  Lines        2981     3079      +98     
==========================================
+ Hits         2063     2091      +28     
- Misses        771      841      +70     
  Partials      147      147              
Files Coverage Δ
p2p/subscription.go 57.14% <75.00%> (+2.97%) :arrow_up:
sync/metrics.go 15.38% <0.00%> (ø)
p2p/subscriber.go 42.68% <33.33%> (-5.85%) :arrow_down:
p2p/subscriber_metrics.go 7.50% <7.50%> (ø)

... and 3 files with indirect coverage changes