----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20430/ -----------------------------------------------------------
Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone. Repository: mesos-git Description ------- Many tests (in particular slave recovery tests) leave containers running at the end of a test. This patch extends Cluster::Slaves::shutdown to query and destroy the containers for any slaves in the cluster, ensuring isolators can properly clean up everything. Note: This is not done for slaves that are stopped (so we can actually do slave recovery tests!) so any test that completes without slaves but with containers still running must do this step manually - see the two modified tests. Diffs ----- src/tests/cluster.hpp 8479fe370d5a64cb2827b69b6f0c626b9d156d66 src/tests/slave_recovery_tests.cpp 72b6d42f0895f10fed041a5a1351e79b003e2d1b Diff: https://reviews.apache.org/r/20430/diff/ Testing ------- Thanks, Ian Downes
