----------------------------------------------------------- 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) <https://reviews.apache.org/r/38363/#comment155690> Does it make sense to have None() as the default value? src/tests/containerizer/isolator_tests.cpp (line 506) <https://reviews.apache.org/r/38363/#comment155691> Have you verified the status (non-error) of namespaces() before you get here? - Niklas Nielsen On Sept. 14, 2015, 1:55 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38363/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2015, 1:55 p.m.) > > > Review request for mesos, Connor Doyle, Jie Yu, and Niklas Nielsen. > > > Bugs: MESOS-3387 > https://issues.apache.org/jira/browse/MESOS-3387 > > > Repository: mesos > > > Description > ------- > > This prepares MesosContainerizer to allow isolators to specify namespaces for > each individual containerizer. Thus, allowing isolators to decide whether or > not > to enable isolation for a particular container. > > > Diffs > ----- > > src/slave/containerizer/launcher.hpp > 8cc832e99838683c14d8158bdf52a267299508c7 > src/slave/containerizer/launcher.cpp > 5267eecbdf31c88cd3c7dc7172b5aff7bac534b0 > src/slave/containerizer/linux_launcher.hpp > bf6bf3f27dac007429c224d4f6798ac68316accb > src/slave/containerizer/linux_launcher.cpp > 12acf4d08cb7fb1161820cf4bb480fd1bc2c6858 > src/slave/containerizer/mesos/containerizer.cpp > 1b83a8725b35435531038e37188b4c97189cef03 > src/tests/containerizer/filesystem_isolator_tests.cpp > ffa371fab69ce5eebe3b02afc2a1724a0f52110f > src/tests/containerizer/isolator_tests.cpp > d34c82a7f24da6f60cfb22790f516dc6065b1f6f > src/tests/containerizer/launcher.hpp > 6f020f94b042eef0e6a69ba8c26dfb697f3e81a3 > > Diff: https://reviews.apache.org/r/38363/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Kapil Arya > >
