Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-20 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107371 --- Patch looks great! Reviews applied: [40371] Passed command:

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-20 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107446 --- Thanks Jojy, I'll fix these myself and merge. - Timothy Chen On

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-20 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107444 --- src/cli/execute.cpp (line 237)

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-19 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107151 --- Patch looks great! Reviews applied: [40371] Passed command:

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-19 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107192 --- Bad patch! Reviews applied: [40284, 40285] Failed command:

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-19 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 20, 2015, 7:45 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-19 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 19, 2015, 3:37 p.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-18 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 19, 2015, 12:39 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-18 Thread Guangya Liu
> On εδΈ€ζœˆ 17, 2015, 3:12 a.m., Guangya Liu wrote: > > src/cli/execute.cpp, lines 216-228 > > > > > > Can you please explain more why using mesos containerizer can also run > > a docker container? Thanks! > > Jojy

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-18 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review107104 --- src/cli/execute.cpp (line 237)

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-17 Thread Jojy Varghese
> On Nov. 17, 2015, 3:12 a.m., Guangya Liu wrote: > > src/cli/execute.cpp, lines 216-228 > > > > > > Can you please explain more why using mesos containerizer can also run > > a docker container? Thanks! The

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review106762 --- Bad patch! Reviews applied: [40285] Failed command:

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 17, 2015, 2:52 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 17, 2015, 3:01 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review106789 --- src/cli/execute.cpp (lines 216 - 228)

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review106790 --- Bad patch! Reviews applied: [40285] Failed command:

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/ --- (Updated Nov. 17, 2015, 1:46 a.m.) Review request for mesos and Timothy Chen.

Re: Review Request 40371: Changed mesos-execute to add containerizer option.

2015-11-16 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40371/#review106782 --- Bad patch! Reviews applied: [40284, 40285] Failed command: