----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69538/#review211153 -----------------------------------------------------------
Ship it! - Sergey Urbanovich On Dec. 10, 2018, 6:11 a.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69538/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2018, 6:11 a.m.) > > > Review request for mesos, Deepak Goel and James Peach. > > > Repository: mesos > > > Description > ------- > > This patch adds the UCR bridge network support for Mesos Mini using CNI > bridge plugin and Mesos port mapper CNI plugin. > > > Diffs > ----- > > support/mesos-mini/Dockerfile f8af422d8b8b0be4c462d40bce2d1337ab8b6e95 > support/mesos-mini/mesos/agent_environment > 48767281987c2e70b3a64f28c5eac139435c17f6 > support/mesos-mini/mesos/master_environment > 0382558744cb693c862246856b8c16ff42a95d19 > support/mesos-mini/mesos/ucr-default-bridge.json PRE-CREATION > > > Diff: https://reviews.apache.org/r/69538/diff/1/ > > > Testing > ------- > > Tested using this Marathon app > ```json > { > "id": "/test", > "instances": 1, > "container": { > "type": "MESOS", > "volumes": [], > "portMappings": [], > "docker": { > "image": "alpine" > } > }, > "cpus": 0.1, > "mem": 128, > "requirePorts": false, > "networks": [ > { > "mode": "container/bridge" > } > ], > "healthChecks": [], > "fetch": [], > "constraints": [], > "cmd": "sleep 100000" > } > ``` > > > Thanks, > > Jie Yu > >