----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9814/ -----------------------------------------------------------
(Updated March 8, 2013, 11:12 p.m.) Review request for mesos, Benjamin Hindman and Ben Mahler. Changes ------- rebased off trunk for posterity. no need for review. Description ------- Pulled out some cgroups test fixtures into test utils, and made them generic, so that they can be re-used by other tests. Also the root cgroup can be configured via flag, helping cgroups isolation module related tests to use a different cgroup (e.g. "mesos_tests"). NOTE: This is based of my earlier cgroups review: https://reviews.apache.org/r/9806/ Diffs (updated) ----- src/slave/cgroups_isolation_module.cpp 438fa3e761dfd21d989158453c5213b08f1f7380 src/slave/flags.hpp c060c17798c96a563524cced1010f38debdeac4e src/tests/cgroups_tests.cpp 6db000e9db0768cb3065211bff019089ddec331c src/tests/utils.hpp 0863b789776408598e9720c68d5494393010ac8b Diff: https://reviews.apache.org/r/9814/diff/ Testing ------- sudo GLOG_v=1 ./bin/mesos-tests.sh --gtest_filter="*Cgroups*" --verbose Thanks, Vinod Kone
