-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38660/#review100298
-----------------------------------------------------------

Ship it!


Ship It!

- Jie Yu


On Sept. 23, 2015, 9:46 p.m., Kapil Arya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38660/
> -----------------------------------------------------------
> 
> (Updated Sept. 23, 2015, 9:46 p.m.)
> 
> 
> Review request for mesos, haosdent huang and Jie Yu.
> 
> 
> Bugs: MESOS-3498
>     https://issues.apache.org/jira/browse/MESOS-3498
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The current logic uses LinuxLauncher if running as root. However, this causes
> problems with cgroups freezer when we launch multiple Agents inside the test
> suite.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/containerizer.cpp 
> b4a77e734706f40dc1d2110939d4f33fcbd1fd8c 
>   src/slave/flags.hpp e31a4183170c3442ac4a15365c229391e7e91480 
>   src/slave/flags.cpp add4196dfd06c0f602ff5ebd39960dc05c4cd11f 
>   src/tests/test_framework_test.sh b13cdd859e7e572ef4cf3d7d2721cb382e5e5acc 
> 
> Diff: https://reviews.apache.org/r/38660/diff/
> 
> 
> Testing
> -------
> 
> sudo make check currently works for all but one test. Here is the failing 
> case:
> 
> ```
> [----------] 1 test from LimitedCpuIsolatorTest
> [ RUN      ] LimitedCpuIsolatorTest.ROOT_CGROUPS_Pids_and_Tids
> ../../src/tests/containerizer/isolator_tests.cpp:742: Failure
> Value of: usage.get().processes()
>   Actual: 2
> Expected: 1U
> Which is: 1
> ../../src/tests/containerizer/isolator_tests.cpp:743: Failure
> Value of: usage.get().threads()
>   Actual: 2
> Expected: 1U
> Which is: 1
> [  FAILED  ] LimitedCpuIsolatorTest.ROOT_CGROUPS_Pids_and_Tids (187 ms)
> [----------] 1 test from LimitedCpuIsolatorTest (188 ms total)
> 
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (199 ms total)
> [  PASSED  ] 0 tests.
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] LimitedCpuIsolatorTest.ROOT_CGROUPS_Pids_and_Tids
> 
>  1 FAILED TEST
> ```
> 
> I am still investigating the failure but thought of putting out the review to 
> get some comments in the meanwhile (and some possible hints :-)).
> 
> 
> Thanks,
> 
> Kapil Arya
> 
>

Reply via email to