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

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 

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