Results 29 comments of Roman Zajic

Thanks for trying out. I was not able to receive message even after modifications you have proposed. What I have tried myself is to check if discovery works well before...

When I zoom out a bit, I admit that two nodes working at the same machine and having bursts in topic subscriptions is not very standard situation. If you @richard-ramos...

I've changed the test to subscribe to just one topic at each node and message still won't get through. Please checkout https://github.com/waku-org/go-waku/pull/1060/commits/8d7e31bde9eaca3eca389ebcf15263b072781960

Issue https://github.com/waku-org/go-waku/issues/1077 related to this PR

I've reverted "TestStaticShardingLimits" to its desired form and marked it as flaky. This way we can unblock this PR and start a collection of flaky tests. Flaky test could take...

> @romanzac CI seems to be failing, we will have to fix that. > > One more comment and once both of these are fixed, we can merge . CI...

> Can be merged once CI is fixed. I've contacted Richard few hours ago. I'll try few things before he responds. CI will fail for mac os on other open...

> 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...

> > > 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....

Issue is related to #980