[
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: [email protected]
For additional commands, e-mail: [email protected]
