Handle connection better: make connectd know which peers are important #7630
Commits on Aug 31, 2024
-
connectd: send self-advertizing gossip rather than having gossipd do it.
It's now trivial for us to do this ourselves, since we have gossmap. Signed-off-by: Rusty Russell <[email protected]>
-
connectd: get our own addresses to contact node from node_announcements.
Let lightningd feed us hints to try first, but we can extract the addresses from node_announcement messages ourselves. (Lightningd used to ask gossipd on our behalf: this is far simpler!) One side effect of this is that we don't hand back address hints given to us by lightningd: it would use these again for reconnecting. This is breaks test_sendpay_grouping, so we disable it temporarily. Signed-off-by: Rusty Russell <[email protected]>
-
connectd: expose --dev-no-reconnect and --dev-fast-reconnect options.
Once connectd is controlling reconnections, it'll need these. Signed-off-by: Rusty Russell <[email protected]>
-
lightningd: generalize peer_any_channel to filter on entire channel, …
…not just state. We're going to use this to ask if there are any channels which make it important to reconnect to the peer. Signed-off-by: Rusty Russell <[email protected]>
-
wallet: save last known address.
If we connected out, remember that address. We always remember the last address, but that may be an incoming address. This is explicitly the last outgoing address which worked. Signed-off-by: Rusty Russell <[email protected]>
-
common: routine to make wireaddr_internal from wireaddr.
Signed-off-by: Rusty Russell <[email protected]>
-
recovery: save last_known_addr for peer if we know it.
This is more useful than the last address, which may be it connecting to us. And use it when we restore it. Signed-off-by: Rusty Russell <[email protected]>
-
connectd: reconnect for non-transient connections.
Rather than have lightningd call us repeatedly to try to connect, have it tell us what peers are transient and aren't, and connectd will automatically try to maintain that connection. There's a new "downgrade_peer" message to tell it a peer is now transient: to make it non-transient we simply tell connectd to connect as a non-transient. Signed-off-by: Rusty Russell <[email protected]> Changelog-Changed: `connectd` now handles maintaining/reconnecting to important peers, and we remember the last successful address we connected to.
-
pytest: restore test_sendpay_grouping test.
Signed-off-by: Rusty Russell <[email protected]>
-
connectd: remove transient flag.
The important flag replaces it, and now we can be more intelligent about eviction in overload. Signed-off-by: Rusty Russell <[email protected]>
-
connectd: limit to 10 connections at once.
We wait until a connection fails, or a subd is connected to the peer, before letting another one through. This should prevent us from overwhelming lightningd on large nodes, but unlike the previous back-off, it's based on how fast lightningd is, not an arbitrary time. We also let one through each second, in case we're connecting to many, but not doing anything but gossip (e.g. 100 explicit connect commands). Signed-off-by: Rusty Russell <[email protected]> Changelog-Changed: Reconnecting to peers at startup should be significantly faster (dependent on machine speed).
-
pytest: add test for connection ratelimiting.
Signed-off-by: Rusty Russell <[email protected]>
