----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10233/ -----------------------------------------------------------
(Updated April 9, 2013, 11:14 p.m.) Review request for mesos, Benjamin Hindman and Ben Mahler. Changes ------- fixed tests to use the latest abstractions. Description ------- See summary. The main crux is two fold 1) Shutdown a slave after all executors have terminated 2) Delete the "latest" symlink under /path/to/meta/slaves/ to make sure a shutdown slave comes up as new slave when restarted. I had to refactor cleanup(), because turns out there are quite a few edge cases to guard against. Diffs (updated) ----- src/slave/process_isolator.cpp 210ea10ad97e08c7a303249da97e70b438dfe11d src/slave/slave.hpp 2529bf500a3265b10ad4cddde10c2d62a6cdb4a0 src/slave/slave.cpp 325231458a6883019436e7cc5a37f85f0f5735fa src/slave/state.cpp e5c32257978d8407535e05ed73f8a50bdc2f651d src/tests/slave_recovery_tests.cpp d0ff9b73e06e89a5409f038be2766333e0a0689e Diff: https://reviews.apache.org/r/10233/diff/ Testing ------- make check sudo GLOG_v=1 ./bin/mesos-tests.sh --gtest_filter="*ShutdownSlave*" --verbose --gtest_repeat=100 --gtest_break_on_failure Thanks, Vinod Kone
