Skip to content

Removes 'nvidia' runtime parameter when Docker version is >= 19.03 (n…#166

Open
bgreer101 wants to merge 1 commit intodessa-oss:masterfrom
bgreer101:fix-atlas-server-start-cuda
Open

Removes 'nvidia' runtime parameter when Docker version is >= 19.03 (n…#166
bgreer101 wants to merge 1 commit intodessa-oss:masterfrom
bgreer101:fix-atlas-server-start-cuda

Conversation

@bgreer101
Copy link
Copy Markdown

With the release of Docker 19.03 there is no longer the need to install nvidia-docker2 to access GPU's. When atlas-server start -g is called, it appends the 'nvidia' runtime flag which no longer is required nor exists and causes the scheduler to fail to start. This patch only add's the runtime flag if gpu is requested, and docker is <= 19.03

@mohammedri
Copy link
Copy Markdown

@mohammedri mohammedri added the bug Something isn't working label Jun 3, 2020
@mohammedri
Copy link
Copy Markdown

@bgreer101 thanks for the PR! The user acceptance test pipeline is 🔴 now, but a fix is on the way.
Let's hold off on merge until then: https://jenkins.shehanigans.net/job/foundations-uat/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants