Skip to content

Feat/p2p discovery layer - #28

Open
aimendj wants to merge 13 commits into
unstablefrom
feat/p2p-discovery-layer
Open

Feat/p2p discovery layer#28
aimendj wants to merge 13 commits into
unstablefrom
feat/p2p-discovery-layer

Conversation

@aimendj

@aimendj aimendj commented Apr 14, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@aimendj aimendj self-assigned this Apr 14, 2026
@aimendj
aimendj force-pushed the feat/p2p-discovery-layer branch from 5823832 to 2f3da48 Compare August 26, 2026 10:07
@aimendj
aimendj marked this pull request as ready for review August 26, 2026 20:00
@aimendj
aimendj requested review from munna0908 and tersec August 26, 2026 20:13
@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown

Unit Test Results

       5 files  ±    0     685 suites  +20   1m 31s ⏱️ +26s
   622 tests +  26     616 ✔️ +  33    6 💤  -   7  0 ±0 
3 270 runs  +130  3 240 ✔️ +165  30 💤  - 35  0 ±0 

Results for commit 5205b9c. ± Comparison against base commit a90edc8.

This pull request removes 1 and adds 27 tests. Note that renamed tests count towards both.
Bootstrap multiaddr: parseBootstrapAddress accepts /dns4/.../udp/.../quic-v1/p2p/...
AddressBook extend merges multiaddrs without duplicates
Bootstrap multiaddr: loadBootstrapNodes accepts /dns4/.../udp/.../quic-v1/p2p/...
connectViaConnQueue: bootstrap-style integration connects a live peer
connectViaConnQueue: multiple concurrent callers coalesce and all resolve true
connectViaConnQueue: timeout marks peer seen and does not hang
connectWorker: in-flight dial transitions to Dialing stage and clears after failure
connected peers are admitted to peerPool with Connected state upon libp2p connection
enqueueKadDiscoveredPeers respects rtable, AddressBook, and peer pool
enqueueKadDiscoveredPeers: nil Kad returns (0, 0)
kadBootstrap: dials bootstrap nodes and runs lookup on success
…

♻️ This comment has been updated with latest results.

Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/discovery.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/network.nim Outdated
Comment thread logos_chain/networking/discovery.nim Outdated
Comment thread tests/logos_chain/sync/test_ibd.nim Outdated
Comment thread logos_chain/conf.nim
Comment thread logos_chain/networking/network.nim Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants