Differential D11800

[avalanche] Verify that polling has started on an item before verifying that it has stopped
ClosedPublic
Actions

Authored by sdulfari on Jul 25 2022, 23:54.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCc3de9a21195e: [avalanche] Verify that polling has started on an item before verifying that it…

Summary

If the node registers the proof too slowly, it's possible for the stale check to complete
all 5 retries before polling has even started. This is rarity, but may be one of the root
causes of flakiness that we're seeing on CI.

Test Plan

./test/functional/test_runner.py abc_p2p_avalanche_proof_voting

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

sdulfari created this revision.Jul 25 2022, 23:54

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 25 2022, 23:54

sdulfari requested review of this revision.Jul 25 2022, 23:54

Harbormaster completed remote builds in B19720: Diff 34511.Jul 26 2022, 00:10

Fabien requested changes to this revision.Jul 26 2022, 09:24

Fabien added a subscriber: Fabien.

Fabien added inline comments.

test/functional/abc_p2p_avalanche_proof_voting.py
252 ↗	(On Diff #34511)	Please apply the same fix here as the same issue might occur

This revision now requires changes to proceed.Jul 26 2022, 09:24

sdulfari requested review of this revision.Jul 26 2022, 14:41

sdulfari added inline comments.

test/functional/abc_p2p_avalanche_proof_voting.py
252 ↗	(On Diff #34511)	This doesn't have the same issue because the test already waits until the proof is accepted before getting here.

Fabien requested changes to this revision.Jul 26 2022, 14:56

Fabien added inline comments.

test/functional/abc_p2p_avalanche_proof_voting.py
252 ↗	(On Diff #34511)	You're right, and also via the above send_and_check_for_polling which can be leverage for the other case (see suggested diff)
465–470 ↗	(On Diff #34511)