----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17013/#review33878 -----------------------------------------------------------
Ship it! src/tests/cgroups_tests.cpp <https://reviews.apache.org/r/17013/#comment63561> kill this. src/tests/cgroups_tests.cpp <https://reviews.apache.org/r/17013/#comment63563> kill this. src/tests/cgroups_tests.cpp <https://reviews.apache.org/r/17013/#comment63564> s/base_hierarchy/baseHierarchy/ src/tests/containerizer.hpp <https://reviews.apache.org/r/17013/#comment63565> s/_// src/tests/containerizer.cpp <https://reviews.apache.org/r/17013/#comment63566> indentation. src/tests/isolator_tests.cpp <https://reviews.apache.org/r/17013/#comment63567> new line? src/tests/isolator_tests.cpp <https://reviews.apache.org/r/17013/#comment63568> new line. src/tests/isolator_tests.cpp <https://reviews.apache.org/r/17013/#comment63569> new line. src/tests/mesos.cpp <https://reviews.apache.org/r/17013/#comment63570> s/base_hierarchy/baseHierarchy/ src/tests/slave_recovery_tests.cpp <https://reviews.apache.org/r/17013/#comment63571> get rid of the comment. here and everywhere else. src/tests/slave_recovery_tests.cpp <https://reviews.apache.org/r/17013/#comment63572> kill this. - Vinod Kone On Feb. 6, 2014, 5:31 a.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17013/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2014, 5:31 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Niklas Nielsen, Jason > Dusek, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Updated tests to use Containerizer. > > A few isolator specific tests haven't been updated. > > The tests require a different cgroup mount configuration; please see > https://issues.apache.org/jira/browse/MESOS-926 for preliminary documentation. > > > Diffs > ----- > > src/slave/containerizer/cgroups_launcher.cpp PRE-CREATION > src/slave/containerizer/mesos_containerizer.cpp PRE-CREATION > src/tests/allocator_tests.cpp c9592d4baf9f8a29624722930eefe82ea9c1e129 > src/tests/cgroups_isolator_tests.cpp > 1f5ce764a895fefb24126e39a283a8efc290d580 > src/tests/cgroups_tests.cpp 0e9316d6561a1339bd2a3fb3482277658beba12b > src/tests/cluster.hpp 065976c19170e995bd3766bcc7a9b0a244776108 > src/tests/containerizer.hpp PRE-CREATION > src/tests/containerizer.cpp PRE-CREATION > src/tests/environment.cpp 41b8a71f63b959abdaafca09bb3e0a5eeeae691b > src/tests/fault_tolerance_tests.cpp > 60e06cc52d102ade687b095db474c0278ad508cc > src/tests/gc_tests.cpp 6638a4ac37cf80c86cf0a7ccd609868d2581716c > src/tests/isolator.hpp 6431dd2b6f441764c1e193aa2f19a6d38848e458 > src/tests/isolator_tests.cpp 45a41ca177efbc200c5e83448f92b4f1d8ca30e1 > src/tests/master_contender_detector_tests.cpp > 5223200ee59b9ce0e1d228320e8ad9e84a567288 > src/tests/master_tests.cpp 815149a6f6794e534c57b5767fdd18c776878dc0 > src/tests/mesos.hpp d7bdaee3cfd2bad95a12daf8737896becd1dcf18 > src/tests/mesos.cpp 1b1b4cc1fd6bd2823a82e08c8e37005ab65f5874 > src/tests/monitor_tests.cpp 7988c90b4b383bd822d1294a7f0318fbd9160dc6 > src/tests/paths_tests.cpp 40c644cf945c08e08ec9bc44e7d828c2547ba4cf > src/tests/slave_recovery_tests.cpp 4779509f95f6a62c22241aff5fc12d4fc2e92084 > src/tests/test_framework_test.sh 277245d563dc129ebeaabff1fae8707110e7879a > > Diff: https://reviews.apache.org/r/17013/diff/ > > > Testing > ------- > > make check # OSX and Linux > > > Thanks, > > Ian Downes > >
