mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinod Kone" <vinodk...@gmail.com>
Subject Review Request: Non-disruptive Slave Restart with Recovery!
Date Fri, 23 Mar 2012 01:16:28 GMT

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

Review request for mesos, Benjamin Hindman and John Sirois.


Summary
-------

Sorry for the huge  CL!

Slave restarts now supports recovery!
--> Non-disruptive restart means running tasks are not lost
--> Re-connects to live executors
--> Checkpoints and reliably sends status updates
--> Ability to kill executors if the slave upgrade is incompatible with running executors


This addresses bug mesos-110.
    https://issues.apache.org/jira/browse/mesos-110


Diffs
-----

  src/Makefile.am 090c07a 
  src/common/hashset.hpp 1feb610 
  src/common/utils.hpp 015fef8 
  src/exec/exec.cpp 53f7e54 
  src/launcher/launcher.cpp 98a4847 
  src/local/local.hpp 55f9eaf 
  src/local/local.cpp affe432 
  src/master/master.cpp 6ae8aef 
  src/messages/messages.proto 7f9cffe 
  src/sched/sched.cpp 43d9717 
  src/slave/constants.hpp f0c8679 
  src/slave/isolation_module.hpp c896908 
  src/slave/lxc_isolation_module.hpp b7beefe 
  src/slave/lxc_isolation_module.cpp 8c25dd4 
  src/slave/main.cpp ac780c4 
  src/slave/process_based_isolation_module.hpp f6f9554 
  src/slave/process_based_isolation_module.cpp e0f3ee8 
  src/slave/slave.hpp b7bc45a 
  src/slave/slave.cpp 9332caa 
  src/tests/fault_tolerance_tests.cpp 130218d 
  src/tests/slave_restart_tests.cpp PRE-CREATION 
  src/tests/utils.hpp 8c038ce 

Diff: https://reviews.apache.org/r/4462/diff


Testing
-------

make check.

Note that only the new test in tests/slave_restart_tests.cpp  engages in recovery!

Recovery is disabled for old tests (though they still checkpoint relevant info!)


Thanks,

Vinod


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message