Page MenuHomePhabricator
Differential D11369

[avalanche] Make quorum detection consider local proofs.
AbandonedPublic
Actions

Authored by tyler-smith on Apr 17 2022, 22:41.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

The local node is part of the quorum and their own stake must be
considered when deciding if we have enough stake to make a quorum.

Test Plan
ninja check-avalanche
./test/functional/test_runner.py abc_p2p_avalanche_quorum.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
TS_fix_quorum_connected_tracking
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18841
Build 37472: Build Diffbuild-diff · lint-circular-dependencies · build-debug · build-without-wallet · build-clang-tidy · build-clang
Build 37471: arc lint + arc unit

Event Timeline

tyler-smith created this revision.Apr 17 2022, 22:41
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 17 2022, 22:41
tyler-smith requested review of this revision.Apr 17 2022, 22:41
tyler-smith planned changes to this revision.Apr 17 2022, 22:52

This is incorrect based on a misunderstanding with the peermanager and local proof.

Harbormaster completed remote builds in B18841: Diff 33245.Apr 17 2022, 22:56
tyler-smith abandoned this revision.Apr 19 2022, 05:31

Superseded by D11367.

Revision Contents
Changeset List

PathSize
src/
avalanche/
8 lines
test/
21 lines
test/
functional/
42 lines

Diff 33245

View Options

src/avalanche/processor.cpp

Loading...
View Options

src/avalanche/test/processor_tests.cpp

Loading...
View Options

test/functional/abc_p2p_avalanche_quorum.py

Loading...