----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51167/#review146241 -----------------------------------------------------------
Patch looks great! Reviews applied: [51155, 51167] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On Aug. 20, 2016, 6:16 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51167/ > ----------------------------------------------------------- > > (Updated Aug. 20, 2016, 6:16 p.m.) > > > Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone. > > > Bugs: MESOS-6051 > https://issues.apache.org/jira/browse/MESOS-6051 > > > Repository: mesos > > > Description > ------- > > The functions added in this commit are not currently used anywhere in > the code base. However, they are required for some changes we are > making in order to persist container state in cases where an agent > crashes. The changes to use these function will come in subsequent > commits. > > > Diffs > ----- > > src/slave/containerizer/mesos/launcher.hpp > bf435e3a9c150648336a1becf2f075fa183428bd > src/slave/containerizer/mesos/launcher.cpp > 9efe8474a2210957ce256fc08cb35694194213c3 > src/slave/containerizer/mesos/linux_launcher.hpp > c1852226c74bc611d045be721e284141e59adcd9 > src/slave/containerizer/mesos/linux_launcher.cpp > 95dee95c5e6e613e526c92d8729ae5583c8b58f1 > src/tests/containerizer/launcher.hpp > 7e5c243efad11d04e70b36876b2ed4db82666d31 > > Diff: https://reviews.apache.org/r/51167/diff/ > > > Testing > ------- > > GTEST_FILTER="" make -j check > src/mesos-tests > sudo src/mesos-tests > > > Thanks, > > Kevin Klues > >