----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38370/#review98917 -----------------------------------------------------------
src/docker/executor.cpp (line 162) <https://reviews.apache.org/r/38370/#comment155638> Seems odd to have other tools referenced here in Mesos codebase. And also I think that even if Mesos-DNS migrates to use labels, are we expecting all users to force upgrade as their old version won't work with latest Mesos? - Timothy Chen On Sept. 14, 2015, 8:55 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38370/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2015, 8:55 p.m.) > > > Review request for mesos, Connor Doyle, Jie Yu, Niklas Nielsen, and Timothy > Chen. > > > Bugs: MESOS-3013 > https://issues.apache.org/jira/browse/MESOS-3013 > > > Repository: mesos > > > Description > ------- > > Updated docker executor to set container IP in TaskStatus::NetworkInfo. > > > Diffs > ----- > > src/docker/executor.cpp 6647075b55e5a79264e3556bb906a1f26a2d673e > > Diff: https://reviews.apache.org/r/38370/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Kapil Arya > >
