Re: Review Request 34141: AppC provsioning backend.

2016-07-09 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review141470 --- Closing this review due to inactivity. Please see our [guidelines

Re: Review Request 34141: AppC provsioning backend.

2015-07-17 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review91685 --- src/slave/containerizer/provisioners/appc/backend.hpp (line 41)

Re: Review Request 34141: AppC provsioning backend.

2015-07-13 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review91338 --- src/slave/containerizer/provisioners/appc/backend.hpp (lines 25 - 2

Re: Review Request 34141: AppC provsioning backend.

2015-07-07 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review90778 --- src/slave/containerizer/provisioners/appc/backend.cpp (line 139)

Re: Review Request 34141: AppC provsioning backend.

2015-07-07 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/ --- (Updated July 7, 2015, 12:43 p.m.) Review request for mesos, Chi Zhang, Paul Br

Re: Review Request 34141: AppC provsioning backend.

2015-06-22 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/ --- (Updated June 22, 2015, 9:53 a.m.) Review request for mesos, Chi Zhang, Paul Br

Re: Review Request 34141: AppC provsioning backend.

2015-06-19 Thread Ian Downes
> On May 21, 2015, 1:25 p.m., Paul Brett wrote: > > src/slave/containerizer/provisioners/appc/backend.cpp, line 136 > > > > > > Running this as an unconstrained subprocess risks impacting the > > performance of every

Re: Review Request 34141: AppC provsioning backend.

2015-05-27 Thread Ian Downes
> On May 18, 2015, 4:38 p.m., Chi Zhang wrote: > > src/slave/containerizer/provisioners/appc/backend.hpp, line 50 > > > > > > Not a big deal, but would it be more flexible for Bankend to take a > > rootImage as an argu

Re: Review Request 34141: AppC provsioning backend.

2015-05-21 Thread Paul Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review84774 --- src/slave/containerizer/provisioners/appc/backend.cpp

Re: Review Request 34141: AppC provsioning backend.

2015-05-18 Thread Chi Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/#review84237 --- src/slave/containerizer/provisioners/appc/backend.hpp

Review Request 34141: AppC provsioning backend.

2015-05-12 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34141/ --- Review request for mesos, Chi Zhang, Paul Brett, Timothy Chen, and Vinod Kone.