This patch allows stake contenders to be polled (if -avalanchestakingpreconsensus is set). They are not yet reconciled, so no avalanche voting is taking place. Manually set winners polled for will respond as accepted.
Depends on D17144
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Owners Package (Owns No Changed Paths) Restricted Project - Commits
- rABC3b44e6192850: [avalanche] Add stake contender polling
ninja check-functional
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- sw-avaresp
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 31133 Build 61767: Build Diff lint-circular-dependencies · build-clang · build-without-wallet · build-debug · build-clang-tidy · build-diff Build 61766: arc lint + arc unit
Event Timeline
| test/functional/abc_p2p_avalanche_contender_voting.py | ||
|---|---|---|
| 96 ↗ | (On Diff #50809) | So my understanding of this (imo very confusing) loop is that the node will respond invalid for the block height the proof was made valid ? This seems like a glitch and not the expected behavior. Please add some comments in that loop also, e.g. the indexes are using the quorum node but are about block height which makes no sense unless you understand that the chain is setup clean so no block initially and each quorum node caused a block to be mined to generate its proof. |
| 129 ↗ | (On Diff #50809) | You should set a mock time to make it explicit the node never had a chance to compute the winner under normal conditions (because all the proofs are too recent to be considered). |
Comment Actions
- set mock time and give some comments why we do this
- clarify a bunch of comments in the goofy loop
| test/functional/abc_p2p_avalanche_contender_voting.py | ||
|---|---|---|
| 103 | I still think this is incorrect. The node has no way to determine if the contender should be picked or not, it should respond something that doesn't impact the vote outcome and not reject it. | |