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 `prepare` and `isolate` methods to be nesting aware.
Date Tue, 20 Sep 2016 17:10:05 GMT

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

(Updated Sept. 20, 2016, 5:10 p.m.)

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


Merged the `prepare` and `isolate` patches, and added comemnts to `prepare` to highlgiht the
cases where we need to store the container information in the `infos` structure.

Summary (updated)

Modified the `prepare` and `isolate` methods to be nesting aware.

Bugs: MESOS-6156

Repository: mesos

Description (updated)

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.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