You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 15, 2020. It is now read-only.
currently TestGossipPropigation is disabled because in some instances the propigation doesn't fully complete, i.e. it doesn't get to every single node in the ring. We need to figure out why, and test lots of other network connection topologies.
The text was updated successfully, but these errors were encountered:
The issues in the gossip propagation tests seem to be due to deeper issues is the DHT model. The failures to propigate don't come from the gossip as much as coming from failures in initial sending to the DHT that happen. For example it seems like in clutter there are some commits that happen during genesis that aren't always getting put to the DHT (perhaps because at the time of genesis those nodes don't have anything in their routing table?). So this ticket is really about adding more tests to prove this, or wait until we update the DHT model to take into account the startup phase.
After much analysis, we realize that the issue is really that gossip needs to not be about entries but about nodes. Thus this ticket is thus now superceeded by #582
currently TestGossipPropigation is disabled because in some instances the propigation doesn't fully complete, i.e. it doesn't get to every single node in the ring. We need to figure out why, and test lots of other network connection topologies.
The text was updated successfully, but these errors were encountered: