You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by James Peach <jp...@apache.org> on 2017/07/11 11:15:33 UTC

Review Request 60767: Allow `network/ports` to co-exist with other network isolators.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60767/
-----------------------------------------------------------

Review request for mesos, Qian Zhang and Jiang Yan Xu.


Bugs: MESOS-7675
    https://issues.apache.org/jira/browse/MESOS-7675


Repository: mesos


Description
-------

Since the `network/ports` isolator does not configure any network
namespaces, it is able to run safely alongside other network
isolators. So when checking for multiple network isolators, we
should not count `network/ports` as a network isolator..


Diffs
-----

  src/slave/containerizer/mesos/containerizer.cpp 9376d14d66f5dc7e91c7c0e9da253f5eb9347539 


Diff: https://reviews.apache.org/r/60767/diff/1/


Testing
-------

make check (Fedora 26)


Thanks,

James Peach