> On Jan. 5, 2015, 5:53 a.m., Adam B wrote: > > src/master/master.cpp, lines 3711-3718 > > <https://reviews.apache.org/r/29556/diff/1/?file=806034#file806034line3711> > > > > Since this code appears twice, it's probably worth pulling out into a > > function. Could be a method on Task, or a free-standing locally-scoped > > function in master.cpp.
Good call, factored out to a function in protobuf_utils. - Connor ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29556/#review66598 ----------------------------------------------------------- On Jan. 6, 2015, 5:29 a.m., Connor Doyle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29556/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 5:29 a.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 > src/tests/health_check_tests.cpp a707398 > > 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 > >
