----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54213/#review162577 -----------------------------------------------------------
src/slave/containerizer/mesos/provisioner/backends/aufs.cpp (lines 153 - 155) <https://reviews.apache.org/r/54213/#comment233925> See my comments in the previous patch. Please adjust accordingly and update aufs.hpp - Jie Yu On Jan. 19, 2017, 12:08 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54213/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2017, 12:08 p.m.) > > > Review request for mesos, Avinash sridharan, Artem Harutyunyan, Jie Yu, Qian > Zhang, and Zhitao Li. > > > Bugs: MESOS-6001 > https://issues.apache.org/jira/browse/MESOS-6001 > > > Repository: mesos > > > Description > ------- > > This issue is similar to MESOS-6000. We use the same solution > (using symlinks) to resolve many-layer image issue with aufs > backend. > > > Diffs > ----- > > src/slave/containerizer/mesos/provisioner/backends/aufs.cpp > c92c6c7174421158b9190ed0bfb00c1c97ef0f7b > > Diff: https://reviews.apache.org/r/54213/diff/ > > > Testing > ------- > > make check > > Tested by the new unit test `ROOT_AUFS_AufsBackendWithManyLayers` (see the > following patch). > > Manually tested using a 37-layer image `gilbertsong/cirros:34`. It failed > before the fix, and it passed after the fix. > > > Thanks, > > Gilbert Song > >
