----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62017/ -----------------------------------------------------------
(Updated Sept. 5, 2017, 9:47 p.m.) Review request for mesos and Avinash sridharan. Bugs: mesos-7923 https://issues.apache.org/jira/browse/mesos-7923 Repository: mesos Description ------- Mesos port mapper cni plugin is a wrapper around bridge plugin to add port mapping functionality to bridge plugin. However, in certain cases the network creator doesn't need port mapping functionality and just want to access bridge plugin. In this case, the creator may not supply `args` in cni config which will makes mesos port mapper plugin to fail. This patch makes `args` in cni config optional for mesos port mapper plugin Diffs (updated) ----- src/slave/containerizer/mesos/isolators/network/cni/plugins/port_mapper/port_mapper.cpp 43cf3e44a55c56dc8195c9cd05f6edd8bf13d448 Diff: https://reviews.apache.org/r/62017/diff/2/ Changes: https://reviews.apache.org/r/62017/diff/1-2/ Testing ------- Thanks, Deepak Goel