mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Artem Harutyunyan" <ar...@mesosphere.io>
Subject Re: Review Request 39604: Added function that verifies prerequisites for using Linux launcher.
Date Mon, 02 Nov 2015 01:35:02 GMT


> On Oct. 30, 2015, 2:16 p.m., Jie Yu wrote:
> > src/slave/containerizer/linux_launcher.cpp, line 190
> > <https://reviews.apache.org/r/39604/diff/6/?file=1105496#file1105496line190>
> >
> >     OK, this is problematic and almost caused us an incident at Twitter.
> >     
> >     Mesos is able to create 'freezer' hierarchy if it's not already mounted. In
some configurations, the host won't pre-mount all cgroups hierarchies. We should remove that
check here.
> 
> Jie Yu wrote:
>     Chatted with Artem, a better way is to check: cgroups::enabled("freezer") here

https://reviews.apache.org/r/39841/


- Artem


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


On Oct. 23, 2015, 4:48 p.m., Artem Harutyunyan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39604/
> -----------------------------------------------------------
> 
> (Updated Oct. 23, 2015, 4:48 p.m.)
> 
> 
> Review request for mesos, Ben Mahler, Jie Yu, Joris Van Remoortere, and Kapil Arya.
> 
> 
> Bugs: MESOS-3800
>     https://issues.apache.org/jira/browse/MESOS-3800
> 
> 
> 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
> -----
> 
>   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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message