-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20430/
-----------------------------------------------------------

(Updated April 16, 2014, 10:17 p.m.)


Review request for mesos, Ben Mahler, Jie Yu, and Vinod Kone.


Bugs: MESOS-976
    https://issues.apache.org/jira/browse/MESOS-976


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

Reply via email to