----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51097/ -----------------------------------------------------------
Review request for mesos, Jie Yu and Qian Zhang. Bugs: MESOS-6023 https://issues.apache.org/jira/browse/MESOS-6023 Repository: mesos Description ------- This class will embody the logic for implementing the CNI port-mapper plugin. Diffs ----- src/Makefile.am d389e02584cfb1a00067cd30de1364118d1c46c7 src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/main.cpp PRE-CREATION src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.hpp PRE-CREATION src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp PRE-CREATION Diff: https://reviews.apache.org/r/51097/diff/ Testing ------- Tested the port-mapper with the following CNI config: { "name": "mynet", "type": "port-mapper", "chain": "MESOS", "delegate": { "type" : "bridge", "bridge": "cni0", "isGateway": true, "ipMasq": true, "ipam": { "type": "host-local", "subnet": "10.22.0.0/16", "routes": [ { "dst": "0.0.0.0/0" } ] } }, "args" : { "org.apache.mesos" : { "network_info" : { "port_mappings": { "host_port" : 80, "container_port" : 80 } } } } } If we remove fields from the above CNI config, the creation of the `PortMapper` correctly fails. However, if config is passed as is it will create the `PortMapper` correctly. Thanks, Avinash sridharan