Differential D6559

[avalanche] Process AvalancheResponse and act based on the result.
ClosedPublic
Actions

Authored by deadalnix on Jun 12 2020, 20:56.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCe7b9fb98c259: [avalanche] Process AvalancheResponse and act based on the result.

Summary

So far, we just processed repsonse and then diteched them away.

Because we do not poll for block in our chain, the park codepath is not tested. This will be the subject of the next patch.

Depends on D6548

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avaresponseprocess

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 11457
Build 21979: Default Diff Build & Tests	Link to the build
Build 21978: Build without Wallet	Link to the build
Build 21977: Build with clang-10	Link to the build
Build 21976: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jun 12 2020, 20:56

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 12 2020, 20:56

deadalnix requested review of this revision.Jun 12 2020, 20:56

deadalnix edited the summary of this revision. (Show Details)Jun 12 2020, 21:02

deadalnix added a parent revision: D6548: Update avalanche integration test to use a quorum.

rework the test a bit

Harbormaster completed remote builds in B11457: Diff 21341.Jun 12 2020, 21:09

Harbormaster completed remote builds in B11458: Diff 21342.Jun 12 2020, 21:17

Fabien accepted this revision.Jun 15 2020, 08:16

This revision is now accepted and ready to land.Jun 15 2020, 08:16

Fabien mentioned this in D6562: [avalanche] Poll all candidate tips.Jun 15 2020, 08:17

Closed by commit rABCe7b9fb98c259: [avalanche] Process AvalancheResponse and act based on the result. (authored by deadalnix). · Explain WhyJun 15 2020, 13:52

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCe7b9fb98c259: [avalanche] Process AvalancheResponse and act based on the result..

Revision Contents
Changeset List

Path

Size

src/

net_processing.cpp

28 lines

test/

functional/

abc_p2p_avalanche.py

39 lines

Diff 21341

View Options

src/net_processing.cpp