----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61866/#review183804 -----------------------------------------------------------
Ship it! Ship It! - Michael Park On Aug. 23, 2017, 3:05 p.m., Deepak Goel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61866/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2017, 3:05 p.m.) > > > Review request for mesos, Avinash sridharan and Michael Park. > > > Bugs: MESOS-7873 > https://issues.apache.org/jira/browse/MESOS-7873 > > > Repository: mesos > > > Description > ------- > > Adds port mapping information in NetworkInfo as part of `state` output. > > > Diffs > ----- > > src/common/http.cpp 43d674e9286170aae92014a952c6a8ba66437ca5 > > > Diff: https://reviews.apache.org/r/61866/diff/1/ > > > Testing > ------- > > Here are the steps that I followed to verify the fix: > 1. Created a cluster with 1 master, 2 private slave and 1 public slave > 2. Launched a task using mesos containerizer in bridge mode > 3. Queried agent `state` endpoint using curl http://<agentIP>:5051/state > 4. Verified that port-mapping information is present as part of NetworkInfo > > > Thanks, > > Deepak Goel > >