< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f0a76b3dbc64...6754e1928b00
< bitcoin-git> bitcoin/master faa0d94 MarcoFalke: fuzz: Avoid timeout in EncodeBase58
< bitcoin-git> bitcoin/master 6754e19 fanquake: Merge bitcoin/bitcoin#21922: fuzz: Avoid timeout in EncodeBase58
< bitcoin-git> [bitcoin] fanquake merged pull request #21922: fuzz: Avoid timeout in EncodeBase58 (master...2105-fuzzRpcTimeout) https://github.com/bitcoin/bitcoin/pull/21922
< bitcoin-git> [bitcoin] fanquake closed pull request #21894: gitignore: Add JetBrains CLion (master...gitignore-clion) https://github.com/bitcoin/bitcoin/pull/21894
< fanquake> Is our no depends fuzzer just constantly timing out now
< fanquake> Is there a recent change that needs addressing / reversion? Or are we just running into CI limits again
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6754e1928b00...03e16cb02730
< bitcoin-git> bitcoin/master b95f7f8 Hennadii Stepanov: build, qt, refactor: Drop sed commands for win32-g++/qmake.conf
< bitcoin-git> bitcoin/master 03e16cb fanquake: Merge bitcoin/bitcoin#21593: build, qt, refactor: Get rid of some sed comm...
< bitcoin-git> [bitcoin] fanquake merged pull request #21593: build, qt, refactor: Get rid of some sed command instances (master...210404-sed) https://github.com/bitcoin/bitcoin/pull/21593
< bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/03e16cb02730...2e30e328a7a4
< bitcoin-git> bitcoin/master 30e4448 Hennadii Stepanov: refactor: Make TraceThread a non-template free function
< bitcoin-git> bitcoin/master a508f71 Hennadii Stepanov: refactor: Use appropriate thread constructor
< bitcoin-git> bitcoin/master 792be53 Hennadii Stepanov: refactor: Replace std::bind with lambdas
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #19064: refactor: Cleanup thread ctor calls (master...200524-bind) https://github.com/bitcoin/bitcoin/pull/19064
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21925: doc: Update bips.md for 0.21.1 (master...2105-docBips) https://github.com/bitcoin/bitcoin/pull/21925
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21927: fuzz: Run const CScript member functions only once (master...2105-fuzzCscript) https://github.com/bitcoin/bitcoin/pull/21927
< MarcoFalke> jnewbery: wumpus: Can delete the branch https://github.com/bitcoin/bitcoin/tree/2021-05-use-mutex-message-handling ?
< promag> fanquake: happy to assist you with #21851
< gribble> https://github.com/bitcoin/bitcoin/issues/21851 | [WIP] build: support cross-compiling for arm64-apple-darwin20 (Apple M1) in depends by fanquake · Pull Request #21851 · bitcoin/bitcoin · GitHub
< MarcoFalke> achow101: fanquake: Might have to delete some old fuzz inputs to speed up. Though, it would be good to find a long term solution and be clear of the 2 h timeout
< MarcoFalke> Maybe increase the cpu for the fuzz task?
< promag> MarcoFalke: they say $10 gives twice the time?
< MarcoFalke> Even if it did, I am not sure if waiting 4 h for CI is reasonable
< promag> yes, for some stuff thats pita
< bitcoin-git> [bitcoin] jonatack reopened pull request #21506: p2p, refactor: make NetPermissionFlags an enum class (master...NetPermissionFlags-enum-class) https://github.com/bitcoin/bitcoin/pull/21506
< bitcoin-git> [bitcoin] Sjors opened pull request #21928: wallet: allow toggling external_signer flag (master...2021/05/hww-toggle) https://github.com/bitcoin/bitcoin/pull/21928
< yanmaani> luke-jr: Ignoring the time factor
< yanmaani> sipa: inside the bitcoin core codebase
< bitcoin-git> [bitcoin] laanwj pushed 11 commits to master: https://github.com/bitcoin/bitcoin/compare/2e30e328a7a4...01624a7db36a
< bitcoin-git> bitcoin/master 30daf76 Carl Dong: guix: Add guix-attest script
< bitcoin-git> bitcoin/master 5926432 Carl Dong: guix: Add guix-verify script
< bitcoin-git> bitcoin/master b5fd89c Carl Dong: guix-attest: Only use cross-platform flags for find+xargs
< bitcoin-git> [bitcoin] laanwj merged pull request #21462: guix: Add guix-{attest,verify} scripts (master...2021-03-guix-verify-and-attest) https://github.com/bitcoin/bitcoin/pull/21462
< yanmaani> sipa: see for example the documentation for `lockunspent`
< yanmaani> > bitcoin-cli lockunspent false "[{\"txid\":\"a08e6907dbbd3d809776dbfc5d82e371b764ed838b5655e72f463568df1aadf0\",\"vout\":1}]"
< yanmaani> instead of '[{"txid":"...","vout":1}]'
< wumpus> MarcoFalke: done
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/01624a7db36a...176842daa857
< bitcoin-git> bitcoin/master fa0ad7b MarcoFalke: doc: Remove mention of priority estimation
< bitcoin-git> bitcoin/master 176842d W. J. van der Laan: Merge bitcoin/bitcoin#21912: doc: Remove mention of priority estimation
< bitcoin-git> [bitcoin] laanwj merged pull request #21912: doc: Remove mention of priority estimation (master...2105-docFeeEst) https://github.com/bitcoin/bitcoin/pull/21912
< MarcoFalke> wumpus: thx
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21929: fuzz: Remove incorrect float round-trip serialization test (master...2105-fuzzFloat) https://github.com/bitcoin/bitcoin/pull/21929
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/176842daa857...91db985bc956
< bitcoin-git> bitcoin/master fa74bfc MarcoFalke: fuzz: Run const CScript member functions only once
< bitcoin-git> bitcoin/master 91db985 MarcoFalke: Merge bitcoin/bitcoin#21927: fuzz: Run const CScript member functions only...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21927: fuzz: Run const CScript member functions only once (master...2105-fuzzCscript) https://github.com/bitcoin/bitcoin/pull/21927
< wumpus> despite the forest of branch protection options added, github still doesn't have a way to prevent accidental creation of branches
< wumpus> "Cirrus CI / [no depends, sanitizers: fuzzer,address,undefined,integer] [focal]" is timing out a lot
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/91db985bc956...d2ec37221a56
< bitcoin-git> bitcoin/master faad68f MarcoFalke: index: Avoid async shutdown on init error
< bitcoin-git> bitcoin/master d2ec372 W. J. van der Laan: Merge bitcoin/bitcoin#21796: index: Avoid async shutdown on init error
< bitcoin-git> [bitcoin] laanwj merged pull request #21796: index: Avoid async shutdown on init error (master...2104-indexInit) https://github.com/bitcoin/bitcoin/pull/21796
< wumpus> looks like it fails "after two hours", but not on any single test hanging just being slow
< bitcoin-git> [bitcoin] MarcoFalke reopened pull request #21920: build: improve macro for testing -latomic requirement (master...2105-buildAtomicLink) https://github.com/bitcoin/bitcoin/pull/21920
< MarcoFalke> ok, let me bump the cpu
< bitcoin-git> [bitcoin] jonatack opened pull request #21930: doc: update enumerator naming in developer notes (master...developer-notes-enum-naming) https://github.com/bitcoin/bitcoin/pull/21930
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/d2ec37221a56...6b49d88a5dda
< bitcoin-git> bitcoin/master 9c891b6 eugene: net: initialize nMessageSize to max uint32_t instead of -1
< bitcoin-git> bitcoin/master 6b49d88 W. J. van der Laan: Merge bitcoin/bitcoin#21905: net: initialize nMessageSize to uint32_t max
< bitcoin-git> [bitcoin] laanwj merged pull request #21905: net: initialize nMessageSize to uint32_t max (master...cmessageheader_signchange_05102021) https://github.com/bitcoin/bitcoin/pull/21905
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21931: ci: Bump cirrus fuzz CPUs to avoid timeout (master...2105-ciCpuFuzz) https://github.com/bitcoin/bitcoin/pull/21931
< bitcoin-git> [bitcoin] kittywhiskers opened pull request #21932: depends: update Qt 5.9 source url (0.21...patch-1) https://github.com/bitcoin/bitcoin/pull/21932
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6b49d88a5dda...79da18a9ebbb
< bitcoin-git> bitcoin/master bc45388 Sjors Provoost: build: add *~ to .gitignore
< bitcoin-git> bitcoin/master 79da18a W. J. van der Laan: Merge bitcoin/bitcoin#21911: build: add configure~ to .gitignore
< bitcoin-git> [bitcoin] laanwj merged pull request #21911: build: add configure~ to .gitignore (master...2021/05/configure) https://github.com/bitcoin/bitcoin/pull/21911
< dongcarl> A naive idea: using _timed_ variants of mutexes (e.g. recursive_timed_mutex) so that for HTTP-based RPCs we can polietly return a 503 after a reasonable amount of time to clients instead of having TCP connections close abruptly.
< wumpus> it's not the server closing TCP connections abruptly
< wumpus> ever
< wumpus> the only timeouts are in the client
< dongcarl> wumpus: Oh right...
< wumpus> that would be strange some commands can take a long time to run, like unspent stats
< luke-jr> however, IIRC, we keep our resources tied up until we would otherwise return a result, even after the client closes the connection
< luke-jr> could this possibly help avoid that?
< wumpus> for long-running commands it could, in principle, periodically check if the RPC connection is still open, i guess
< wumpus> not sure how practical this is or it violates various levels of abstraction
< bitcoin-git> [gui] hebasto opened pull request #325: Align numbers in the "Peer Id" column to the right (master...210512-peer) https://github.com/bitcoin-core/gui/pull/325
< bitcoin-git> [bitcoin] laanwj merged pull request #21584: Fix assumeutxo crash due to invalid base_blockhash (master...2104-assumeutxoCrash02) https://github.com/bitcoin/bitcoin/pull/21584
< bitcoin-git> [bitcoin] laanwj pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/79da18a9ebbb...ee9befe8b4bb
< bitcoin-git> bitcoin/master faa921f MarcoFalke: move-only: Add util/hash_type
< bitcoin-git> bitcoin/master 0000007 MarcoFalke: refactor: Remove unused code
< bitcoin-git> bitcoin/master fa5668b MarcoFalke: refactor: Use type-safe assumeutxo hash
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/ee9befe8b4bb...0ab6ff5e3759
< bitcoin-git> bitcoin/master fa397a6 MarcoFalke: ci: Bump cirrus fuzz CPUs to avoid timeout
< bitcoin-git> bitcoin/master 0ab6ff5 MarcoFalke: Merge bitcoin/bitcoin#21931: ci: Bump cirrus fuzz CPUs to avoid timeout
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21931: ci: Bump cirrus fuzz CPUs to avoid timeout (master...2105-ciCpuFuzz) https://github.com/bitcoin/bitcoin/pull/21931
< bitcoin-git> [bitcoin] luke-jr opened pull request #21934: RPC/blockchain: getblockchaininfo: Include versionbits signalling details during LOCKED_IN (master...rpc_getblockchaininfo_lockedin_statistics) https://github.com/bitcoin/bitcoin/pull/21934
< bitcoin-git> [bitcoin] Sjors opened pull request #21935: Enable external signer support for GUI builds (master...2021/05/hww-qt-compile) https://github.com/bitcoin/bitcoin/pull/21935
< bitcoin-git> [bitcoin] practicalswift opened pull request #21936: fuzz: Terminate immediately if a fuzzing harness ever tries to create a TCP socket (belt and suspenders) (master...terminate-on-tcp-socket) https://github.com/bitcoin/bitcoin/pull/21936