💬 glozow commented on pull request "validate package transactions with their in-package ancestor sets":
(https://github.com/bitcoin/bitcoin/pull/26711#discussion_r1142230795)
Ah thanks!

💬 Sjors commented on issue "Memory leak in wallet_tests with BDB using address sanitizer":
(https://github.com/bitcoin/bitcoin/issues/27283#issuecomment-1476364655)
I went back to the #18727 merge commit 0f204dd3f21b997334a0e99954c939db154b64ca where this test was introduced. In order to test that old commit, I used `contrib/install_db4.sh` as it existed at the time, but otherwise didn't use depends (can't get that old version of boost to build). I then recreated a narrowed down test:

```cpp
auto chain = interfaces::MakeChain(m_node);
auto wallet = TestLoadWallet(*chain);
TestUnloadWallet(std::move(wallet));
wallet = TestLoadWallet(*c
...

💬 TheCharlatan commented on pull request "refactor, kernel: Decouple ArgsManager from blockstorage":
(https://github.com/bitcoin/bitcoin/pull/27125#discussion_r1142232528)
> I feel this though, I'll try out the first and second option you laid out.

Both are branching out a bit too much in my taste for the refactor we seek to achieve here. If the `const BlockManager` is a non-starter, I can always revert back to https://github.com/bitcoin/bitcoin/commit/f87c39399823e22c553b797cc66fa4063462a32b

💬 dergoegge commented on issue "Continuous benchmark tracking":
(https://github.com/bitcoin/bitcoin/issues/27284#issuecomment-1476374459)
Honestly I think https://codespeed.bitcoinperf.com/ is pretty close to what we want here. It does seem like that hasn't been running for a while? But getting that running again and adding some kind of notification system is probably all we need.

💬 jessebarton commented on pull request "doc: FreeBSD DataDirectoryGroupReadable Setting":
(https://github.com/bitcoin/bitcoin/pull/26741#issuecomment-1476374767)
@vasild

✅ jessebarton closed a pull request: "doc: FreeBSD DataDirectoryGroupReadable Setting"
(https://github.com/bitcoin/bitcoin/pull/26741)

💬 pinheadmz commented on pull request "Support JSON-RPC 2.0 when requested by client":
(https://github.com/bitcoin/bitcoin/pull/27101#issuecomment-1476400553)
This PR is ready for code review if any of you fine handsome concept-ACKers have the time ❤️

📝 jessebarton reopened a pull request: "doc: FreeBSD DataDirectoryGroupReadable Setting"
(https://github.com/bitcoin/bitcoin/pull/26741)
Updating tor.md doc to include mention of FreeBSD requiring the DataDirectoryGroupReadable be set to 1.
Default per the FreeBSD man page is 0.


DataDirectoryGroupReadable 0|1
If this option is set to 0, don't allow the filesystem group to
read the DataDirectory. If the option is set to 1, make the
DataDirectory readable by the default GID. (Default: 0)

💬 virtu commented on pull request "mempool: Add mempool tracepoints":
(https://github.com/bitcoin/bitcoin/pull/26531#issuecomment-1476426500)
- [As requested here](https://github.com/bitcoin/bitcoin/pull/26531#issuecomment-1472375220), the`mempool:removed` tracepoint now includes the time the removed transaction originally entered the mempool so mempool residency times can be calculated: [4684aa3](https://github.com/bitcoin/bitcoin/commit/4684aa3d1721a772862f3087b4154f7c07854051)
- Addressed [feedback on demo script memory usage](https://github.com/bitcoin/bitcoin/pull/26531#issuecomment-1472375220) by introducing explicit counters f
...

💬 MarcoFalke commented on pull request "ci: Cache more stuff in the ci images: msan, iwyu, pip, sdks":
(https://github.com/bitcoin/bitcoin/pull/27028#issuecomment-1476427547)
Would be good to make progress here. This will allow Cirrus CI to use docker images with a cache and thus avoid all 3rd party network errors like (https://github.com/bitcoin/bitcoin/pull/27277#issuecomment-1475912615) , unless they happen inside of Cirrus CI itself, or GitHub.

💬 LarryRuane commented on issue "Continuous benchmark tracking":
(https://github.com/bitcoin/bitcoin/issues/27284#issuecomment-1476436648)
Yes, this would be very valuable. I'd like to attempt to get this going; @dergoegge, would that be okay? I made a related [comment](https://github.com/bitcoin/bitcoin/pull/26957#issuecomment-1474874821) last week before I was aware of these websites (which are definitely better than what I suggested).

💬 pinheadmz commented on issue "Processing of new blocks slower than necessary due to overly selective peer selection":
(https://github.com/bitcoin/bitcoin/issues/21803#issuecomment-1476444242)
> slightly changing the code

Do you still have this patch available?

Is it the case that `> first cmpctblock received` is from a high-bandwidth CB peer but `> first node that announced the block (via headers).` is a low-bandwidth peer? And you're seeing bitcoin ignore the block message because it received the headers first and is requesting the block from the low-bandwidth peer?

💬 pinheadmz commented on issue "Export a watch wallet only (with descriptors and without private keys) for an air gap setup":
(https://github.com/bitcoin/bitcoin/issues/24829#issuecomment-1476453733)
I don't understand how an "exportwatchonly" command is any more efficient or straight forward than `listdescriptors` -> `importdescriptors` ?

💬 dergoegge commented on issue "Continuous benchmark tracking":
(https://github.com/bitcoin/bitcoin/issues/27284#issuecomment-1476458871)
@LarryRuane cool, please do!

💬 dongcarl commented on issue ""Unable to parse settings.json" at startup after disk was full":
(https://github.com/bitcoin/bitcoin/issues/21340#issuecomment-1476464194)
One of my boxes running `v24.0.1` just encountered a low storage situation, and `/var/lib/bitcoind/settings.json` was truncated to 0 size.

From my `journalctl`:
```
Mar 20 11:33:05 hostname bitcoind[935629]: Error: Failed loading settings file:
Mar 20 11:33:05 hostname bitcoind[935629]: - Unable to parse settings file /var/lib/bitcoind/settings.json
Mar 20 11:33:05 hostname systemd[1]: bitcoind.service: Control process exited, code=exited, status=1/FAILURE
```

> This seems bad. I woul
...

💬 pinheadmz commented on issue "Bitcoind repeatedly crashing at `UpdateTip` with no error message":
(https://github.com/bitcoin/bitcoin/issues/24483#issuecomment-1476467951)
@wbobeirne any updates on this issue or more clues to help debug?

💬 instagibbs commented on pull request "Update MANDATORY_SCRIPT_VERIFY_FLAGS":
(https://github.com/bitcoin/bitcoin/pull/26291#issuecomment-1476478682)
strong concept ACK on making it more clear on what flags are doing what(I find those names super unhelpful)

Completely unsure on increasing the scope of what we disconnect for. Are we seeing these messages in the wild?

💬 pinheadmz commented on issue "Error: specified data directory "\\IP.Ad.re.ss\release\Folder"does not exist":
(https://github.com/bitcoin/bitcoin/issues/25868#issuecomment-1476482356)
I wonder if this is related to https://github.com/bitcoin/bitcoin/issues/27246 and fixed by https://github.com/bitcoin/bitcoin/pull/27273 ?

👍 ryanofsky approved a pull request: "wallet: Replace use of purpose strings with an enum"
(https://github.com/bitcoin/bitcoin/pull/27217)
Code review ACK 0d18142f595b43070bd07b608d43c9ed9b37cdcc. Suggested a few comment/whitespace tweaks, but this is also fine as-is.

💬 ryanofsky commented on pull request "wallet: Replace use of purpose strings with an enum":
(https://github.com/bitcoin/bitcoin/pull/27217#discussion_r1142325608)
In commit "wallet: Replace use of purpose strings with an enum" (ea3cc035ddf8f15fd18407231c3ff9edde8891c1)

s/address/addresses/

💬 ryanofsky commented on pull request "wallet: Replace use of purpose strings with an enum":
(https://github.com/bitcoin/bitcoin/pull/27217#discussion_r1142308217)
In commit "wallet: Add wallet/types.h for simple public enum and struct types" (f00f5dad29d81b9700dd84ec08a6e72f915988a0)

Looks like there might be an extra 3 lines of space instead of 2 (but fine to keep if that's intentional)