mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nishant Suneja" <nishantsun...@gmail.com>
Subject Re: Review Request 29720: Bug fix: Added an executor container launch timeout timer
Date Sat, 10 Jan 2015 16:21:16 GMT

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

(Updated Jan. 10, 2015, 4:21 p.m.)


Review request for mesos and Timothy Chen.


Changes
-------

Changes to the dependent patch.


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


Repository: mesos-git


Description
-------

A new executor launch timeout timer has been added. This timer gates the time in which a successful
executor container launch should happen. The executor registration timer starts after the
successful container launch.


Diffs (updated)
-----

  src/slave/constants.hpp 761cfafb3b1b342af4d1dbdb2dec39a45dd62794 
  src/slave/constants.cpp 83d9fc1cd385722deb31883c17c824fd0e1de8a6 
  src/slave/flags.hpp f1b8dfbb7391167b67a9498561742aa9ab9089a6 
  src/slave/slave.hpp 70bd8c1fde4ea09fa54c76aa93424a1adb0309f6 
  src/slave/slave.cpp b234f5359a91a293b93f97f035b08ef0a4c9b20d 
  src/tests/slave_tests.cpp a85a0282947278ce6e02e38093f8b88af4cb612a 

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


Testing
-------

make check succeeds.
Added the unit test : SlaveTest:: ExecutorLaunchTimeoutTrigger.
Added the unit test : SlaveTest:: ExecutorNoLaunchTimeoutTrigger.


Thanks,

Nishant Suneja


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