zebra
zebra copied to clipboard
ZcashFoundation
The number of outbound connections reported by progress bars periodically glitches
Description
When syncing on Testnet, I noticed the number of outbound connections reported by the new progress bars periodically jumps from ~7 to the limit of 75 and then back to ~7. The period seems to be equal to the crawl_new_peer_interval setting.
Analysis
That's probably because the connection counter actually comes from the connection limit, which is updated when we start making a connection. (Not when we successfully connect.)
So every few minutes, we re-try all the failed peers at the same time, the connection limit goes to 75, and then it drops back when they all fail.
Possible Solutions
Reducing the progress bar update interval would hide this, but it would still occasionally happen. (And be visible for longer.)
The real fix would be to move the progress bar to the peer set. We track inbound/outbound on the ConnectedAddr type in ConnectionInfo, which is part of the LoadTrackedClient in the PeerSet.
