Differential D12273

p2p: log addrman consistency checks
ClosedPublic
Actions

Authored by Fabien on Oct 17 2022, 11:53.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCaca8b55dd2a9: p2p: log addrman consistency checks

Summary

[...] log addrman consistency checks in the BCLOG::ADDRMAN category when they are enabled with the -checkaddrman=<n> configuration option for values of n greater than 0.

Backport of core#22696.

Depends on D12272.

Test Plan

ninja check

bitcoind -checkaddrman=1 -debug=addrman

Check the logs are printed.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Oct 17 2022, 11:53

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 17 2022, 11:53

Fabien requested review of this revision.Oct 17 2022, 11:53

teamcity edited the summary of this revision. (Show Details)Oct 17 2022, 11:53

Harbormaster completed remote builds in B20546: Diff 35745.Oct 17 2022, 12:09

PiRK accepted this revision.Oct 17 2022, 13:14

This revision is now accepted and ready to land.Oct 17 2022, 13:14

Closed by commit rABCaca8b55dd2a9: p2p: log addrman consistency checks (authored by Fabien). · Explain WhyOct 17 2022, 14:34

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCaca8b55dd2a9: p2p: log addrman consistency checks.

Revision Contents
Changeset List

Path

Size

src/

addrman.cpp

5 lines

Diff 35752

View Options

p2p: log addrman consistency checksClosedPublicActions