🤔 w0xlt reviewed a pull request: "doc: Add legacy wallet removal release notes"
(https://github.com/bitcoin/bitcoin/pull/33075#pullrequestreview-3068599342)
ACK https://github.com/bitcoin/bitcoin/pull/33075/commits

💬 achow101 commented on pull request "doc: Add legacy wallet removal release notes":
(https://github.com/bitcoin/bitcoin/pull/33075#issuecomment-3133503651)
ACK fa45ccc15dfc52e798da62548dc43d1bd7889c9a

🚀 achow101 merged a pull request: "doc: Add legacy wallet removal release notes"
(https://github.com/bitcoin/bitcoin/pull/33075)

💬 achow101 commented on pull request "wallet: Fix relative path backup during migration.":
(https://github.com/bitcoin/bitcoin/pull/32273#issuecomment-3133523088)
I'm going to go ahead and merge this now as @davidgumberg is out for the next couple weeks. Remaining comments can be addressed in a followup.

💬 achow101 commented on pull request "doc: add note for watch-only wallet migration":
(https://github.com/bitcoin/bitcoin/pull/32866#issuecomment-3133529680)
ACK 5888b4a2a5566c64141b78a0e7660a166ec99775

💬 maflcko commented on pull request "Migrate CI to hosted Cirrus Runners":
(https://github.com/bitcoin/bitcoin/pull/32989#issuecomment-3133536724)
The checkout action by default picks the "rebase"/merge with master, so this should happen even without you rebasing. In fact, it should happen even when not pushing at all, but just re-running the task. However, I just noticed this does not happen (https://github.com/bitcoin/bitcoin/actions/runs/16521447783/job/46969566419?pr=29641#step:2:90 picks a 4-day old merge). This breaks the use-case of being able to detect silent merge conflicts before a maintainer detects them (or misses them).

Not
...

🚀 achow101 merged a pull request: "wallet: Fix relative path backup during migration."
(https://github.com/bitcoin/bitcoin/pull/32273)

💬 achow101 commented on pull request "[POC] wallet: Add Support for BIP-353 DNS-Based Bitcoin Address via External Resolver":
(https://github.com/bitcoin/bitcoin/pull/33069#issuecomment-3133584656)
> How is it different than any other multi-process binary?

The multiprocess binaries are produced by this project, built using the same deterministic build system, using the same code as the monolithic binaries. There are already concrete ways that the binaries can be distributed with minimal changes to our own release process. Users who run the multiprocess binaries don't have to do any special configuration for them to work. This is different from requiring users to set a configuration opti
...

⚠️ jonatack opened an issue: "spurious failure in p2p_leak.py --v1transport"
(https://github.com/bitcoin/bitcoin/issues/33090)
### Is there an existing issue for this?

- [x] I have searched the existing issues

### Current behaviour

https://github.com/bitcoin/bitcoin/actions/runs/16601603414/job/46962514184?pr=31886

### Expected behaviour

This test should have passed.

### Steps to reproduce

See logs in

https://github.com/bitcoin/bitcoin/actions/runs/16601603414/job/46962514184?pr=31886

I cannot reproduce it locally.

### Relevant log output

_No response_

### How did you obtain Bitcoin Core

Compiled from sourc
...

💬 jonatack commented on pull request "cli: return local services in -netinfo":
(https://github.com/bitcoin/bitcoin/pull/31886#issuecomment-3133622663)
Yes, I opened an issue to report it: https://github.com/bitcoin/bitcoin/issues/33090

I don't see how to re-start that test.

💬 luke-jr commented on pull request "init: [gui] Avoid UB/crash in InitAndLoadChainstate":
(https://github.com/bitcoin/bitcoin/pull/32987#issuecomment-3133628881)
Alternative to the test option, Core could just add support for reindex=auto (#22072)

🚀 achow101 merged a pull request: "doc: add note for watch-only wallet migration"
(https://github.com/bitcoin/bitcoin/pull/32866)

💬 achow101 commented on pull request "wallet: Add `exportwatchonlywallet` RPC to export a watchonly version of a wallet":
(https://github.com/bitcoin/bitcoin/pull/32489#discussion_r2240683030)
The cache can only be generated if private keys are available, so we need this to write the cache to disk of the watchonly wallet.

💬 achow101 commented on pull request "wallet: Add `exportwatchonlywallet` RPC to export a watchonly version of a wallet":
(https://github.com/bitcoin/bitcoin/pull/32489#discussion_r2240684156)
I think export is a reasonable name since listing them is also an export.

💬 warp-records commented on issue "fuzz: Speed up mini_miner fuzz target?":
(https://github.com/bitcoin/bitcoin/issues/32870#issuecomment-3133776794)
> Looks like it is somewhat slow (c.f. https://cirrus-ci.com/task/6602192649453568?logs=ci#L5401), so it would be nice to make it faster.
>
> Possibly with txgraph, see [bitcoin-core/qa-assets#227 (comment)](https://github.com/bitcoin-core/qa-assets/pull/227#issuecomment-3032434420)

Can you close this now?

💬 achow101 commented on pull request "util: Abort on failing CHECK_NONFATAL in debug builds":
(https://github.com/bitcoin/bitcoin/pull/32588#issuecomment-3133809122)
ACK fa37153288ca420420636046ef6b8c4ba7e5a478

💬 maflcko commented on pull request "assumevalid: log every script validation state change":
(https://github.com/bitcoin/bitcoin/pull/32975#discussion_r2240783171)
Python does not have lifetimes in block scopes and `p2p1` will be visible outside of the block. Also, on a failure, the debug log snippet of this scope will be repeated, and will include irrelevant details twice.

📝 TheTyg opened a pull request: "circle paymaster"
(https://github.com/bitcoin/bitcoin/pull/33091)
implementation of circle pay

<!--
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:

* Any test impro
...

✅ hebasto closed a pull request: "circle paymaster"
(https://github.com/bitcoin/bitcoin/pull/33091)

💬 l0rinc commented on pull request "assumevalid: log every script validation state change":
(https://github.com/bitcoin/bitcoin/pull/32975#discussion_r2240810267)
Ok, sure, thanks for the context, narrowed the scope to send and sync only