----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59233/#review176161 -----------------------------------------------------------
Fix it, then Ship it! src/slave/slave.cpp Lines 4424 (patched) <https://reviews.apache.org/r/59233/#comment249514> Not that I'm opposed to you setting this explicitly can we also make this the default so that we don't have to have funky if checking code other places where we want to determine if it's v4 or v6? - Benjamin Hindman On May 12, 2017, 10:22 p.m., Avinash sridharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59233/ > ----------------------------------------------------------- > > (Updated May 12, 2017, 10:22 p.m.) > > > Review request for mesos and Jie Yu. > > > Bugs: MESOS-7488 > https://issues.apache.org/jira/browse/MESOS-7488 > > > Repository: mesos > > > Description > ------- > > Currently the agent is populating only the v4 address for containers > running on the host network. However, clusters running on a dual stack > network need to report v4 and v6 address for containers running on the > host network. This change allows v6 address specified by operators to be > advertised for containers running on the host network. > > > Diffs > ----- > > src/slave/slave.cpp 209aff180db1a68ae360d7c278937fe645efdb2b > > > Diff: https://reviews.apache.org/r/59233/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Avinash sridharan > >