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

[Epic] Targeted dogfooding for Status Communities #97

Closed
3 tasks done
fryorcraken opened this issue Oct 20, 2023 · 18 comments
Closed
3 tasks done

[Epic] Targeted dogfooding for Status Communities #97

fryorcraken opened this issue Oct 20, 2023 · 18 comments
Assignees
Labels
E:Targeted Status Communities dogfooding See https://github.com/waku-org/pm/issues/97 for details Epic Tracks a sub-team Epic.

Comments

@fryorcraken
Copy link
Contributor

fryorcraken commented Oct 20, 2023

Epic label: E:Targeted Status Communities dogfooding

Summary

Dogfooding of delivery for Status Communities (static sharding, PostgreSQL, opt-in message signing).

Acceptance Criteria

Tasks

@fryorcraken fryorcraken added the Epic Tracks a sub-team Epic. label Oct 20, 2023
@fryorcraken fryorcraken added this to Waku Oct 20, 2023
@fryorcraken fryorcraken added the E:Targeted Status Communities dogfooding See https://github.com/waku-org/pm/issues/97 for details label Oct 20, 2023
@fryorcraken
Copy link
Contributor Author

@richard-ramos I think we can close waku-org/nwaku#1914 so let's use this epic to track the work you are doing to test static sharding with Status Desktop.

@richard-ramos
Copy link
Member

richard-ramos commented Oct 20, 2023

Weekly Update

  • achieved: hardcoded bootnodes ENRs in addition to DNS Discovery URLs as a way to overcome nameserver issues. Use a static shard instead of the default pubsub topic. Update tool to crawl and discover nodes via discv5.
  • next: fix if necessary strange behavior with discv5 when ENRs in DNS discovery URL do not contain shards. Document steps for dogfooding.

@fryorcraken
Copy link
Contributor Author

@richard-ramos shared instructions on how to run Status Desktop with static sharding: https://discord.com/channels/1110799176264056863/1166125031793901589/1166125169480306708 (Waku Discord - private channel)

@richard-ramos
Copy link
Member

Weekly Update

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: fix in status-go to use correct unprotected pubsub topic for community point of contacts, added pubsub topic bridge feature to go-waku, stop filtering out bootnodes on discovery, minimize noise on logs when selecting peers and not having peers available.
  • next: deploy bridge between default pubsub topic and shard 32

@richard-ramos
Copy link
Member

richard-ramos commented Nov 12, 2023

Weekly Update

  • achieved: deployed bridge between default pubsub topic and shard 32, added UseShardAsDefaultTopic node config option to indicate whether the client should use a shard or the default waku topic. Merged open status-go PRs related to sharding. Initial connection to peers + dns-discovery no longer blocks the login. Updated open status-desktop PR for dogfooding to take into account status-go latest sharding related changes.
  • next: get feat: shards.test fleet status-im/status-desktop#12344 merged.

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: logout / login freeze, fix request on correct pubsub topic, and add missing shard information on community invite
  • next: dogfooding

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: fix pubsub topic used for store queries, logic for finding free port when initializing torrent service, add back changes related to default shard that were reverted before, store clusterID in app database, refactor mailserver cycle to not require having an active connection to a store node
  • next: dogfooding

@chaitanyaprem
Copy link

chaitanyaprem commented Nov 27, 2023

Weekly Update

  • achieved: fix issue of using default clusterID as 16 for shards fleet, dogfooding of status-desktop with shards fleet, debug issues in connection to fleet. See dogfooding report
  • next: continue dogfooding

@chaitanyaprem
Copy link

chaitanyaprem commented Nov 28, 2023

Interim report of dogfooding shards-fleet using status desktop by Waku Team

Summary & Observations

  • 5 CCs have participated in dogfooding the sharding fleet using various desktop builds (mac-aarch64, windows, linux)
  • 2 test Communities were created and assigned shards where participants have joined and sent messages and verified history retrieval. 1:1 messages were tested by various participants.
  • Peer connectivity was verified to ensure healthy peer connections for the shard
  • Fetching message history for community before a user joined
  • Identified and fixed issues in status-go to work with sharding fleet (Refer below)
  • Raised an issue wrt fleet config that is causing desktop not to connect with different store nodes Update dns discovery url for shards.test status-im/infra-status#23
  • Continue dogfooding once fleet config isssue is addressed

Issues observed/Raised in status desktop

UI related

Issues resolved in status-go

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: troubleshoot / fix issues reported by QA on sharding PR
  • next: continue dogfooding / fixing issues

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: fix peer manager to take into account the ENR seq to determine if a peer is new or not and fix incorrect number of connected peers per topic. Register lightpush protocol correctly in go-waku. Drop pubsub topic bridging. Use shards.test fleet as default (in status-go and desktop)
  • next: continue dogfooding / fixing issues

@chaitanyaprem
Copy link

Weekly Update

  • achieved: status dogfooding, raise issues and optimizations needed wrt light protocols usage in status-go and improvements to be done in nwaku

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: fix panic on logout, publish messages async, fix loading message history for communities, fix invalid bootnodes being used on status desktop.
  • next: continue dogfooding / fixing issues

@chaitanyaprem
Copy link

Weekly Update

  • achieved: analyze and repro filter delay issue reported in mobile e2e automated test by dogfooding
  • next: continue status-mobile dogfooding with desktop / fixing issues

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: attempt to identify root cause of filter delay issue, work on upgrading status-go to use Go 1.21 (to use latest go-libp2p), use latest go-waku version and identify missing commits between release and develop branch
  • next: continue dogfooding / fixing issues

@chaitanyaprem
Copy link

Weekly Update

  • achieved: status-mobile dogfooding, debug and attempt to identify root cause of filter delay issue
  • next: continue dogfooding / fixing issues

@fryorcraken
Copy link
Contributor Author

Waku dogfooding has concluded and Status QA has taken over testing.
Waku team is now providing support to Status testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E:Targeted Status Communities dogfooding See https://github.com/waku-org/pm/issues/97 for details Epic Tracks a sub-team Epic.
Projects
Status: Done
Development

No branches or pull requests

3 participants