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: sharding tests update #1060

Merged
merged 107 commits into from
May 4, 2024
Merged

chore: sharding tests update #1060

merged 107 commits into from
May 4, 2024

Conversation

romanzac
Copy link
Collaborator

Description

Batch of tests for Waku sharding functionality to extend existing tests

Changes

Tests available to run:
cd go-waku/waku/v2/node
go test

@status-im-auto
Copy link

status-im-auto commented Mar 19, 2024

Jenkins Builds

Click to see older builds (55)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 0e0ff46 #1 2024-03-19 10:19:22 ~1 min nix-flake 📄log
✔️ bbf7487 #2 2024-03-19 10:21:29 ~2 min nix-flake 📄log
✔️ 77468d6 #3 2024-03-20 04:15:07 ~1 min nix-flake 📄log
✔️ abec5b0 #4 2024-03-20 07:02:23 ~1 min nix-flake 📄log
✔️ a8689eb #5 2024-03-20 09:47:55 ~1 min nix-flake 📄log
✔️ a3ec282 #6 2024-03-20 10:01:01 ~2 min nix-flake 📄log
✔️ 3fe24f6 #7 2024-03-21 10:39:08 ~1 min nix-flake 📄log
✔️ c0e44b6 #8 2024-03-21 10:41:24 ~1 min nix-flake 📄log
✔️ 77f1f59 #9 2024-03-21 14:13:51 ~1 min nix-flake 📄log
✔️ 6495860 #10 2024-03-23 05:17:43 ~1 min nix-flake 📄log
✔️ ff01699 #11 2024-03-23 05:27:28 ~1 min nix-flake 📄log
✔️ 95f3ff0 #12 2024-03-23 05:45:13 ~1 min nix-flake 📄log
✔️ d26f178 #13 2024-03-23 05:56:32 ~1 min nix-flake 📄log
✔️ 265909a #14 2024-03-26 00:03:17 ~1 min nix-flake 📄log
✔️ 54edf8b #15 2024-03-26 00:28:21 ~1 min nix-flake 📄log
✔️ 5a76773 #16 2024-03-26 06:18:36 ~1 min nix-flake 📄log
✔️ dc4ed27 #17 2024-03-27 10:15:43 ~1 min nix-flake 📄log
✔️ 4266513 #18 2024-03-28 02:38:51 ~1 min nix-flake 📄log
✔️ b9e700f #19 2024-04-01 10:45:46 ~1 min nix-flake 📄log
✔️ c15768b #20 2024-04-01 10:49:07 ~3 min nix-flake 📄log
✔️ 2c66e91 #21 2024-04-02 07:12:42 ~1 min nix-flake 📄log
✔️ 4e89a8f #22 2024-04-02 07:19:49 ~1 min nix-flake 📄log
✔️ def82f2 #23 2024-04-02 07:24:52 ~1 min nix-flake 📄log
✔️ 3b571ba #24 2024-04-02 07:26:49 ~1 min nix-flake 📄log
✔️ c80456d #25 2024-04-02 11:14:07 ~2 min nix-flake 📄log
✔️ 873caa8 #26 2024-04-03 11:10:49 ~1 min nix-flake 📄log
✔️ 502ff9d #27 2024-04-04 05:55:26 ~1 min nix-flake 📄log
✔️ 72be483 #28 2024-04-04 23:48:34 ~1 min nix-flake 📄log
✔️ 9693bed #29 2024-04-08 02:26:33 ~1 min nix-flake 📄log
✔️ b6fc2cb #30 2024-04-08 02:32:49 ~1 min nix-flake 📄log
✔️ 5de4bfa #31 2024-04-08 11:06:39 ~6 min nix-flake 📄log
✔️ 8efec1e #32 2024-04-08 11:11:17 ~1 min nix-flake 📄log
✔️ 1cd52ee #33 2024-04-10 12:49:34 ~7 min nix-flake 📄log
✔️ 891d9a9 #34 2024-04-10 12:53:53 ~4 min nix-flake 📄log
✔️ d27c0ed #35 2024-04-10 13:01:57 ~1 min nix-flake 📄log
✔️ e1210c7 #36 2024-04-10 13:15:42 ~2 min nix-flake 📄log
✔️ b562771 #37 2024-04-11 02:12:40 ~1 min nix-flake 📄log
✔️ 8d7e31b #38 2024-04-15 06:29:43 ~1 min nix-flake 📄log
✔️ 99e3f33 #39 2024-04-22 02:52:46 ~5 min nix-flake 📄log
✔️ adaace3 #40 2024-04-22 02:55:47 ~2 min nix-flake 📄log
✔️ 590e4ff #41 2024-04-23 02:36:14 ~2 min nix-flake 📄log
✔️ 7404bb4 #42 2024-05-02 02:29:00 ~2 min nix-flake 📄log
✔️ 12a8923 #43 2024-05-02 07:26:37 ~2 min nix-flake 📄log
✔️ 73a90e0 #44 2024-05-02 09:33:43 ~2 min nix-flake 📄log
✔️ fa01d17 #45 2024-05-02 11:05:39 ~2 min nix-flake 📄log
✔️ 093ea36 #46 2024-05-02 11:21:38 ~2 min nix-flake 📄log
✔️ dd960ef #47 2024-05-03 06:44:41 ~2 min nix-flake 📄log
✔️ 2d0af55 #48 2024-05-03 10:02:35 ~2 min nix-flake 📄log
✔️ 77f6917 #49 2024-05-03 11:02:20 ~2 min nix-flake 📄log
✔️ 6a72e6d #50 2024-05-03 11:14:15 ~2 min nix-flake 📄log
✖️ 1afb587 #51 2024-05-03 11:27:49 ~14 sec nix-flake 📄log
✖️ 3a1a7e6 #52 2024-05-03 11:30:27 ~16 sec nix-flake 📄log
✔️ 0741101 #53 2024-05-03 11:41:00 ~2 min nix-flake 📄log
✔️ 8968e59 #54 2024-05-03 11:55:11 ~2 min nix-flake 📄log
✔️ a047dd8 #55 2024-05-03 11:57:54 ~2 min nix-flake 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 01803de #56 2024-05-03 12:01:21 ~2 min nix-flake 📄log
✔️ a369fa4 #57 2024-05-04 01:22:36 ~2 min nix-flake 📄log

