----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67728/#review205340 -----------------------------------------------------------
Patch looks great! Reviews applied: [67728] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On June 25, 2018, 7 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67728/ > ----------------------------------------------------------- > > (Updated June 25, 2018, 7 p.m.) > > > Review request for mesos, Qian Zhang and Sagar Patwardhan. > > > Bugs: MESOS-9025 > https://issues.apache.org/jira/browse/MESOS-9025 > > > Repository: mesos > > > Description > ------- > > The bug was introduced in https://reviews.apache.org/r/65987/ when we > want to allow nested container to have a separate network namespace than > its parent (MESOS-8534). > > The original code misses a `continue` statement after recovering regular > containers. > > I am surprised that the unit tests didn't catch the bug. We'll add a new > unit test for catching the regression. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp > b6ad4fcf9f1596c07ddeb9bbb134f4619d189671 > > > Diff: https://reviews.apache.org/r/67728/diff/1/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Jie Yu > >