Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-17 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45947 --- Ship it! Ship It! - Jie Yu On June 16, 2014, 9:58 p.m., Ian Down

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-17 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45944 --- Bad patch! Reviews applied: [20817, 20818, 22064] Failed command:

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-16 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- (Updated June 16, 2014, 2:58 p.m.) Review request for mesos and Jie Yu. Repos

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-16 Thread Ian Downes
> On June 11, 2014, 3:39 p.m., Jie Yu wrote: > > src/slave/containerizer/linux_launcher.cpp, lines 128-131 > > > > > > In fact, we just discovered an issue which will occur when an isolator > > cleans up an orphaned c

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45553 --- Bad patch! Reviews applied: [22064] Failed command: git apply --in

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-11 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- (Updated June 11, 2014, 11:31 p.m.) Review request for mesos and Jie Yu. Chan

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-11 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45440 --- src/slave/containerizer/isolators/cgroups/cpushare.cpp

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-11 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- (Updated June 11, 2014, 3:21 p.m.) Review request for mesos, Jie Yu and Vinod K

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-11 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45396 --- src/slave/containerizer/linux_launcher.cpp

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-11 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- (Updated June 11, 2014, 10:27 a.m.) Review request for mesos, Jie Yu and Vinod

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review45057 --- Bad patch! Reviews applied: [22064] Failed command: git apply --in

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-06-08 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- (Updated June 8, 2014, 9:15 p.m.) Review request for mesos, Jie Yu and Vinod Ko

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-05-31 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review44463 --- Bad patch! Reviews applied: [20818] Failed command: git apply --in

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-05-30 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review44405 --- src/slave/containerizer/linux_launcher.cpp

Re: Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-05-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/#review44392 --- Ship it! src/slave/containerizer/isolators/cgroups/cpushare.cpp

Review Request 22064: Use a timeout to discard cgroups::destroy.

2014-05-29 Thread Ian Downes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22064/ --- Review request for mesos, Jie Yu and Vinod Kone. Repository: mesos-git Descri