Swarm Bootstrap Nodes

[mriise]

Description

Adds bootstrap nodes in settings that will be connected to on node startup.

Link to issue

Please add a link to any relevant issues/tickets.

Type of change

• New feature (non-breaking change that adds functionality)

Please delete options that are not relevant.

Test plan (required)

Parsing testing is added with the other settings test. Tests actually dialing nodes will be added in a different PR since this is a perquisite to gossip/receipt testing anyways.

[codecov]

Codecov Report

Merging #201 (c37d577) into main (63c23aa) will increase coverage by 0.04%.
The diff coverage is 40.62%.

@@            Coverage Diff             @@
##             main     #201      +/-   ##
==========================================
+ Coverage   75.20%   75.25%   +0.04%     
==========================================
  Files          65       65              
  Lines        6107     6135      +28     
==========================================
+ Hits         4593     4617      +24     
- Misses       1514     1518       +4     

Files Changed	Coverage Δ
homestar-runtime/src/event_handler/swarm_event.rs	27.01% <0.00%> (+2.17%)	⬆️
homestar-runtime/src/network/swarm.rs	58.09% <42.10%> (+2.41%)	⬆️
homestar-runtime/src/event_handler.rs	92.68% <100.00%> (+0.18%)	⬆️
homestar-runtime/src/settings.rs	96.29% <100.00%> (+0.04%)	⬆️

[mriise]

tests will probably fail at the moment since i havent setup nix properly on my laptop yet. needed to get this change uploaded though.

[zeeshanlakhani]

Ok, putting together some additional work here @mriise, to close this out:

• In swarm_event.rs, need to match on the SwarmEvent::ConnectionEstablished and add kademlia.add_address to add the peer id and peer address. I had implemented this earlier but wanted to tackle this separately again, as here, so 👍🏽.
  • As part of this, we'll want the event handler to keep a local set (fnvhashset) of trusted peers that are actually "connected", vs those we configured.
  • On dial, in the startup here, add addresses to the set and to kademlia. On, SwarmEvent::OutgoingConnectionError and SwarmEvent::ConnectionClosed, remove from the set.
  • Currently, the set is just a "view" on trusted ones that are connected. Could be useful for query for now.
• The explicit peer piece we discussed, as these are trusted, bootstrap nodes, since we configured them.

[zeeshanlakhani]

@mriise some minor cleanup mentions here and a suggestion for startup cleanup.

[zeeshanlakhani]

@mriise I'm about to push a small reworking of when scheduling runs vs the ahead-of-time parsing. Let's try to get the last few bits lined up for this one and get it in. I know you've been pushing to the other branch as well :).

[mriise]

ci shouldn't be failing like this...

[zeeshanlakhani]

Good work @mriise!