stellar-deprecated / stellar_core_commander

A system of creating isolated Stellar test networks into which you can play transactions and record results.
Apache License 2.0
19 stars 27 forks source link

Avoid mutating quorum list, which may be reused between processes. #53

Closed graydon closed 9 years ago

graydon commented 9 years ago

Turns out when we use a single quorum-list variable in a recipe and thread it through multiple processes, it accumulates a new member for each place it's used. This causes topologies with core/listener splits to have listeners that treat their neighbours as quorum-members (and history members), and the complex-topology test fails.

Not mutating the quorum-list fixes it.

nullstyle commented 9 years ago

:+1: