----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9814/#review17632 -----------------------------------------------------------
Ship it! Wonder how I could make use of these in my templated Isolator tests, only when instantiated with CgroupsIsolator. Maybe I'll have to make push most of the testing logic into a test fixture that I can re-use, as you mentioned. src/tests/utils.hpp <https://reviews.apache.org/r/9814/#comment37391> So the setup runs the teardown? Maybe a comment as to why? - Ben Mahler On March 8, 2013, 7:45 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9814/ > ----------------------------------------------------------- > > (Updated March 8, 2013, 7:45 p.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > 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 > ----- > > src/slave/cgroups_isolation_module.cpp > c7ca32c8122ce97f77a5ba68f6ad3dc1f17704f7 > src/slave/flags.hpp 9cd436cae5772b906b3566eb1f344904d5894214 > src/tests/cgroups_tests.cpp ed419eb174a095786fb6de9e181d42587e5b40e2 > src/tests/utils.hpp 0d4edad4787f92f86b1486b6b510f46a62d7d541 > > Diff: https://reviews.apache.org/r/9814/diff/ > > > Testing > ------- > > sudo GLOG_v=1 ./bin/mesos-tests.sh --gtest_filter="*Cgroups*" --verbose > > > Thanks, > > Vinod Kone > >
