Github user vixns commented on a diff in the pull request:

    https://github.com/apache/mesos/pull/299#discussion_r197728745
  
    --- Diff: src/slave/containerizer/mesos/isolators/network/cni/cni.cpp ---
    @@ -428,6 +428,10 @@ Future<Nothing> NetworkCniIsolatorProcess::recover(
           }
         }
     
    +    if (infos.contains(containerId)) {
    +      continue;
    +    }
    +
    --- End diff --
    
    are you sure ? if we move the continue to L421, L423 will have no effect.



---

Reply via email to