Differential D7624

[backport#15615] Add log output during initial header sync
ClosedPublic
Actions

Authored by majcosta on Sep 28 2020, 21:18.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC526bdd96cfd3: [backport#15615] Add log output during initial header sync

Summary

Add log output during initial header sync (Jonas Schnelli)

Pull request description:

The non debug log output is completely quiet during the header sync. I see two main reasons to add infos about the state of the initial header sync...
* users may think the node did fail to start sync
* it's a little complicate to check if your getting throttled during header sync (repeatedly calling `getchaintips` or similar)

Backport of Core PR15615

Test Plan

ninja check check-functional

run bitcoind with a fresh datadir and see the new header sync log output

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

majcosta created this revision.Sep 28 2020, 21:18

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 28 2020, 21:18

majcosta requested review of this revision.Sep 28 2020, 21:18

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Harbormaster completed remote builds in B12995: Diff 23922.Sep 28 2020, 21:30

deadalnix accepted this revision.Sep 28 2020, 21:41

This revision is now accepted and ready to land.Sep 28 2020, 21:41

Closed by commit rABC526bdd96cfd3: [backport#15615] Add log output during initial header sync (authored by Wladimir J. van der Laan <laanwj@protonmail.com>, committed by majcosta). · Explain WhySep 28 2020, 22:11

This revision was automatically updated to reflect the committed changes.

majcosta added a commit: rABC526bdd96cfd3: [backport#15615] Add log output during initial header sync.

Revision Contents
Changeset List

Path

Size

src/

13 lines

Diff 23928

src/validation.cpp

Loading...