Skip to content

p2p: metrics#6278

Merged
alexanderbez merged 5 commits intomasterfrom
bez/p2p-metrics
Mar 26, 2021
Merged

p2p: metrics#6278
alexanderbez merged 5 commits intomasterfrom
bez/p2p-metrics

Conversation

@alexanderbez
Copy link
Contributor

Address remaining metrics from the legacy p2p set of metrics.

@alexanderbez alexanderbez added the C:p2p Component: P2P pkg label Mar 25, 2021
@codecov
Copy link

codecov bot commented Mar 25, 2021

Codecov Report

Merging #6278 (6ba5e59) into master (a554005) will decrease coverage by 0.18%.
The diff coverage is 40.00%.

❗ Current head 6ba5e59 differs from pull request most recent head 26ebee0. Consider uploading reports for the commit 26ebee0 to get more accurate results

@@            Coverage Diff             @@
##           master    #6278      +/-   ##
==========================================
- Coverage   60.87%   60.69%   -0.19%     
==========================================
  Files         281      281              
  Lines       26614    26611       -3     
==========================================
- Hits        16201    16151      -50     
- Misses       8733     8771      +38     
- Partials     1680     1689       +9     
Impacted Files Coverage Δ
p2p/metrics.go 98.50% <ø> (-0.15%) ⬇️
p2p/pqueue.go 26.12% <0.00%> (-0.48%) ⬇️
p2p/wdrr_queue.go 100.00% <100.00%> (ø)
privval/signer_listener_endpoint.go 80.00% <0.00%> (-7.06%) ⬇️
privval/signer_endpoint.go 75.75% <0.00%> (-6.07%) ⬇️
privval/signer_server.go 89.47% <0.00%> (-5.27%) ⬇️
consensus/reactor.go 67.39% <0.00%> (-4.86%) ⬇️
privval/socket_listeners.go 78.72% <0.00%> (-4.26%) ⬇️
privval/secret_connection.go 72.68% <0.00%> (-3.61%) ⬇️
consensus/peer_state.go 78.94% <0.00%> (-3.08%) ⬇️
... and 13 more

Name: "peer_pending_send_bytes",
Help: "Number of pending bytes to be sent to a given peer.",
}, append(labels, "peer_id")).With(labelsAndValues...),
NumTxs: prometheus.NewGaugeFrom(stdprometheus.GaugeOpts{
Copy link
Contributor Author

@alexanderbez alexanderbez Mar 25, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note to reviewer: this metric was never used

@alexanderbez alexanderbez marked this pull request as ready for review March 25, 2021 22:03
@alexanderbez alexanderbez self-assigned this Mar 25, 2021
Copy link
Contributor

@tac0turtle tac0turtle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexanderbez alexanderbez added the S:automerge Automatically merge PR when requirements pass label Mar 26, 2021
@alexanderbez alexanderbez merged commit b5e4147 into master Mar 26, 2021
@alexanderbez alexanderbez deleted the bez/p2p-metrics branch March 26, 2021 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:p2p Component: P2P pkg S:automerge Automatically merge PR when requirements pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants