----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42516/#review119265 -----------------------------------------------------------
include/mesos/v1/mesos.proto (line 1534) <https://reviews.apache.org/r/42516/#comment180574> Remove extra space. src/docker/docker.cpp (line 521) <https://reviews.apache.org/r/42516/#comment180575> Capitalize the first letter of your comment ( // User defined...) src/docker/docker.cpp (line 528) <https://reviews.apache.org/r/42516/#comment180576> Space between next if src/docker/docker.cpp (line 530) <https://reviews.apache.org/r/42516/#comment180577> Fix the formatting, need to move one more char to the right. - Timothy Chen On Feb. 11, 2016, 1:51 p.m., Ezra Silvera wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42516/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2016, 1:51 p.m.) > > > Review request for mesos and Timothy Chen. > > > Bugs: MESOS-4369 > https://issues.apache.org/jira/browse/MESOS-4369 > > > Repository: mesos > > > Description > ------- > > Signed-off-by: Ezra Silvera <e...@il.ibm.com> > > > Diffs > ----- > > include/mesos/mesos.proto 5d1a20b4ff647f43701a353472b56a4b74b6bbc3 > include/mesos/v1/mesos.proto 415a20d9f3385e9627635dd89d78c58d03e3c116 > src/docker/docker.cpp 52728707d985517e57525af7e470ccb468039373 > > Diff: https://reviews.apache.org/r/42516/diff/ > > > Testing > ------- > > Using Swarm running on Mesos create a network with "docker network create > --driver=bridge myNetwork" and then create a container on that network: > "docker run --net=myNetwork...." > > > Thanks, > > Ezra Silvera > >