Skip to content

network: p2p traffic exchange for algorand node#5939

Merged
algorandskiy merged 53 commits intomasterfrom
feature/p2p
Jun 28, 2024
Merged

network: p2p traffic exchange for algorand node#5939
algorandskiy merged 53 commits intomasterfrom
feature/p2p

Conversation

@algorandskiy
Copy link
Copy Markdown
Contributor

@algorandskiy algorandskiy commented Feb 21, 2024

Summary

Functional p2p support for gossip network. Includes:

  1. DHT advertisement and peers capabilities
  2. Hybrid networking when a node listens and communicates on both legacy websocket and libp2p networks
  3. HTTP over p2p support for catching up from p2p nodes
  4. p2p network cluster test scenarios
  5. libp2p and DHT metrics and logging handling

Contains PRs #5702, #5827, #5800, #5833, #5867, #5898, #5922, #5926, #5924, #5929, #5931, #5933, #5935, #5941, #5982, #5976, #5962, #6038

Test Plan

Test note

node_test re-enables some tests for local run as well as adds new tests resulting go test ./node takes 90s on this branch vs 30s on master

Eric-Warehime and others added 24 commits November 2, 2023 15:26
@algorandskiy algorandskiy added New Feature p2p Work related to the p2p project labels Feb 21, 2024
@algorandskiy algorandskiy requested review from cce and gmalouf February 21, 2024 21:10
@algorandskiy algorandskiy self-assigned this Feb 21, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 21, 2024

Codecov Report

Attention: Patch coverage is 54.79662% with 589 lines in your changes missing coverage. Please review.

Project coverage is 51.40%. Comparing base (63c0d5b) to head (4564473).

Files Patch % Lines
network/p2pNetwork.go 64.90% 101 Missing and 25 partials ⚠️
network/hybridNetwork.go 0.00% 111 Missing ⚠️
network/p2p/logger.go 0.00% 45 Missing ⚠️
network/p2p/p2p.go 47.67% 45 Missing ⚠️
network/p2p/streams.go 0.00% 41 Missing ⚠️
data/txHandler.go 56.62% 33 Missing and 3 partials ⚠️
network/p2p/http.go 0.00% 27 Missing ⚠️
network/p2p/peerstore/peerstore.go 39.47% 23 Missing ⚠️
network/wsNetwork.go 76.19% 19 Missing and 1 partial ⚠️
network/p2p/peerID.go 0.00% 18 Missing ⚠️
... and 20 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5939      +/-   ##
==========================================
- Coverage   55.90%   51.40%   -4.50%     
==========================================
  Files         482      488       +6     
  Lines       68608    69532     +924     
==========================================
- Hits        38353    35746    -2607     
- Misses      27650    31061    +3411     
- Partials     2605     2725     +120     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread util/metrics/opencensus.go
Comment thread network/p2pNetwork.go
Comment thread network/p2pNetwork.go
Comment thread network/p2pNetwork.go Outdated
Comment thread network/p2p/streams.go
Comment thread network/p2p/http.go
@algorandskiy
Copy link
Copy Markdown
Contributor Author

merged master to resolve conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New Feature p2p Work related to the p2p project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants