< yanmaani> How do I disable tsan when compiling bitcoin?
< bitcoin-git> [bitcoin] Faber-Yu opened pull request #22560: Create just a try (master...patch-1) https://github.com/bitcoin/bitcoin/pull/22560
< yanmaani> It gives me a compile time error (something to do with linking), but I don't want it and haven't enabled it
< bitcoin-git> [bitcoin] fanquake closed pull request #22560: Create just a try (master...patch-1) https://github.com/bitcoin/bitcoin/pull/22560
< sipa> yanmaani: you must be doing something wrong; it is off by default
< prayank> luke-jr: I have few questions about this PR https://github.com/bitcoin/bitcoin/pull/11803
< prayank> 1. hdseedid added for reserve=1 key type the only change this makes?
< prayank> 2. What other information can be considered metadata and added in dump?
< prayank> 3. Any functional test required for this PR?
< bitcoin-git> [bitcoin] cuongvng opened pull request #22561: Fix typo in comment (master...cuongvng-patch-1) https://github.com/bitcoin/bitcoin/pull/22561
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/1488f55fa57a...7075a52b67dc
< bitcoin-git> bitcoin/master 2565478 Russell Yanofsky: wallet test refactor: add CreateSyncedWallet function
< bitcoin-git> bitcoin/master fe6dc76 Russell Yanofsky: wallet test: Add test for subtract fee from recipient behavior
< bitcoin-git> bitcoin/master 7075a52 MarcoFalke: Merge bitcoin/bitcoin#22155: wallet test: Add test for subtract fee from r...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #22155: wallet test: Add test for subtract fee from recipient behavior (master...pr/subfee) https://github.com/bitcoin/bitcoin/pull/22155
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7075a52b67dc...f372623807c5
< bitcoin-git> bitcoin/master 8858e88 Sebastian Falbesoner: p2p: refactor: tidy up `PeerManagerImpl::Misbehaving(...)`
< bitcoin-git> bitcoin/master f372623 MarcoFalke: Merge bitcoin/bitcoin#22495: p2p: refactor: tidy up `PeerManagerImpl::Misb...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #22495: p2p: refactor: tidy up `PeerManagerImpl::Misbehaving(...)` (master...202107-net-tidy_up_misbehaving) https://github.com/bitcoin/bitcoin/pull/22495
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f372623807c5...979f410e69a0
< bitcoin-git> bitcoin/master 4148c52 Cuong V. Nguyen: Fix typo in comment
< bitcoin-git> bitcoin/master 979f410 MarcoFalke: Merge bitcoin/bitcoin#22561: Fix typo in comment
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #22561: Fix typo in comment (master...cuongvng-patch-1) https://github.com/bitcoin/bitcoin/pull/22561
< bitcoin-git> [bitcoin] Fuzzbawls opened pull request #22562: [Refactor] Apply clang-format to netaddress.h (master...2021_btc_netaddress-clang-format) https://github.com/bitcoin/bitcoin/pull/22562
< b10c> I've build an experimental metrics exporter based on the eBPF and USDT tracepoints added in #22006 - https://github.com/0xB10C/bitcoind-observer
<@gribble> https://github.com/bitcoin/bitcoin/issues/22006 | tracing: first tracepoints and documentation on User-Space, Statically Defined Tracing (USDT) by 0xB10C · Pull Request #22006 · bitcoin/bitcoin · GitHub
< b10c> currently reporting P2P message count and traffic by message type and block connection timings (most useful during IBD): https://bitcoind.observer
< b10c> MarkoFalke jamesob: I think a combination of the block connection timings and bitcoinperf.com (in some way) would be a good fit for per PR/branch perf reporting
< b10c> MarcoFalke not Marko ^
< laanwj> b10c: neat!
< laanwj> happy to see the metrics stuff being adapted even before it's merged :)
< vasild> CNetAddr::GetGroup() and the places where it is called are build around the assumption that it is easier to obtain addresses with common prefix (e.g. belonging to one IP subnet). However this assumption is not true for Tor, I2P and CJDNS addresses. Is there any work or plan to resolve this?
< vasild> E.g. the code would avoid connecting to aabcd.onion if it already has connection to aaxyz.onion, thinking that the common prefix makes it more likely they belong to the same actor.
< vasild> Actually, in Tor it is more difficult to obtain addresses with common prefix, not easier! :) (one has to brute force)
< bitcoin-git> [bitcoin] vasild opened pull request #22563: addrman: treat Tor/I2P/CJDNS as a single group (master...addrman_per_group_bucketing) https://github.com/bitcoin/bitcoin/pull/22563
< prayank> vasild: Interesting PR. Change looks quite obvious for Tor and i2p. Not sure about CJDNS. Although there must be some reason it wasn't done until now, would like to read comments from others who understand p2p and networking better.
< prayank> Also not sure if it breaks something else or changes functionality
< vasild> CJDNS addresses are also "random" ones
< vasild> it is a 16 bytes derived from the owner's public key, represented as an IPv6 address
< vasild> with the twist that generating a new one is done by brute-forcing until it begins with 0xFD so that it falls into some reserved/unused IPv6 range (IIRC)
< bitcoin-git> [bitcoin] laanwj pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/979f410e69a0...61cefde7a749
< bitcoin-git> bitcoin/master 84ace9a 0xb10c: doc: Add initial USDT documentation
< bitcoin-git> bitcoin/master 469b71a 0xb10c: doc: document systemtap dependency
< bitcoin-git> bitcoin/master 4224dec 0xb10c: tracing: Tracepoints for in- and outbound P2P msgs
< jamesob> b10c: yeah agreed, will look forward to checking out your bpf stuff in more detail
< bitcoin-git> [bitcoin] dongcarl opened pull request #22564: [WIP] refactor: Move mutable globals cleared in `::UnloadBlockIndex` to `BlockManager` (master...2021-07-kirby-inhale-global-muts) https://github.com/bitcoin/bitcoin/pull/22564
< bitcoin-git> [bitcoin] NikhilBartwal opened pull request #22565: [script] Update signet's getcoins.py for handling custom signet network (master...update_signet_getcoins) https://github.com/bitcoin/bitcoin/pull/22565
< hebasto> user Daoudia removed German (de) translation from Transifex.com 21 hours ago
< hebasto> another act of vandalism?
< fjahr> #22047 might be RFM with 3 ACKs :)
<@gribble> https://github.com/bitcoin/bitcoin/issues/22047 | index, rpc: Coinstatsindex follow-ups by fjahr · Pull Request #22047 · bitcoin/bitcoin · GitHub