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

Shane Kumpf edited comment on YARN-8141 at 4/10/18 10:19 PM:
-------------------------------------------------------------

Thanks for reporting this, [~leftnoteasy] - 
{{YARN_CONTAINER_RUNTIME_DOCKER_MOUNTS}} is intended to be used for the purpose 
you call out. When that variable was added, 
{{YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS}} was retained due to its 
existing use in native services. Is there a case where 
{{YARN_CONTAINER_RUNTIME_DOCKER_MOUNTS}} won't work for your need? Maybe it is 
time we look to consolidate these two.


was (Author: shaneku...@gmail.com):
Thanks for reporting this, [~leftnoteasy] - 
{{YARN_CONTAINER_RUNTIME_DOCKER_MOUNTS}} is intended to be used for the purpose 
you call out. When that variable was added, 
{{YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS}} was retained due to its 
existing use in native services. Is there a case where 
{{YARN_CONTAINER_RUNTIME_DOCKER_MOUNTS}} won't work for your need? Maybe it is 
time we do look to consolidate these two.

> YARN Native Service: Respect 
> YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS specified in service spec
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-8141
>                 URL: https://issues.apache.org/jira/browse/YARN-8141
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: yarn-native-services
>            Reporter: Wangda Tan
>            Priority: Critical
>
> Existing YARN native service overwrites 
> YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS regardless if user 
> specified this in service spec or not. It is important to allow user to mount 
> local folders like /etc/passwd, etc.
> Following logic overwrites the 
> YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS environment:
> {code:java}
> StringBuilder sb = new StringBuilder();
> for (Entry<String,String> mount : mountPaths.entrySet()) {
>   if (sb.length() > 0) {
>     sb.append(",");
>   }
>   sb.append(mount.getKey());
>   sb.append(":");
>   sb.append(mount.getValue());
> }
> env.put("YARN_CONTAINER_RUNTIME_DOCKER_LOCAL_RESOURCE_MOUNTS", 
> sb.toString());{code}
> Inside AbstractLauncher.java



--
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