Skip to content

chore: extra reliability tests 2 #2450

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

Merged
merged 20 commits into from
Jul 29, 2025
Merged

Conversation

fbarbu15
Copy link
Collaborator

@fbarbu15 fbarbu15 commented Jul 8, 2025

  • Added 3 more tests on reliability side involving sending and receiving messages via filter protocol under unreliable network conditions (similar on how we did on status side): low bandwidth, packet loss, high latency (using tc package on linux)
  • Extracted common code into reusable methods
  • See results https://github.com/waku-org/js-waku/actions/runs/16274416839
  • Moved the test timeout to a single place

Copy link

github-actions bot commented Jul 8, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
Waku node 70.31 KB (0%) 1.5 s (0%) 898 ms (+13.06% 🔺) 2.4 s
Waku Simple Light Node 122.09 KB (0%) 2.5 s (0%) 1.4 s (+33.18% 🔺) 3.8 s
ECIES encryption 20.86 KB (0%) 418 ms (0%) 582 ms (+63.66% 🔺) 999 ms
Symmetric encryption 20.3 KB (0%) 406 ms (0%) 262 ms (-41.71% 🔽) 668 ms
DNS discovery 51.95 KB (0%) 1.1 s (0%) 564 ms (-30.51% 🔽) 1.7 s
Peer Exchange discovery 52.4 KB (0%) 1.1 s (0%) 620 ms (+3.76% 🔺) 1.7 s
Local Peer Cache Discovery 46.1 KB (0%) 922 ms (0%) 528 ms (-19.27% 🔽) 1.5 s
Privacy preserving protocols 53.54 KB (0%) 1.1 s (0%) 672 ms (-13.01% 🔽) 1.8 s
Waku Filter 55.25 KB (0%) 1.2 s (0%) 931 ms (+13.95% 🔺) 2.1 s
Waku LightPush 52.91 KB (0%) 1.1 s (0%) 565 ms (+26.72% 🔺) 1.7 s
History retrieval protocols 59.18 KB (0%) 1.2 s (0%) 841 ms (+2.97% 🔺) 2.1 s
Deterministic Message Hashing 28.42 KB (0%) 569 ms (0%) 352 ms (-52.57% 🔽) 920 ms

@fbarbu15 fbarbu15 changed the title Chore/extra reliability tests 2 chore: extra reliability tests 2 Jul 9, 2025
@fbarbu15 fbarbu15 marked this pull request as ready for review July 15, 2025 08:35
@fbarbu15 fbarbu15 requested a review from a team as a code owner July 15, 2025 08:35
@fbarbu15 fbarbu15 requested review from weboko and adklempner July 15, 2025 08:35
@weboko
Copy link
Collaborator

weboko commented Jul 17, 2025

@fbarbu15 should it be reviewed?

@fbarbu15
Copy link
Collaborator Author

@fbarbu15 should it be reviewed?

@weboko yes please

@fbarbu15 fbarbu15 merged commit c1f9471 into master Jul 29, 2025
12 of 13 checks passed
@fbarbu15 fbarbu15 deleted the chore/extra-reliability-tests-2 branch July 29, 2025 07:17
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