----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25033/#review51435 -----------------------------------------------------------
src/slave/containerizer/mesos/containerizer.cpp <https://reviews.apache.org/r/25033/#comment89753> Do you want to check executorInfo.container as well here? src/slave/containerizer/mesos/containerizer.cpp <https://reviews.apache.org/r/25033/#comment89752> Remove this LOG(INFO)? If one uses --containerizers=mesos,docker, you will see this log line all over the place - Jie Yu On Aug. 25, 2014, 7:47 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25033/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2014, 7:47 p.m.) > > > Review request for mesos, Jie Yu and Timothy Chen. > > > Bugs: MESOS-1732 > https://issues.apache.org/jira/browse/MESOS-1732 > > > Repository: mesos-git > > > Description > ------- > > Fixed MesosContainerizer to reject tasks that have ContainerInfo set. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > d0676c55e3bb30bcc8e32b27c091080ec58b0c81 > > Diff: https://reviews.apache.org/r/25033/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
