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

Reply via email to