----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39769/#review105419 -----------------------------------------------------------
Ship it! Let's just rebase it and then it's ready to go. src/slave/containerizer/mesos/containerizer.hpp (lines 130 - 137) <https://reviews.apache.org/r/39769/#comment164006> Don't need this one either. src/slave/containerizer/mesos/containerizer.cpp (lines 300 - 600) <https://reviews.apache.org/r/39769/#comment164005> Let's remove this change now. - Kapil Arya On Nov. 5, 2015, 7:50 p.m., Spike Curtis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39769/ > ----------------------------------------------------------- > > (Updated Nov. 5, 2015, 7:50 p.m.) > > > Review request for mesos, Jie Yu and Kapil Arya. > > > Bugs: MESOS-3810 > https://issues.apache.org/jira/browse/MESOS-3810 > > > Repository: mesos > > > Description > ------- > > Always copy ContainerInfo to ExecutorInfo > > Remove MesosContainerizer check for TaskInfo->ContainerInfo > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.hpp > 4aad8a3be43b331efc6b8157b2fae090df16c1b4 > src/slave/containerizer/mesos/containerizer.cpp > 9fd69c1738e2300dbb843d259727010e24523cff > src/slave/slave.cpp 5f9b52b41eaab0c24965f28e192074340e00bde5 > src/tests/slave_tests.cpp 91dbdba56c7d3a374e56be92d88c0b367c7a2e1c > > Diff: https://reviews.apache.org/r/39769/diff/ > > > Testing > ------- > > make check > > passes network testing framework battery of tests at > https://github.com/mesosphere/net-modules/tree/integration/0.25 > > > Thanks, > > Spike Curtis > >
