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_: bump go-waku with filter loop fix #5909

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

chaitanyaprem
Copy link
Contributor

@chaitanyaprem chaitanyaprem commented Oct 4, 2024

updates go-waku with many fixes and improvements along with filter loop fix.

this PR is to be used for testing and identifying issues being noticed in status-mobile tests wrt peers not connected apart from store nodes.

all underlying commits have been dogfooded separately. @richard-ramos can confirm the same.

Important changes:

  • added check to prevent a filter loop from flooding logs

Closes #

@status-im-auto
Copy link
Member

status-im-auto commented Oct 4, 2024

Jenkins Builds

Click to see older builds (65)
Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ d568d84 #1 2024-10-04 06:15:45 ~57 sec tests 📄log
d568d84 #1 2024-10-04 06:16:04 ~1 min ios 📄log
d568d84 #1 2024-10-04 06:16:21 ~1 min android 📄log
d568d84 #1 2024-10-04 06:16:34 ~1 min linux 📄log
✖️ d568d84 #1 2024-10-04 06:16:43 ~1 min tests-rpc 📄log
7cdec8b #2 2024-10-04 06:32:50 ~55 sec android 📄log
7cdec8b #2 2024-10-04 06:33:02 ~57 sec linux 📄log
✖️ 7cdec8b #2 2024-10-04 06:33:11 ~1 min tests-rpc 📄log
7cdec8b #2 2024-10-04 06:33:12 ~1 min ios 📄log
✖️ 7cdec8b #2 2024-10-04 06:33:22 ~1 min tests 📄log
c88980c #3 2024-10-04 06:54:36 ~37 sec android 📄log
c88980c #3 2024-10-04 06:54:50 ~34 sec linux 📄log
c88980c #3 2024-10-04 06:54:50 ~41 sec ios 📄log
✖️ c88980c #3 2024-10-04 06:54:53 ~48 sec tests-rpc 📄log
✖️ c88980c #3 2024-10-04 06:55:03 ~57 sec tests 📄log
552c359 #4 2024-10-04 06:57:29 ~38 sec android 📄log
552c359 #4 2024-10-04 06:57:36 ~42 sec ios 📄log
552c359 #4 2024-10-04 06:57:41 ~44 sec linux 📄log
✖️ 552c359 #4 2024-10-04 06:57:59 ~1 min tests-rpc 📄log
552c359 #5 2024-10-04 07:23:22 ~40 sec android 📄log
✔️ 552c359 #4 2024-10-04 07:29:59 ~33 min tests 📄log
c3e5c34 #6 2024-10-04 07:30:25 ~34 sec android 📄log
c3e5c34 #5 2024-10-04 07:30:41 ~46 sec linux 📄log
c3e5c34 #5 2024-10-04 07:30:50 ~53 sec ios 📄log
✖️ c3e5c34 #5 2024-10-04 07:31:08 ~1 min tests-rpc 📄log
✔️ c3e5c34 #5 2024-10-04 08:01:21 ~31 min tests 📄log
ae9102f #7 2024-10-04 07:37:45 ~33 sec android 📄log
ae9102f #6 2024-10-04 07:37:58 ~44 sec ios 📄log
ae9102f #6 2024-10-04 07:37:58 ~42 sec linux 📄log
✖️ ae9102f #6 2024-10-04 07:38:19 ~1 min tests-rpc 📄log
0fc4d70 #8 2024-10-04 07:49:48 ~37 sec android 📄log
0fc4d70 #7 2024-10-04 07:49:58 ~44 sec ios 📄log
0fc4d70 #7 2024-10-04 07:49:59 ~44 sec linux 📄log
✖️ 0fc4d70 #7 2024-10-04 07:50:21 ~1 min tests-rpc 📄log
✖️ 0fc4d70 #6 2024-10-04 08:31:48 ~30 min tests 📄log
d91f1d0 #9 2024-10-04 13:55:11 ~38 sec android 📄log
d91f1d0 #8 2024-10-04 13:55:26 ~53 sec ios 📄log
d91f1d0 #8 2024-10-04 13:55:48 ~1 min linux 📄log
✖️ d91f1d0 #8 2024-10-04 13:56:04 ~1 min tests-rpc 📄log
✔️ d91f1d0 #7 2024-10-04 14:27:48 ~33 min tests 📄log
0d876d1 #10 2024-10-04 14:30:13 ~40 sec android 📄log
0d876d1 #9 2024-10-04 14:30:20 ~45 sec ios 📄log
0d876d1 #9 2024-10-04 14:30:22 ~44 sec linux 📄log
✖️ 0d876d1 #9 2024-10-04 14:30:43 ~1 min tests-rpc 📄log
✖️ 0d876d1 #8 2024-10-04 14:31:04 ~1 min tests 📄log
✔️ 9e2f128 #10 2024-10-04 14:43:02 ~3 min tests-rpc 📄log
✔️ 9e2f128 #10 2024-10-04 14:43:55 ~4 min ios 📦zip
✔️ 9e2f128 #11 2024-10-04 14:44:08 ~4 min android 📦aar
✔️ 9e2f128 #10 2024-10-04 14:44:15 ~4 min linux 📦zip
✔️ 9e2f128 #9 2024-10-04 15:11:01 ~31 min tests 📄log
✔️ b30d5ba #12 2024-10-07 09:53:05 ~2 min android 📦aar
✔️ b30d5ba #11 2024-10-07 09:54:49 ~3 min ios 📦zip
✔️ b30d5ba #11 2024-10-07 09:54:56 ~3 min tests-rpc 📄log
✔️ b30d5ba #11 2024-10-07 09:55:03 ~3 min linux 📦zip
✔️ b30d5ba #10 2024-10-07 10:23:30 ~32 min tests 📄log
✖️ 898a5e2 #11 2024-10-08 08:18:36 ~1 min tests 📄log
✔️ 898a5e2 #13 2024-10-08 08:19:16 ~2 min android 📦aar
✔️ 898a5e2 #12 2024-10-08 08:20:22 ~3 min linux 📦zip
✔️ 898a5e2 #12 2024-10-08 08:20:38 ~3 min tests-rpc 📄log
✔️ 898a5e2 #12 2024-10-08 08:20:58 ~3 min ios 📦zip
✔️ e3f02ee #14 2024-10-09 15:23:01 ~1 min android 📦aar
✔️ e3f02ee #13 2024-10-09 15:23:34 ~2 min linux 📦zip
✔️ e3f02ee #13 2024-10-09 15:25:29 ~4 min tests-rpc 📄log
✔️ e3f02ee #13 2024-10-09 15:25:29 ~4 min ios 📦zip
✔️ e3f02ee #12 2024-10-09 15:54:58 ~33 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ cba9f25 #15 2024-10-09 15:26:04 ~2 min android 📦aar
✔️ cba9f25 #14 2024-10-09 15:26:29 ~2 min linux 📦zip
✔️ 2ff0766 #16 2024-10-09 15:28:04 ~1 min android 📦aar
✔️ 2ff0766 #15 2024-10-09 15:29:03 ~2 min linux 📦zip
✔️ 2ff0766 #14 2024-10-09 15:29:31 ~3 min ios 📦zip
✔️ 2ff0766 #14 2024-10-09 15:31:42 ~6 min tests-rpc 📄log
✔️ 2ff0766 #13 2024-10-09 16:28:09 ~33 min tests 📄log

