[ 
https://issues.apache.org/jira/browse/YARN-7814?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16339234#comment-16339234
 ] 

Shane Kumpf commented on YARN-7814:
-----------------------------------

Thanks [~miklos.szeg...@cloudera.com]. I have cleaned up the other JIRAs.

I attached a patch that removes the automatic mounting. I'm not finding any 
tests that check for its presence, so no tests added or removed.

While it is not currently in use, I did leave the code related to 
{{CgroupsHandler#getCGroupMounthPath}} as it seems that could have value 
elsewhere. It was added to get rid of the findbugs warning regarding the 
hardcoded /sys/fs/cgroup path. If others feel it should just be removed, I can 
do so.

> Remove automatic mounting of the cgroups root directory into Docker containers
> ------------------------------------------------------------------------------
>
>                 Key: YARN-7814
>                 URL: https://issues.apache.org/jira/browse/YARN-7814
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Shane Kumpf
>            Assignee: Shane Kumpf
>            Priority: Major
>         Attachments: YARN-7814.001.patch
>
>
> Currently, all Docker containers launched by {{DockerLinuxContainerRuntime}} 
> get /sys/fs/cgroup automatically mounted. Now that user supplied mounts 
> (YARN-5534) are in, containers that require this mount can request it (with a 
> properly configured mount whitelist).
> I propose we remove the automatic mounting of /sys/fs/cgroup into Docker 
> containers.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to