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: switch wakuv2 fleet to waku #1910

Merged
merged 5 commits into from
Apr 7, 2024
Merged

Conversation

yakimant
Copy link
Member

@yakimant yakimant commented Mar 12, 2024

See status-im/infra-nim-waku#91 for details

Mostly done automatically. Some parts manually.

I've added some TODO comments to the addresses, which I don't know how to fix.

@yakimant yakimant requested a review from a team as a code owner March 12, 2024 16:33
@yakimant yakimant requested review from fryorcraken and danisharora099 and removed request for a team March 12, 2024 16:34
Comment on lines 317 to 318
# TODO: wakuv2 is being decomissioned. What to do with such addresses?
const wsMultiaddr = multiaddr(
"/dns4/node-01.do-ams3.wakuv2.prod.statusim.net/tcp/8000/wss"
);
Copy link
Collaborator

@danisharora099 danisharora099 Mar 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for this @yakimant!

We'll have to change the tests to account for the new addr(s). I'm happy to add commits to this PR to refactor these

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps this warrants a larger discussion:
Considering these fleets now run The Waku Network, we should refactor not only the tests, but also the docs, to make this the new default
cc @waku-org/js-waku-developers

Copy link
Member Author

@yakimant yakimant Mar 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A, it's just a test psec. Let me try.

For example "/dns4/node-01.do-ams3.wakuv2.prod.statusim.net/tcp/8000/wss"
will be "/dns4/node-01.do-ams3.waku.sandbox.statusim.net/tcp/8000/wss"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danisharora099, done. Acn you please review again?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danisharora099, done. Acn you please review again?

The CI checks seem to be failing. Can you take another look? Let me know if you need any help!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danisharora099, can you please have a look at the 2 issues below.
Looks like I need to do smth with @waku/dns-discovery and protobuf.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yakimant yakimant requested a review from danisharora099 March 19, 2024 14:27
@yakimant
Copy link
Member Author

One issues from CI:

 src/utils/discovery.ts:17:23 - error TS7053: Element implicitly has an 'any' type because expression of type '"SANDBOX"' can't be used to index type '{ TEST: string; PROD: string; }'.
  Property 'SANDBOX' does not exist on type '{ TEST: string; PROD: string; }'.

17     wakuDnsDiscovery([enrTree["SANDBOX"]], DEFAULT_NODE_REQUIREMENTS),

Which are taken from

import { enrTree, wakuDnsDiscovery } from "@waku/dns-discovery";

Which is taken from
https://www.npmjs.com/package/@waku/dns-discovery

Which is taken from
https://github.com/waku-org/js-waku/

I wonder, how can we update the dns-discovery (and maybe other packages) with new names?

@yakimant yakimant force-pushed the switch_wakuv2_to_waku branch from 2e0e16a to c5786be Compare March 22, 2024 15:59
@yakimant
Copy link
Member Author

Second issue is Check all protobuf code was committed:

Run res=$(git status --short --ignore-submodules)
  res=$(git status --short --ignore-submodules)
  echo -n "'$res'" # For debug purposes
  [ $(echo -n "$res"|wc -l) -eq 0 ]
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    NODE_JS: 18
    npm_config_cache: /home/runner/.npm
' M packages/core/src/lib/predefined_bootstrap_nodes.ts
 M packages/discovery/src/dns/constants.ts
 M packages/enr/src/enr.spec.ts
 M packages/tests/tests/dns-peer-discovery.spec.ts'
Error: Process completed with exit code 1.

@danisharora099 danisharora099 force-pushed the switch_wakuv2_to_waku branch from c5786be to f09558e Compare April 7, 2024 10:14
Copy link

github-actions bot commented Apr 7, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
Waku node 180.85 KB (+0.15% 🔺) 3.7 s (+0.15% 🔺) 12.5 s (-4.38% 🔽) 16.1 s
Waku Simple Light Node 180.73 KB (-0.02% 🔽) 3.7 s (-0.02% 🔽) 14.4 s (+31.14% 🔺) 18 s
ECIES encryption 23.11 KB (0%) 463 ms (0%) 3.8 s (-1.86% 🔽) 4.2 s
Symmetric encryption 22.58 KB (0%) 452 ms (0%) 6.8 s (+164.66% 🔺) 7.2 s
DNS discovery 72.4 KB (0%) 1.5 s (0%) 8.8 s (-22.39% 🔽) 10.2 s
Peer Exchange discovery 73.97 KB (0%) 1.5 s (0%) 13.7 s (+39.15% 🔺) 15.2 s
Local Peer Cache Discovery 67.71 KB (0%) 1.4 s (0%) 7.6 s (-13.72% 🔽) 8.9 s
Privacy preserving protocols 38.75 KB (0%) 775 ms (0%) 7.3 s (+27.68% 🔺) 8 s
Waku Filter 20.17 KB (0%) 404 ms (0%) 3.2 s (+6.58% 🔺) 3.6 s
Waku LightPush 110.12 KB (0%) 2.3 s (0%) 10.3 s (+6.6% 🔺) 12.6 s
History retrieval protocols 110.62 KB (0%) 2.3 s (0%) 10.8 s (-10.13% 🔽) 13 s
Deterministic Message Hashing 4.83 KB (0%) 97 ms (0%) 654 ms (-55% 🔽) 751 ms

@danisharora099 danisharora099 force-pushed the switch_wakuv2_to_waku branch from 34ebf8b to f638883 Compare April 7, 2024 11:22
@danisharora099 danisharora099 force-pushed the switch_wakuv2_to_waku branch from a42032a to 048f6c7 Compare April 7, 2024 12:04
@danisharora099 danisharora099 merged commit 9f2c0ed into master Apr 7, 2024
8 of 10 checks passed
@danisharora099 danisharora099 deleted the switch_wakuv2_to_waku branch April 7, 2024 12:21
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