< phantomcircuit> luke-jr, i dont understand why the v3 addresses are more vulnerable to the directory nodes being ddos'd than the v2 addresses are
< phantomcircuit> that seems quite ... strange
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/094402430925...c4458cc3a1d1
< bitcoin-git> bitcoin/master fad1f0f MarcoFalke: net: Remove unused cs_feeFilter
< bitcoin-git> bitcoin/master c4458cc fanquake: Merge #18819: net: Replace cs_feeFilter with simple std::atomic
< bitcoin-git> [bitcoin] fanquake merged pull request #18819: net: Replace cs_feeFilter with simple std::atomic (master...2004-netNoRecursiveMutex) https://github.com/bitcoin/bitcoin/pull/18819
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c4458cc3a1d1...bd6af53e1f8e
< bitcoin-git> bitcoin/master faa8f68 MarcoFalke: Replace boost::variant with std::variant
< bitcoin-git> bitcoin/master bd6af53 fanquake: Merge #20480: Replace boost::variant with std::variant
< bitcoin-git> [bitcoin] fanquake merged pull request #20480: Replace boost::variant with std::variant (master...2011-noBoostVariant) https://github.com/bitcoin/bitcoin/pull/20480
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/bd6af53e1f8e...9c0b76c709c1
< bitcoin-git> bitcoin/master faabc26 MarcoFalke: test: Replace getmempoolentry with testmempoolaccept in MiniWallet
< bitcoin-git> bitcoin/master 9c0b76c MarcoFalke: Merge #20876: test: Replace getmempoolentry with testmempoolaccept in Mini...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #20876: test: Replace getmempoolentry with testmempoolaccept in MiniWallet (master...2101-testMiniWalletNoSubmit) https://github.com/bitcoin/bitcoin/pull/20876
< bitcoin-git> [gui] jonasschnelli merged pull request #161: Add PeerTableModel::StatsRole to prevent data layer violation (master...201223-ban) https://github.com/bitcoin-core/gui/pull/161
< bitcoin-git> [bitcoin] jonasschnelli pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/9c0b76c709c1...616eace02acd
< bitcoin-git> bitcoin/master 35007ed Hennadii Stepanov: qt: Add PeerTableModel::StatsRole
< bitcoin-git> bitcoin/master 49c6040 Hennadii Stepanov: qt: Use PeerTableModel::StatsRole
< bitcoin-git> bitcoin/master b3e9bca Hennadii Stepanov: qt, refactor: Drop no longer used PeerTableModel::getNodeStats function
< aj> MarcoFalke: (not sure if you're comment on the 22.0 schedule was related to mine; i just fixed the year from last year to this year, and made a joke about it still being last march)
< aj> *your
< wumpus> aj: lol!
< bitcoin-git> [bitcoin] gruve-p closed pull request #20665: Build: update clang patch to use heredoc (master...master) https://github.com/bitcoin/bitcoin/pull/20665
< bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/616eace02acd...675af2a515f2
< bitcoin-git> bitcoin/master 94d335d Vasil Dimov: net: allow CSubNet of non-IP networks
< bitcoin-git> bitcoin/master 39b4329 Vasil Dimov: test: add test for banning of non-IP addresses
< bitcoin-git> bitcoin/master 675af2a Wladimir J. van der Laan: Merge #20852: net: allow CSubNet of non-IP networks
< bitcoin-git> [bitcoin] laanwj merged pull request #20852: net: allow CSubNet of non-IP networks (master...non_ip_subnets) https://github.com/bitcoin/bitcoin/pull/20852
< bitcoin-git> [bitcoin] fanquake closed pull request #20849: net: disconnect peers by address without using a subnet (master...disconnect_by_subnet_fix) https://github.com/bitcoin/bitcoin/pull/20849
< MarcoFalke> aj: Wasn't related. Just a comment on the schedule in general
< MarcoFalke> Maybe I should bring it up in the meeting?
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #20901: [0.21.1]: rc1 Backports (0.21...2101-2101Backports) https://github.com/bitcoin/bitcoin/pull/20901
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/675af2a515f2...5cce60710525
< bitcoin-git> bitcoin/master acebb79 Hennadii Stepanov: refactor, move-only: Relocate CNode private members
< bitcoin-git> bitcoin/master 3642b2e Hennadii Stepanov: refactor, net: Increase CNode data member encapsulation
< bitcoin-git> bitcoin/master 5cce607 MarcoFalke: Merge #20373: refactor, net: Increase CNode data member encapsulation
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #20373: refactor, net: Increase CNode data member encapsulation (master...201111-cnode) https://github.com/bitcoin/bitcoin/pull/20373
< wumpus> i still type 2020 for year automatically but i have that every year, but the eternal march feeling is definitely a thing
< wumpus> MarcoFalke: i'm okay with moving the schedule back by a month, but then i really want 0.21.0 out soon
< wumpus> aligning with ubuntu doesn't seem that relevant to me
< vasild> MarcoFalke: https://github.com/bitcoin/bitcoin/pull/20852#issuecomment-757934752 you mean all commands with the latest code?
< wumpus> in any csae, i've always based the schedule on the *real* release date (-final)
< wumpus> normally i don't even post the schedule for the next release while the previous one is still open, but it was slipping too much and this at least gives some kind of time window
< wumpus> it's likely that 22.0 will slip too by at least a month
< wumpus> so i'm not sure how much it makes sense to plan more than one release ahead in the first place
< jonasschnelli> agree
< MarcoFalke> Whatever we do, it just seems suboptimal to do any kind of release process (branch-off, bugfixes, -final) during december
< MarcoFalke> vasild: yes
< MarcoFalke> vasild: Maybe I checked out the previous commit. Sorry, I destroyed the vm I was testing in. Will re-check
< MarcoFalke> wumpus: It is not about aligning with ubuntu. It is more about picking months that don't collide with major holidays (like december or august)
< bitcoin-git> [gui] hebasto closed pull request #174: refactor: No need to implement index for non-hierarchical models (master...210102-index) https://github.com/bitcoin-core/gui/pull/174
< bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/5cce60710525...6d81d7aa87a1
< bitcoin-git> bitcoin/master fa39cdd MarcoFalke: refactor: Use C++17 std::array deduction for OUTPUT_TYPES
< bitcoin-git> bitcoin/master aaaa987 MarcoFalke: refactor: Use C++17 std::array deduction for ALL_FEE_ESTIMATE_HORIZONS
< bitcoin-git> bitcoin/master 6d81d7a fanquake: Merge #20787: Use C++17 std::array deduction for OUTPUT_TYPES, ALL_FEE_EST...
< bitcoin-git> [bitcoin] fanquake merged pull request #20787: Use C++17 std::array deduction for OUTPUT_TYPES, ALL_FEE_ESTIMATE_HORIZONS (master...2012-WswitchFeeEstimateHorizon) https://github.com/bitcoin/bitcoin/pull/20787
< MarcoFalke> vasild: Just checked that I was using the latest version
< vasild> trying now...
< vasild> MarcoFalke: I also get ::/0 instead of the .onion address :/
< MarcoFalke> thx, but also :((
< MarcoFalke> Let's hope it is easy to fix
< vasild> I agree the unit test from #20849 is valuable, will include it with the fix of the above
< gribble> https://github.com/bitcoin/bitcoin/issues/20849 | net: disconnect peers by address without using a subnet by vasild · Pull Request #20849 · bitcoin/bitcoin · GitHub
< vasild> oh, of course we can't serialize a torv3 addr in a stream that does not have ADDRV2_FORMAT
< vasild> MarcoFalke: wumpus: #20852 serializes the torv3 addr in banlist.dat as 16 zeroes in addrv1 format (:: IPv6 address)
< gribble> https://github.com/bitcoin/bitcoin/issues/20852 | net: allow CSubNet of non-IP networks by vasild · Pull Request #20852 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] ryanofsky closed pull request #18600: [wallet] Track conflicted transactions removed from mempool and fix UI notifications (master...2020-03-wallet-track-conflicts) https://github.com/bitcoin/bitcoin/pull/18600
< wumpus> MarcoFalke: Whatever we do, it just seems suboptimal to do any kind of release process (branch-off, bugfixes, -final) during december <- I agree with that and will take that into account for 0.23 planning, what I was doubtful about is whether that needs to affect the 0.22 schedule
< wumpus> vasild: whoops
< MarcoFalke> wumpus: Probably not affecting 22.0, but good to keep in mind for 23.0, thx
< bitcoin-git> [bitcoin] vasild opened pull request #20904: net: (de)serialize CSubNet in addrv2 format (master...subnet_ser_addrv2) https://github.com/bitcoin/bitcoin/pull/20904
< vasild> wumpus: MarcoFalke: ^
< vasild> I am sorry. #20852 was not well thought and was not ready for merging.
< gribble> https://github.com/bitcoin/bitcoin/issues/20852 | net: allow CSubNet of non-IP networks by vasild · Pull Request #20852 · bitcoin/bitcoin · GitHub
< * jonatack> twice bitten, adds "serialization file compat" to review checklist
< MarcoFalke> can someone block sheffine, please
< sipa> MarcoFalke: done
< MarcoFalke> thx
< bitcoin-git> [bitcoin] gruve-p opened pull request #20906: contrib: embed C++11 patch in install_db4.sh (master...master) https://github.com/bitcoin/bitcoin/pull/20906
< bitcoin-git> [bitcoin] MarcoFalke pushed 7 commits to master: https://github.com/bitcoin/bitcoin/compare/6d81d7aa87a1...6af013792f1b
< bitcoin-git> bitcoin/master 5bc04e8 Amiti Uttarwar: [rpc/net] Introduce addconnection to test outbounds & blockrelay
< bitcoin-git> bitcoin/master 3997ab9 Amiti Uttarwar: [test] Add test framework support to create outbound connections.
< bitcoin-git> bitcoin/master 99791e7 Amiti Uttarwar: [test/refactor] P2PBlocksOnly - simplify transaction creation using blockt...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #19315: [tests] Allow outbound & block-relay-only connections in functional tests. (master...2020-06-test-outbounds) https://github.com/bitcoin/bitcoin/pull/19315