@chaitanyaprem chaitanyaprem force-pushed the fix/filter-loop branch 3 times, most recently from c88980c to 552c359 Compare October 4, 2024 06:56
Copy link

codecov bot commented Oct 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.45%. Comparing base (ba37c32) to head (2ff0766).
Report is 18 commits behind head on develop.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5909      +/-   ##
===========================================
- Coverage    47.79%   47.45%   -0.34%     
===========================================
  Files          834      834              
  Lines       137749   137818      +69     
===========================================
- Hits         65843    65408     -435     
- Misses       64308    64635     +327     
- Partials      7598     7775     +177     
Flag Coverage Δ
functional 10.41% <ø> (+0.19%) ⬆️
unit 46.76% <ø> (-0.33%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 76 files with indirect coverage changes

@richard-ramos
Copy link
Member

Let's see if it works now after rebasing the .gitignore change!

@siddarthkay
Copy link
Contributor

@richard-ramos : you might have forgotten to push vendor change 😉

@richard-ramos
Copy link
Member

richard-ramos commented Oct 4, 2024

@siddarthkay: you might have forgotten to push vendor change 😉

le sigh...

@chaitanyaprem chaitanyaprem force-pushed the fix/filter-loop branch 2 times, most recently from cba9f25 to 2ff0766 Compare October 9, 2024 15:25
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.

5 participants