Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

2015-09-16 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/ --- (Updated Sept. 16, 2015, 6 p.m.) Review request for mesos, Connor Doyle, Jie Yu

Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

2015-09-16 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/#review99281 --- Ship it! Ship It! - Niklas Nielsen On Sept. 14, 2015, 1:55 p.m.,

Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

2015-09-15 Thread Kapil Arya
> On Sept. 14, 2015, 6:32 p.m., Niklas Nielsen wrote: > > src/tests/containerizer/isolator_tests.cpp, line 507 > > > > > > Have you verified the status (non-error) of namespaces() before you get > > here? The Isola

Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

2015-09-14 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/#review98934 --- src/tests/containerizer/isolator_tests.cpp (line 202)

Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

2015-09-14 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38363/ --- Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. Bugs: MESOS