mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Avinash sridharan <>
Subject Re: Review Request 51857: Modified the `network/cni` isolator to be nesting aware.
Date Wed, 21 Sep 2016 20:18:47 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Sept. 21, 2016, 8:18 p.m.)

Review request for mesos, Gilbert Song, Jie Yu, Joseph Wu, and Qian Zhang.


Addressed Jie's comments.

Bugs: MESOS-6156

Repository: mesos


The network file setup in the `network/cni` isolator is now nesting
aware. Since the children share the network and UTS namespace with the
parent, the network files need to be created only for the parent
container. For the child containers, the network files will be simply
a bind mount of the parents network files.

Diffs (updated)

  src/slave/containerizer/mesos/isolators/network/cni/cni.hpp 949da8f70fb1cd13d6359780b032cb170693ea3e

  src/slave/containerizer/mesos/isolators/network/cni/cni.cpp 359479083894e887647a694a1a133dce44817073



make check
sudo ./bin/

The only tests that failed were the SUDO make check tests:
[  FAILED  ] 3 tests, listed below:
[  FAILED  ] CgroupsAnyHierarchyWithCpuMemoryTest.ROOT_CGROUPS_Listen
[  FAILED  ] CgroupsAnyHierarchyMemoryPressureTest.ROOT_IncreaseRSS
[  FAILED  ] LinuxFilesystemIsolatorTest.ROOT_RecoverOrphanedPersistentVolume


Avinash sridharan

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message