----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40036/#review105574 -----------------------------------------------------------
Bad patch! Reviews applied: [40036] Failed command: ./support/apply-review.sh -n -r 40036 Error: 2015-11-07 05:03:45 URL:https://reviews.apache.org/r/40036/diff/raw/ [6668/6668] -> "40036.patch" [1] error: patch failed: src/tests/master_tests.cpp:3339 error: src/tests/master_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Nov. 7, 2015, 1:47 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40036/ > ----------------------------------------------------------- > > (Updated Nov. 7, 2015, 1:47 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-3845 > https://issues.apache.org/jira/browse/MESOS-3845 > > > Repository: mesos > > > Description > ------- > > Along the lines of health checks, the container_status should also be sent > inside reconciliation updates. > > > Diffs > ----- > > src/common/protobuf_utils.hpp 44a2b1d3bab227294f5c5854acd87b7bd75ef589 > src/common/protobuf_utils.cpp b3c2d1183647ec1961f5d404355d08a5ce1d42a0 > src/master/master.cpp 25b94c8b41f65599327e0a0459ba86c6078895a8 > src/tests/master_tests.cpp 856440559de7d58be5bcf2ab6be911b5c67001cd > > Diff: https://reviews.apache.org/r/40036/diff/ > > > Testing > ------- > > `make check` with updated `MasterTest.TaskStatusContainerStatus` test. > > > Thanks, > > Kapil Arya > >
