[Chronik] Add `Chronik`, connect it to `CValidationInterface`
ClosedPublic
Actions

Authored by tobias_ruck on Sep 22 2022, 16:40.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC31df2802ac38: [Chronik] Add `Chronik`, connect it to `CValidationInterface`

Summary

Add Chronik, which will contain all the data/handles required to run Chronik.
Make setup_bridge return a boxed Chronik.
Add ChronikValidationInterface, to capture all the tx/block events happening in bitcoind.
Add Start-/StopChronikValidationInterface to create a Chronik and start + register / stop + unregister ChronikValidationInterface.

Test Plan

ninja
./src/bitcoind -regtest -reindex -debug=chronik -> displays "Chronik: block connected" (for the genesis block)
In a new window ./src/bitcoin-cli -regtest generatetoaddress ecregtest:qqmuvc70ttys4lg8se5tt7rgk53q6cmthuamcuaexg -> displays

"Chronik: block connected" in bitcoind

./src/bitcoin-cli invalidateblock <hash> -> displays "Chronik: block disconnected" in bitcoind
Can also test adding/removing tx to/from mempool

A proper python-based HTTP test will be added in future diffs.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-validationinterface

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20130
Build 39947: Build Diff	build-chronik
Build 39946: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Sep 22 2022, 16:40

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 22 2022, 16:40

tobias_ruck requested review of this revision.Sep 22 2022, 16:40

Harbormaster completed remote builds in B20130: Diff 35113.Sep 22 2022, 17:05

Fabien accepted this revision.Sep 23 2022, 09:23

This revision is now accepted and ready to land.Sep 23 2022, 09:23

Rename ChronikIndexer to Chronik.

Then we can use ChronikIndexer for the stuff that only does the indexing, but e.g. none of the HTTP stuff, and Chronik then bundles
everything that’s Chronik

tobias_ruck retitled this revision from [Chronik] Add `ChronikIndexer`, connect it to `CValidationInterface` to [Chronik] Add `Chronik`, connect it to `CValidationInterface`.Sep 23 2022, 09:36

tobias_ruck edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B20151: Diff 35153.Sep 23 2022, 09:50

Fabien accepted this revision.Sep 23 2022, 10:05

Closed by commit rABC31df2802ac38: [Chronik] Add `Chronik`, connect it to `CValidationInterface` (authored by tobias_ruck). · Explain WhySep 23 2022, 10:34

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC31df2802ac38: [Chronik] Add `Chronik`, connect it to `CValidationInterface`.