Skip to content

Conversation

@rlve
Copy link
Contributor

@rlve rlve commented Sep 8, 2025

Changes:

  • split libp2p/protocols/rendezvous.nim into:
    • libp2p/protocols/rendezvous/rendezvous.nim
    • libp2p/protocols/rendezvous/protobuf.nim
  • add tests for Rendezvous Protobuf

Note

No other code changes to the Rendezvous and Rendezvous Protobuf implementations besides updating imports.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025

🏁 Performance Summary

Commit: d2c5f8d00148a440b0334586ebf3e008b8ac5eb0

Scenario Nodes Total messages sent Total messages received Latency min (ms) Latency max (ms) Latency avg (ms)
Base test 10 100 900 0.305 2.482 1.049
Low Bandwidth rate 256kbit burst 8kbit limit 5000 10 100 900 0.326 21.626 3.252
Packet Reorder 15% 40% with 2ms delay 10 100 900 0.279 5.703 2.917
Queue Limit 5 10 100 900 0.302 2.459 0.950
Latency 100ms 20ms 10 100 900 36.226 214.795 120.370
Burst Loss 8% 30% 10 100 900 0.379 2.351 1.047
Duplication 2% 10 100 900 0.352 2.717 1.072
Corruption 0.5% 10 100 900 0.374 2.827 1.070
Packet Loss 5% 10 100 900 0.343 2.343 1.072
Combined Network Conditions 10 100 900 0.235 249.532 118.572

📊 View Latency History and full Container Resources in the Workflow Summary

@rlve rlve marked this pull request as ready for review September 8, 2025 15:56
@rlve rlve requested a review from a team as a code owner September 8, 2025 15:56
@rlve rlve requested review from gmelodie and vladopajic September 8, 2025 15:56
@github-project-automation github-project-automation bot moved this from new to In Progress in nim-libp2p Sep 8, 2025
@rlve rlve merged commit 62388a7 into master Sep 9, 2025
15 of 16 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to done in nim-libp2p Sep 9, 2025
@rlve rlve deleted the test-rendezvous-4 branch September 9, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants