signalfx / maestro-ng

Orchestration of Docker-based, multi-host environments
https://signalfx.com
Apache License 2.0
685 stars 83 forks source link

Conductor: fix _ordered_containers() to use fresh result list fo… #208

Closed wlopata-sfx closed 4 years ago

wlopata-sfx commented 4 years ago

I would like to call _ordered_containers() method multiple times in my code, but each invocation of that method appends new results to a list with results accumulated from all previous invocations.

The problem is underneath in _order_dependencies() method, which declares mutable default empty list argument.