> On Aug. 21, 2016, 8:56 p.m., Avinash sridharan wrote: > > src/slave/containerizer/mesos/containerizer.cpp, line 837 > > <https://reviews.apache.org/r/51141/diff/2/?file=1480417#file1480417line837> > > > > Is Volumes the field ? If yes then: > > s/Volumes/`Volumes`
This comment is incorrect. Fixed in a followup patch to avoid breaking Jie's patches. - Gilbert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51141/#review146286 ----------------------------------------------------------- On Aug. 21, 2016, 7:10 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51141/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2016, 7:10 p.m.) > > > Review request for mesos, Benjamin Hindman, Benjamin Mahler, Artem > Harutyunyan, Jie Yu, Kevin Klues, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This patch removes the image volume provisioning code in > MesosContainerizer. The logic will be moved to an isolator. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.hpp > 499396121de4e9e654f78f73493e76053fb83d42 > src/slave/containerizer/mesos/containerizer.cpp > 77a502f853e3e04ea8e274419544601778be9421 > > Diff: https://reviews.apache.org/r/51141/diff/ > > > Testing > ------- > > make > > > Thanks, > > Gilbert Song > >
