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.

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

chore: upgrade nwaku to 0.34.0 and update tests suite for compatibility #2170

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

danisharora099
Copy link
Collaborator

@danisharora099 danisharora099 commented Oct 9, 2024

Description

This pull request upgrades the nwaku dependency to version 0.34.0 and includes various improvements and fixes in the test suite to accommodate this upgrade. Key changes include:

  • Updates to the CI workflow configuration to use the new nwaku version.
  • Enhancements and adjustments to the test suite:
    • service nodes (nwaku) spawned together connect to each other via Relay
    • adds deduplication logic for ecies encrypted payloads to avoid being considered as multiple messages
    • single node lightpush tests now expected to fail (nwaku requires minimum 1 peer part of the mesh)

Notes

Contribution checklist:

  • covered by unit tests;
  • covered by e2e test;
  • add ! in title if breaks public API;

@danisharora099 danisharora099 changed the title chore: upgrade to 0.32 chore: upgrade nwaku to 0.32 Oct 9, 2024
Copy link

github-actions bot commented Oct 9, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
Waku node 86.63 KB (0%) 1.8 s (0%) 9.1 s (-3.35% 🔽) 10.8 s
Waku Simple Light Node 137.06 KB (0%) 2.8 s (0%) 16.6 s (+28% 🔺) 19.3 s
ECIES encryption 22.88 KB (0%) 458 ms (0%) 6.6 s (+63.32% 🔺) 7.1 s
Symmetric encryption 22.37 KB (0%) 448 ms (0%) 4.5 s (+28.45% 🔺) 4.9 s
DNS discovery 70.54 KB (0%) 1.5 s (0%) 10 s (-12.04% 🔽) 11.4 s
Peer Exchange discovery 71.82 KB (0%) 1.5 s (0%) 8.9 s (-6.9% 🔽) 10.3 s
Local Peer Cache Discovery 65.17 KB (0%) 1.4 s (0%) 9.3 s (+4.58% 🔺) 10.6 s
Privacy preserving protocols 76.16 KB (0%) 1.6 s (0%) 12.2 s (-6.45% 🔽) 13.8 s
Waku Filter 80.98 KB (0%) 1.7 s (0%) 12.4 s (+8.86% 🔺) 14 s
Waku LightPush 75.72 KB (0%) 1.6 s (0%) 10.1 s (+33.18% 🔺) 11.6 s
History retrieval protocols 77.63 KB (0%) 1.6 s (0%) 10.1 s (+2.99% 🔺) 11.7 s
Deterministic Message Hashing 7.39 KB (0%) 148 ms (0%) 4.2 s (+74.02% 🔺) 4.3 s

@danisharora099 danisharora099 changed the title chore: upgrade nwaku to 0.32 chore: upgrade nwaku to 0.33 Oct 10, 2024
@danisharora099 danisharora099 changed the title chore: upgrade nwaku to 0.33 chore: upgrade nwaku to 0.34.0 Jan 17, 2025
@danisharora099 danisharora099 marked this pull request as ready for review January 22, 2025 12:11
@danisharora099 danisharora099 requested a review from a team as a code owner January 22, 2025 12:11
@danisharora099 danisharora099 linked an issue Jan 22, 2025 that may be closed by this pull request
@danisharora099 danisharora099 changed the title chore: upgrade nwaku to 0.34.0 chore: upgrade nwaku to 0.34.0 and update tests suite for compatibility Jan 22, 2025
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.

chore: upgrade nwaku to 0.34.0 and check interop tests
1 participant