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

(Updated Oct. 23, 2015, 2:35 p.m.)


Review request for mesos, Ben Mahler, Joris Van Remoortere, and Kapil Arya.


Changes
-------

Addressed comments.


Repository: mesos


Description
-------

We recently switched to using LinuxLauncher by default 
(https://github.com/apache/mesos/blame/master/src/slave/containerizer/mesos/containerizer.cpp#L217-L234).
 This causes problems with running the slave in a Docker container. 

This patch introduces a function for checking whether the machine is suited for 
running Linux launcher. For the time being it checks whether the freezer cgroup 
subsystem enabled, as the Linux launcher evolves we might want to add more 
checks.


Diffs (updated)
-----

  src/slave/containerizer/linux_launcher.hpp 
35b3315498d690ed66616617aa7d51455371fb5b 
  src/slave/containerizer/linux_launcher.cpp 
c03b89eb0678825b03a052874d6262f377a39e13 
  src/slave/containerizer/mesos/containerizer.cpp 
d1fc5a460e7313828014eea999cf4e63dde01921 

Diff: https://reviews.apache.org/r/39604/diff/


Testing
-------

- Ran Mesoss tests in a Docker container where cgroup was not available.
- Ran the new Jenkins script (https://reviews.apache.org/r/37787/).


Thanks,

Artem Harutyunyan

Reply via email to