Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Seeker autoconnect #7798

Merged
merged 9 commits into from
Nov 22, 2024
Merged

Seeker autoconnect #7798

merged 9 commits into from
Nov 22, 2024

Commits on Nov 22, 2024

  1. common/test: fix -O3 compile error with gcc-12 (Ubuntu 12.3.0-17ubunt… 

    …u1) 12.3.0

```
common/test/run-splice_script.c: In function ‘main’:
common/test/run-splice_script.c:349:17: error: ‘%.*s’ directive argument is null [-Werror=format-overflow=]
  349 |         printf("%.*s\n", (int)len, str);
      |                 ^~~~
cc1: all warnings being treated as errors
make: *** [Makefile:297: common/test/run-splice_script.o] Error 1
make: *** Waiting for unfinished jobs....
```

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
    @rustyrussell
    rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    8b948af View commit details
    Browse the repository at this point in the history

  2. gossipd: seeker: allocate gossiper array at init 

    This will let us change the default gossipers at runtime
    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    4822544 View commit details
    Browse the repository at this point in the history

  3. gossipd: add request to connect to new gossip peer 

    Gossipd uses this to ask lightningd -> connectd to initiate
a connection to a new gossip peer.  This can be used when
there are insufficient peers already connected to gossip with.

Changelog-Changed: Gossipd can now request connections to additional nodes for improved gossip sync
    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    e40cd5f View commit details
    Browse the repository at this point in the history

  4. gossipd: seeker: select random peer and tell lightningd 

    This does not validate a node announcement and address, but it
does select a node at random from the gossmap and asks lightningd
to attempt a connection to it.
    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    42faa9f View commit details
    Browse the repository at this point in the history

  5. pytest: Add seeker autoconnect test

    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    c0f7e16 View commit details
    Browse the repository at this point in the history

  6. lightningd: add option for minimum seeker autoconnect peers 

    Changelog-added: Added option --autoconnect-seeker-peers, allowing seeker to reach out to new nodes for additional gossip.
    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    c119cc1 View commit details
    Browse the repository at this point in the history

  7. gossipd: use autoconnect-seeker-peers setting

    @endothermicdev @rustyrussell
    endothermicdev authored and rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    cda922e View commit details
    Browse the repository at this point in the history

  8. gossmap: implement gossmap_random_node(), use it in gossipd. 

    It's easy for gossmap, since it has access to the htable.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
    @rustyrussell
    rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    47f4ba1 View commit details
    Browse the repository at this point in the history

  9. gossipd: hand gossmap to gossmap_manage_get_node_addresses, not gossm… 

    …ap_manage.

We don't want to to refresh the gossmap internally: this could invalidate the
gossmap held by the current callers.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
    @rustyrussell
    rustyrussell committed Nov 22, 2024
    Configuration menu
    Copy the full SHA
    1bfe6aa View commit details
    Browse the repository at this point in the history