Differential D11782

[avalanche] Add the dangling proofs count to the getavalancheinfo RPC
ClosedPublic
Actions

Authored by Fabien on Jul 21 2022, 13:01.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

rABCd7757fe625f7: [avalanche] Add the dangling proofs count to the getavalancheinfo RPC

Summary

Returns the count of dangling proofs (i.e. proofs with no node connected) as well as their associated stake amount.

Test Plan

./test/functional/test_runner.py abc_rpc_getavalancheinfo.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_dangling_peers_getavalancheinfo

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19687
Build 39093: Build Diff	build-debug · build-without-wallet · build-diff · lint-circular-dependencies · build-clang · build-clang-tidy
Build 39092: arc lint + arc unit

Event Timeline

Fabien created this revision.Jul 21 2022, 13:01

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 21 2022, 13:01

Fabien requested review of this revision.Jul 21 2022, 13:01

Unknown Object (User) added a subscriber: Unknown Object (User).Jul 21 2022, 13:11

This comment was removed by Fabien.

Harbormaster completed remote builds in B19687: Diff 34452.Jul 21 2022, 13:14

I'm not sure the dangling proof count is helpful, but reducing mental math for the dangling stake could be.

This revision is now accepted and ready to land.Jul 21 2022, 18:02

Closed by commit rABCd7757fe625f7: [avalanche] Add the dangling proofs count to the getavalancheinfo RPC (authored by Fabien). · Explain WhyJul 21 2022, 19:41

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCd7757fe625f7: [avalanche] Add the dangling proofs count to the getavalancheinfo RPC.

Revision Contents
Changeset List

Path

Size

src/

rpc/

avalanche.cpp

6 lines

test/

functional/

abc_rpc_getavalancheinfo.py

14 lines

Diff 34452

View Options

src/rpc/avalanche.cpp