Let's wait to see if tests pass on CI or we get port collisions. The direct parent diff is a draft, this one was supposed to be one too.

rebase

Moved modules/bitcoinsuite-chronik-client next to other modules in workspace. Moved chronik-proto = { path = "../../chronik/chronik-proto/"} to other local dependencies

@bot build-chronik build-chronik-plugins

This unbreaks the explorer deployment, however we should find a fix for this issue

These issues are likely port collisions

2024-12-13T20:49:51.570827Z [chronik/chronik-lib/src/bridge.rs:181] [<chronik unknown>] Starting Chronik Electrum interface bound to [(127.0.0.1:33252, Tcp)]
2024-12-13T20:49:51.574474Z [chronik/chronik-indexer/src/indexer.rs:1491] [<chronik unknown>] Chronik Electrum failed serving: Address already in use (os error 98)
2024-12-13T20:49:51.574503Z [/work/src/shutdown.cpp:22] [AbortNode] *** ERROR Chronik in ChronikElectrumServer::serve
2024-12-13T20:49:51.574521Z [/work/src/noui.cpp:42] [noui_ThreadSafeMessageBox] Error: Chronik Electrum failed serving: Address already in use (os error 98)

Updating lint

There seems to be some kind if intermittent issue introduced by this diff or it's parent. To be investigated.

Deleted newline in hash.rs

Import chronik-proto from /chronik

remove the small intermediate macros that only have one callsite. They just make the logic harder to follow.

c'est cassé

Updated everything in this diff - please see summary for more indepth

rebase

waiting for len(getavalanchepeerinfo) == QUORUM_NODE_COUNT should be enough here

Open link in new tab
Change order

In D17344#394125, @Fabien wrote:

Any reason for the selected ordering ?

tested and approved 💪

Any reason for the selected ordering ? Poker first then pay2stay then paybutton ?

Build Bitcoin ABC Diffs / Diff Testing (preview-e.cash) passed.
Preview is available at http://51.68.37.192:41084 for the next 60 minutes.

@bot preview-e.cash