Differential D17284

[avalanche] Fix a race in abc_p2p_avalanche_contender_voting
ClosedPublic
Actions

Authored by Fabien on Tue, Dec 3, 11:34.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCe0fa65ae9f86: [avalanche] Fix a race in abc_p2p_avalanche_contender_voting

Summary

Looking at the log while checking the isfinalrpc is racy and not redundant.

Test Plan

./test/functional/test_runner.py abc_p2p_avalanche_contender_voting.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix_stake_contender_vote_test

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 31500
Build 62497: Build Diff	build-without-wallet · build-debug · build-diff · build-clang · build-clang-tidy
Build 62496: arc lint + arc unit

Event Timeline

Fabien created this revision.Tue, Dec 3, 11:34

Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Dec 3, 11:34

Fabien requested review of this revision.Tue, Dec 3, 11:34

Harbormaster completed remote builds in B31500: Diff 51365.Tue, Dec 3, 11:49

PiRK accepted this revision.Tue, Dec 3, 13:24

This revision is now accepted and ready to land.Tue, Dec 3, 13:24

Closed by commit rABCe0fa65ae9f86: [avalanche] Fix a race in abc_p2p_avalanche_contender_voting (authored by Fabien). · Explain WhyTue, Dec 3, 13:29

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCe0fa65ae9f86: [avalanche] Fix a race in abc_p2p_avalanche_contender_voting.

Revision Contents
Changeset List

Path

Size

test/

functional/

abc_p2p_avalanche_contender_voting.py

3 lines

Diff 51365

View Options

test/functional/abc_p2p_avalanche_contender_voting.py

[avalanche] Fix a race in abc_p2p_avalanche_contender_votingClosedPublicActions