@romanzac romanzac marked this pull request as ready for review March 25, 2024 01:00
waku/v2/node/wakunode2_test.go Show resolved Hide resolved
waku/v2/node/wakunode2_test.go Outdated Show resolved Hide resolved
waku/v2/node/wakunode2_test.go Show resolved Hide resolved
@romanzac romanzac marked this pull request as draft March 26, 2024 08:02
romanzac and others added 24 commits May 4, 2024 09:17
- subscribe to both relays with a breath time
- clarify intent for using different shardIDs
- use clusterID other than 0 - to test unsubscribed topic
- TestWakuLightPushWithStaticSharding
@romanzac romanzac merged commit a453c02 into master May 4, 2024
11 of 12 checks passed
@romanzac romanzac deleted the chore-sharding-tests-update branch May 4, 2024 01:26
@chaitanyaprem
Copy link
Collaborator

chaitanyaprem commented May 4, 2024

Why is this branch showing soo much diff with master?
Let us not merge when diff is like this..it means the git tree is probably messed up.
Ideally, we should rebase from master, pass CI and then merge it.

Hopefully nothing got messed up.

@romanzac
Copy link
Collaborator Author

romanzac commented May 4, 2024

Why is this branch showing soo much diff with master? Let us not merge when diff is like this..it means the git tree is probably messed up. Ideally, we should rebase from master, pass CI and then merge it.

Hopefully nothing got messed up.

This is what I have done. Rebased from master, passed CI and then merged. No worries.

@chaitanyaprem
Copy link
Collaborator

Why is this branch showing soo much diff with master? Let us not merge when diff is like this..it means the git tree is probably messed up. Ideally, we should rebase from master, pass CI and then merge it.

Hopefully nothing got messed up.

This is what I have done. Rebased from master, passed CI and then merged. No worries.

Oh, interesting...in that case the diff shouldn't be this big right

@romanzac
Copy link
Collaborator Author

romanzac commented May 4, 2024

Why is this branch showing soo much diff with master? Let us not merge when diff is like this..it means the git tree is probably messed up. Ideally, we should rebase from master, pass CI and then merge it.
Hopefully nothing got messed up.

This is what I have done. Rebased from master, passed CI and then merged. No worries.

Oh, interesting...in that case the diff shouldn't be this big right

I remember I've done merge at some point too. What is interesting, the final merge into master has normal (expected) diff a453c02 My IDE also shows the same.

image

Should we be fine ? @chaitanyaprem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants