----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50162/#review142645 -----------------------------------------------------------
src/slave/containerizer/mesos/isolators/network/cni/cni.cpp (line 1463) <https://reviews.apache.org/r/50162/#comment208266> shouldn't we just do if (flags.etc_hosts_path.isSome()) { ..... } instead of having a separate check fo `flags.etc_hosts_path.isNone()` ? src/slave/containerizer/mesos/isolators/network/cni/cni.cpp (line 1533) <https://reviews.apache.org/r/50162/#comment208267> Not yours but s/`flags.hostname1/`flags.hostname` ? - Avinash sridharan On July 18, 2016, 9:24 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50162/ > ----------------------------------------------------------- > > (Updated July 18, 2016, 9:24 p.m.) > > > Review request for mesos, Avinash sridharan and Qian Zhang. > > > Bugs: MESOS-5806 > https://issues.apache.org/jira/browse/MESOS-5806 > > > Repository: mesos > > > Description > ------- > > Ignored /etc/hosts if it does not exist in CNI isolator. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp > 81121a6a83fa9f62eea06a64ab96b1f564a63da8 > > Diff: https://reviews.apache.org/r/50162/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Jie Yu > >
