Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jie Yu
On Aug. 12, 2015, 10:47 p.m., Jiang Yan Xu wrote: This thing is rather complex and it deals with Mesos managed persisetent volumes and other system volumes in different but related ways; rootfs may or may not be used; and we'll provision image volumes later. I hope we iterate on this

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jie Yu
On Aug. 12, 2015, 10:47 p.m., Jiang Yan Xu wrote: src/slave/containerizer/isolators/filesystem/linux.cpp, line 485 https://reviews.apache.org/r/37330/diff/2/?file=1037046#file1037046line485 s/other/another/ 'other' sounds fine to me here. On Aug. 12, 2015, 10:47 p.m., Jiang Yan

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/ --- (Updated Aug. 12, 2015, 11:21 p.m.) Review request for mesos, Lily Chen, Ian

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/#review95154 --- This thing is rather complex and it deals with Mesos managed

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/#review95208 --- Ship it! Ship It! - Jiang Yan Xu On Aug. 12, 2015, 4:21 p.m.,

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jie Yu
On Aug. 12, 2015, 10:47 p.m., Jiang Yan Xu wrote: src/slave/containerizer/isolators/filesystem/linux.cpp, line 579 https://reviews.apache.org/r/37330/diff/2/?file=1037046#file1037046line579 What happens when persistent volumes are used without a new rootfs? Jie Yu wrote:

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-12 Thread Jie Yu
On Aug. 12, 2015, 1:43 a.m., Timothy Chen wrote: src/slave/containerizer/isolators/filesystem/linux.cpp, line 399 https://reviews.apache.org/r/37330/diff/2/?file=1037046#file1037046line399 Can we also log the container id and it's the linux filesystem isolator? It's much easier

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-11 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/#review95046 --- src/slave/containerizer/isolators/filesystem/linux.cpp (line 399)

Re: Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/ --- (Updated Aug. 11, 2015, 1:57 a.m.) Review request for mesos, Lily Chen, Ian

Review Request 37330: Added persistent volume support for linux filesystem isolator.

2015-08-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37330/ --- Review request for mesos, Lily Chen, Ian Downes, Jojy Varghese, Timothy Chen,