Differential D18070

[chronik] Use the chronik category for logging chronik broadcast errors
ClosedPublic
Actions

Authored by Fabien on Fri, May 9, 10:01.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC7a25796cc35a: [chronik] Use the chronik category for logging chronik broadcast errors

Summary

If a block notification fails to be sent by chronik, the log message should use the chronik log category.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik_subs_nits

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33255
Build 65992: Build Diff	chronik-client-integration-tests · ecash-lib-integration-tests · build-chronik-plugins · build-chronik · ecash-agora-integration-tests
Build 65991: arc lint + arc unit

Event Timeline

Fabien created this revision.Fri, May 9, 10:01

Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, May 9, 10:01

Fabien requested review of this revision.Fri, May 9, 10:01

Harbormaster completed remote builds in B33255: Diff 53917.Fri, May 9, 10:24

PiRK accepted this revision.Fri, May 9, 11:25

This revision is now accepted and ready to land.Fri, May 9, 11:25

Closed by commit rABC7a25796cc35a: [chronik] Use the chronik category for logging chronik broadcast errors (authored by Fabien). · Explain WhyFri, May 9, 12:18

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC7a25796cc35a: [chronik] Use the chronik category for logging chronik broadcast errors.

Revision Contents
Changeset List

Path

Size

chronik/

chronik-indexer/

src/

subs.rs

4 lines

Diff 53917

View Options

chronik/chronik-indexer/src/subs.rs

[chronik] Use the chronik category for logging chronik broadcast errorsClosedPublicActions