-
Notifications
You must be signed in to change notification settings - Fork 1
Description
Describe the bug
During each experiment execution, a network topology is defined, and further enforced by telling the nodes via RPC calls to which other nodes they have to connect.
After implementing a mitigation for the bug #48 in the PR #46 , a regression was introduced, making impossible for some nodes to connect as they're told to do.
The problematic behaviour was caused by the fact that two networks were created (one as a workaround for the InstaJustify problem, and another one for the experiment itself), but sharing a subset of nodes. These nodes cached some information about peers and tried to connect on their own without complying with the intended topology.
To Reproduce
Checkout the code from the PR #46, and run an experiment, with high probability it will stall before starting the whole network.
Expected behavior
A clear and concise description of what you expected to happen.
Environment
It doesn't matter.
Additional context
Already fixed in the PR #47 (still to be merged).