-
Notifications
You must be signed in to change notification settings - Fork 42
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
bug: same cluster, different shard: nodes connect
tests fail if using a shard different than 0
#1848
Comments
Huh, this is interesting.. Perhaps something @waku-org/nwaku-developers might have context on especially since these tests are only failing on |
I think this is expected after the new sharded peer management waku-org/nwaku#2332 |
The sharded peer manager is disabled by default but even if enabled it should still work. One thing I notice is that the js code allow autosharding with cluster != 1 which may cause problems. cluster 0 == named sharding only Also if 2 nodes don't share a shard then they should not connect per the Metadata protocol logic. |
Yes, I saw that if they share at least one shard than the test works for any cluster and connection works |
@SionoiS if two nodes are on the same cluster, but different shards, they should connect right? |
No the metadata protocol logic prevents it. |
Ah, I was under the belief that two nodes only need to be on the same cluster and not necessarily on the same shard. Gotcha! |
As long as 2 nodes share at least one shard they will connect (not disconnect). |
This is a bug report
Problem
Those 2 tests(1 and 2) fail if instead of cluster 0 we use a different one (ex 2)
This happens with
harbor.status.im/wakuorg/nwaku:latest
and doesn't reproduce with nwaku v.0.24.0Log
The text was updated successfully, but these errors were encountered: