> On Sept. 7, 2016, 5:02 a.m., Jie Yu wrote: > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp, lines 1537-1538 > > <https://reviews.apache.org/r/51643/diff/2/?file=1491981#file1491981line1537> > > > > I would mention that this step is necessary because we change the > > hostname to be containerID and libprocess (executor uses) needs to resolve > > that hostname.
We actually talk about this in the comments above: https://github.com/apache/mesos/blob/master/src/slave/containerizer/mesos/isolators/network/cni/cni.cpp#L1525 Have updated those comments to be a bit more explicit about the hostname change to 'continer ID' and the reasoning for this bind mount. - Avinash ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51643/#review147973 ----------------------------------------------------------- On Sept. 7, 2016, 7:02 p.m., Avinash sridharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51643/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2016, 7:02 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, Joseph Wu, and Qian Zhang. > > > Bugs: MESOS-6052 > https://issues.apache.org/jira/browse/MESOS-6052 > > > Repository: mesos > > > Description > ------- > > In case /etc/hosts and /etc/hostname files are not present in the host > filesystem, we were ignoring these files and assuming that they would > not be required by the executor when it is launched in a new network > namespace. This assumption is incorrect, since the executor needs > /etc/hosts in the new network namespace to resolve its hostname. > Hence, we are explicitly creating these files in the host file system > in case they are not present, so that containers /etc/hosts and > /etc/hostname can be mounted on these mount points. This solves the > problem in distributions such as CoreOS that don't have /etc/hosts in > their host filesystem. > > > Diffs > ----- > > src/slave/containerizer/mesos/isolators/network/cni/cni.cpp > d17a45fe17bb74cbf9ae421dc8a492e5dc5f1a00 > > Diff: https://reviews.apache.org/r/51643/diff/ > > > Testing > ------- > > make check > > Also, build an open DC/OS image with this patch to run on CoreOS and tried > launch a unified containerizer through Marathon using dcos-cli to test this > patch on CoreOS. > > > Thanks, > > Avinash sridharan > >
