weaveworks / scope

Monitoring, visualisation & management for Docker & Kubernetes
https://www.weave.works/oss/scope/
Apache License 2.0
5.84k stars 708 forks source link

fix: merging active controls on host #3859

Closed bboreham closed 3 years ago

bboreham commented 3 years ago

Need to do the special-case merge before n.Latest.Merge(), otherwise it picks just one set of controls.

Also modified the test to call Merge() where the problem happened.