Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-17 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review191418 --- Ship it! Ship It! - Gilbert Song On Nov. 17, 2017, 8:51

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-17 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- (Updated Nov. 17, 2017, 4:51 p.m.) Review request for mesos, Gilbert Song,

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-16 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review191283 --- Fix it, then Ship it!

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-13 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- (Updated Nov. 14, 2017, 6:20 a.m.) Review request for mesos, Gilbert Song,

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-13 Thread Gilbert Song
> On Nov. 9, 2017, 9:25 p.m., Zhitao Li wrote: > > src/slave/containerizer/mesos/provisioner/provisioner.cpp > > Lines 531 (patched) > > > > > > This is the layerRootfs from store. > > > > However, I do not

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-13 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review190888 --- Fix it, then Ship it! LGTM!

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-10 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- (Updated Nov. 10, 2017, 7:33 p.m.) Review request for mesos, Gilbert Song,

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-09 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review190672 --- src/slave/containerizer/mesos/provisioner/provisioner.cpp Lines

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-09 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review190665 --- src/slave/containerizer/mesos/provisioner/provisioner.cpp Lines

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-11-07 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review190353 --- src/slave/containerizer/mesos/provisioner/paths.hpp Line 55

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-10-27 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- (Updated Oct. 27, 2017, 6:03 p.m.) Review request for mesos, Gilbert Song,

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-10-26 Thread Zhitao Li
> On Oct. 26, 2017, 11:45 p.m., Gilbert Song wrote: > > src/slave/containerizer/mesos/provisioner/provisioner.hpp > > Lines 170 (patched) > > > > > > why is it an `Option`? Similar to `ContainerConfig`, we might be

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-10-26 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/#review189378 --- src/slave/containerizer/mesos/provisioner/provisioner.hpp Lines

Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-10-17 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- (Updated Oct. 17, 2017, 5:04 p.m.) Review request for mesos, Gilbert Song,

Review Request 62997: Added checkpoint and recover capability for layers in provisioner.

2017-10-13 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62997/ --- Review request for mesos, Gilbert Song, Jason Lai, and Jie Yu. Bugs: