----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29556/#review66753 -----------------------------------------------------------
Patch looks great! Reviews applied: [29556] All tests passed. - Mesos ReviewBot On Jan. 5, 2015, 11:17 p.m., Connor Doyle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29556/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2015, 11:17 p.m.) > > > Review request for mesos, Adam B and Timothy Chen. > > > Bugs: MESOS-2206 > https://issues.apache.org/jira/browse/MESOS-2206 > > > Repository: mesos-git > > > Description > ------- > > Task statuses sent to the framework scheduler from the master do not include > a value for the `healthy` field. The result is that the scheduler is left > with incomplete knowledge about the health state of running tasks > indefinitely, until by chance the executor sends a new status update. > > - Update the createTaskStatus auxilliary function in protobuf utils to take > an optional health flag. > - Pass known health to createTaskStatus during implicit and explicit task > reconciliation. > > > Diffs > ----- > > src/common/protobuf_utils.hpp e42aaa5 > src/common/protobuf_utils.cpp 8ab5cdd > src/master/master.cpp d6651e2 > > Diff: https://reviews.apache.org/r/29556/diff/ > > > Testing > ------- > > - make > - make check > > Manual testing using a development branch of the Marathon[1] framework: > > - Launched two tasks with health checks. > - Waited for both tasks to enter the running state. > - Waited for both tasks to report healthy. > - Shut down the scheduler. > - Re-started (re-registered) the scheduler. > - Observed indeterminate health state for both tasks as reported by the > framework. > - Waited for the framework to initiate task reconciliation. > - Observed a TaskStatus with the healthy field set to `true` for both tasks. > > [1] http://github.com/mesosphere/marathon > > > Thanks, > > Connor Doyle > >
