The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

• #24456 (blockman: Properly guard blockfile members by dongcarl)
• #24230 (indexes: Stop using node internal types and locking cs_main, improve sync logic by ryanofsky)
• #24220 (validation: don't re-acquire cs_main during IBD in CChainState::IsInitialBlockDownload() by jonatack)
• #24178 (p2p: Respond to getheaders if we have sufficient chainwork by sdaftuar)
• #24008 (assumeutxo: net_processing changes by jamesob)
• #23443 (p2p: Erlay support signaling by naumenkogs)
• #22778 (net processing: Reduce resource usage for inbound block-relay-only connections by jnewbery)
• #22564 (refactor: Move mutable globals cleared in ::UnloadBlockIndex to BlockManager by dongcarl)
• #21878 (Make all networking code mockable by vasild)
• #21160 (net/net processing: Move tx inventory into net_processing by jnewbery)
• #20799 (net processing: Only support version 2 compact blocks by jnewbery)
• #20726 (p2p: Add DISABLETX message for negotiating block-relay-only connections by sdaftuar)
• #20196 (net: fix GetListenPort() to derive the proper port by vasild)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

🐙 This pull request conflicts with the target branch and needs rebase.

_{Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".}

Thanks! But instead of splitting up net specifically, I think a better option is to add logging verbosity in general. Closing in favor of #24464.