[avalanche] Clear the remote proofs upon node removal
ClosedPublic
Actions

Authored by Fabien on Oct 20 2023, 20:21.

Details

Reviewers

PiRK
bytesofman

Group Reviewers

Restricted Project

Commits

rABC49ad3d3990d9: [avalanche] Clear the remote proofs upon node removal

Summary

If a peer disconnects, clear its remote proofs set.

Depends on D14659.

Test Plan

ninja check-avalanche-peermanager_tests

Diff Detail

Repository

rABC Bitcoin ABC

Branch

remove_remote_proofs

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25386
Build 50353: Build Diff	lint-circular-dependencies · build-without-wallet · build-clang-tidy · build-diff · build-debug · build-clang
Build 50352: arc lint + arc unit

Event Timeline

Fabien created this revision.Oct 20 2023, 20:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 20 2023, 20:21

Fabien requested review of this revision.Oct 20 2023, 20:21

Fabien added a child revision: D14661: [avalanche] Limit the remote proofs per peer.

Harbormaster completed remote builds in B25386: Diff 42721.Oct 20 2023, 20:37

PiRK accepted this revision.Oct 21 2023, 15:53

This revision is now accepted and ready to land.Oct 21 2023, 15:53

bytesofman accepted this revision.Oct 23 2023, 18:08

Closed by commit rABC49ad3d3990d9: [avalanche] Clear the remote proofs upon node removal (authored by Fabien). · Explain WhyOct 25 2023, 10:18

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC49ad3d3990d9: [avalanche] Clear the remote proofs upon node removal.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

peermanager.cpp

5 lines

test/

peermanager_tests.cpp

27 lines

Diff 42721

View Options

src/avalanche/peermanager.cpp