Eric Badger commented on YARN-7446:

Hey [~eyang], just a few minor things on this patch.

nit: To be consistent with the rest of the code, we should move the frees that 
aren't in the if statements to the end of the functions, just before the return.

+  if (privileged == NULL || strcmp(privileged, "false") == 0) {
This should use {{strcasecmp}} to be consistent with the other privilege checks.

> Docker container privileged mode and --user flag contradict each other
> ----------------------------------------------------------------------
>                 Key: YARN-7446
>                 URL: https://issues.apache.org/jira/browse/YARN-7446
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 3.0.0
>            Reporter: Eric Yang
>            Assignee: Eric Yang
>            Priority: Major
>         Attachments: YARN-7446.001.patch, YARN-7446.002.patch, 
> YARN-7446.003.patch
> In the current implementation, when privileged=true, --user flag is also 
> passed to docker for launching container.  In reality, the container has no 
> way to use root privileges unless there is sticky bit or sudoers in the image 
> for the specified user to gain privileges again.  To avoid duplication of 
> dropping and reacquire root privileges, we can reduce the duplication of 
> specifying both flag.  When privileged mode is enabled, --user flag should be 
> omitted.  When non-privileged mode is enabled, --user flag is supplied.

This message was sent by Atlassian JIRA

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