----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67195/#review203731 -----------------------------------------------------------
Patch looks great! Reviews applied: [67271, 67195] 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 May 23, 2018, 6:33 p.m., Xudong Ni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67195/ > ----------------------------------------------------------- > > (Updated May 23, 2018, 6:33 p.m.) > > > Review request for mesos, James Peach and Jiang Yan Xu. > > > Bugs: MESOS-8340 > https://issues.apache.org/jira/browse/MESOS-8340 > > > Repository: mesos > > > Description > ------- > > To reduce deployment risk, a nonenforce mode is added for network > port isolator. When this flag is set as false(default is false), > even task uses ports not in the container resources, the container > will not raise any limitation. > > Add new test for this flag and update the existing tests > > > Diffs > ----- > > docs/isolators/network-ports.md ea63968481ce52c46e0a98e242da49baf6962009 > src/slave/containerizer/mesos/isolators/network/ports.hpp > ba71087194a3ae74c7e40dffa9c108b02ffa10ad > src/slave/containerizer/mesos/isolators/network/ports.cpp > 1f84ed4fb2a30fd095e2faec1038de1fa19a15c5 > src/slave/flags.hpp a839591a2b66444ad97fced0620201dde656352d > src/slave/flags.cpp a319b5ea633c41fd8a252c5e1617ac52d1480ba5 > src/tests/containerizer/ports_isolator_tests.cpp > c5b9f926047792e7f9d1f0937fa5355b1dd77965 > > > Diff: https://reviews.apache.org/r/67195/diff/3/ > > > Testing > ------- > > New test added for the flag; Related unit tests passed. > > [ OK ] NetworkPortsIsolatorTest.ROOT_NC_AllocatedPorts (1906 ms) > [ OK ] NetworkPortsIsolatorTest.ROOT_NC_NoPortsResource (1788 ms) > [ OK ] NetworkPortsIsolatorTest.ROOT_NC_NoPortEnforement (17001 ms) > > > Thanks, > > Xudong Ni > >
