Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[Rest Server] Check default runtime before starting Docker #2754

Merged
merged 1 commit into from
May 15, 2019

Conversation

abuccts
Copy link
Member

@abuccts abuccts commented May 14, 2019

Check default runtime before starting Docker, fix #2739.

If nvidia-docker is installed and nvidia runtime is enabled by default, start nvidia-docker and isolate GPU through NVIDIA_VISIBLE_DEVICES provided by nvidia runtime.

Check default runtime before starting Docker, fix #2739.
If nvidia-docker is installed and nvidia runtime is enabled by default,
start nvidia-docker and isolate GPU through NVIDIA_VISIBLE_DEVICES
provided by nvidia runtime.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 53.075% when pulling ddd095a on xiongyf/check-default-runtime into a32a51c on master.

@@ -142,7 +142,7 @@ const generateYarnContainerScript = (frameworkName, userName, config, frameworkD
azRDMA: azureEnv.azRDMA === 'true' ? true : false,
reqAzRDMA: false,
inspectPidFormat: '{{.State.Pid}}',
inspectOOMKilledFormat: '{{.State.OOMKilled}}',
infoDefaultRuntimeFormat: '{{json .DefaultRuntime}}',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems this field is static and we can put it into the shell script.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the {{ ... }} will be replaced by mustache in shell script template, so we have to put it here as a workaround.

@abuccts abuccts merged commit e1b8f5d into master May 15, 2019
@abuccts abuccts deleted the xiongyf/check-default-runtime branch May 15, 2019 08:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Rest Server] Cannot start job container by nvidia-docker
3 participants