mirror of
https://github.com/games-on-whales/gow.git
synced 2026-10-15 21:02:02 +00:00
[GH-ISSUE #120] Docker compose syntax change required #41
Labels
No labels
bug
enhancement
enhancement
github_actions
good first issue
help wanted
help wanted
invalid
needs testers
pull-request
question
x11-images
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
games-on-whales/gow#41
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @MetalGooseSolid on GitHub (Jan 17, 2023).
Original GitHub issue: https://github.com/games-on-whales/gow/issues/120
At some recent version of Nvidia Container Toolkit (I think that's what's causing the breakage?), there's a compatibility change with docker compose yaml. The line
runtime: ${DOCKER_RUNTIME}is broken and needs to be replaced with something like:@zb140 commented on GitHub (Feb 5, 2023):
My understanding is there's a difference between the "NVIDIA Container Toolkit" and the "NVIDIA Container Runtime"; the Runtime is what we're currently configured to use, while the Toolkit uses this new syntax. The Toolkit is the preferred method for using GPUs in Docker going forward, so we're definitely going to need to support this. There's a couple of things we'll need to figure out:
docker-composeto be able to use the new syntax? If so, we'll need to make surerun-gowis requiring the correct version.run-gow?@dennispg commented on GitHub (Jun 22, 2023):
I was able to have both in my compose file, like this:
Didn't need to specify the driver with this. This is with docker compose version 2.18.1 and nvidia-container-toolkit 1.13.2-1.
The deploy option was added in compose spec v3, which has been available since docker-compose version 1.13. The min required right now is 2.6, so I think there should be no harm in adding this.
@ABeltramo commented on GitHub (Jun 14, 2024):
The
run-gowscript has been deprecated in favour of Wolf