hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Varun Vasudev (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (YARN-6623) Add support to turn off launching privileged containers in the container-executor
Date Mon, 25 Sep 2017 16:23:00 GMT

     [ https://issues.apache.org/jira/browse/YARN-6623?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Varun Vasudev updated YARN-6623:
--------------------------------
    Attachment: YARN-6623.001.patch

Thanks for the review [~ebadger].
{noformat}
Hey Varun Vasudev, thanks for the updated patch! The changes related to YARN-4266 look good.
Just had one comment on the code.

1850 +static int set_network(const struct configuration *command_config,
1851 +                       const struct configuration *conf, char *out,
1852 +                       const size_t outlen) {
1853 +
1854 +  int ret = 0;
1855 +  ret = add_param_to_command_if_allowed(command_config, conf, "net",
1856 +                                        "docker.allowed.networks", "--net=",
1857 +                                        0, 0, out, outlen);
1858 +  if (ret != 0) {
1859 +    fprintf(ERRORFILE, "Could not find requested network in allowed networks\n");
1860 +    ret = INVALID_DOCKER_NETWORK;
1861 +  }
1862 +  if (ret != 0) {
1863 +    memset(out, 0, outlen);
1864 +  }
1865 +  return ret;
1866 +}

Something I noticed in passing. We should combine the if(ret != 0) statements here. This same
thing also occurs in set_capabilities() and set_devices().

{noformat}

Fixed.


> Add support to turn off launching privileged containers in the container-executor
> ---------------------------------------------------------------------------------
>
>                 Key: YARN-6623
>                 URL: https://issues.apache.org/jira/browse/YARN-6623
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>            Reporter: Varun Vasudev
>            Assignee: Varun Vasudev
>            Priority: Blocker
>         Attachments: YARN-6623.001.patch, YARN-6623.001.patch, YARN-6623.002.patch, YARN-6623.003.patch,
YARN-6623.004.patch, YARN-6623.005.patch, YARN-6623.006.patch, YARN-6623.007.patch, YARN-6623.008.patch,
YARN-6623.009.patch, YARN-6623.010.patch, YARN-6623.011.patch, YARN-6623.012.patch
>
>
> Currently, launching privileged containers is controlled by the NM. We should add a flag
to the container-executor.cfg allowing admins to disable launching privileged containers at
the container-executor level.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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


Mime
View raw message