Skip to content
This repository has been archived by the owner on Oct 27, 2023. It is now read-only.

Support for openSUSE Leap 15.1 #32

Closed
wants to merge 150 commits into from

Conversation

dev-zero
Copy link
Contributor

No description provided.

flx42 and others added 30 commits September 5, 2017 18:54
Signed-off-by: Felix Abecassis <fabecassis@nvidia.com>
It gives the opportunity to perform cleanup in the main function.
Any environment variable with the prefix "NVIDIA_REQUIRE_" will have
its value passed to nvidia-container-cli.

NVIDIA_CUDA_VERSION is no longer used and the CUDA images will be
replaced by a NVIDIA_REQUIRE_CUDA variable.

Those checks can be disabled globally in the configuration file or for
each container using NVIDIA_DISABLE_REQUIRE.
RenaudWasTaken and others added 7 commits August 1, 2019 23:50
Ensure base-builder stage only runs when base dockerfiles are changed

See merge request nvidia/container-toolkit/nvidia-container-runtime!14
Signed-off-by: Renaud Gaubert <rgaubert@nvidia.com>
Add XDG override for config directory

See merge request nvidia/container-toolkit/nvidia-container-runtime!15
Signed-off-by: Renaud Gaubert <rgaubert@nvidia.com>
Add initial runtime container

See merge request nvidia/container-toolkit/nvidia-container-runtime!16
Signed-off-by: Jordan Jacobelli <jjacobelli@nvidia.com>
Update default runtime configuration

See merge request nvidia/container-toolkit/nvidia-container-runtime!17
Makefile Outdated Show resolved Hide resolved
dev-zero added a commit to dev-zero/nvidia-docker that referenced this pull request Aug 14, 2019
This requires support for openSUSE in the nvidia-container-runtime, see:

  NVIDIA/nvidia-container-runtime#32

Fixes NVIDIA#657

Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
dev-zero added a commit to dev-zero/nvidia-docker that referenced this pull request Aug 14, 2019
This requires support for openSUSE in the nvidia-container-runtime, see:

  NVIDIA/nvidia-container-runtime#32

Fixes NVIDIA#657

Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
@dev-zero dev-zero force-pushed the opensuse-support branch 2 times, most recently from 2d2d533 to f5cdc80 Compare August 14, 2019 12:08
@dev-zero
Copy link
Contributor Author

@RenaudWasTaken thanks for the extensive explanation. Being at the boundary of corporate and open source can be quiet stressful, I get that. It is nice to see more activity again.

dev-zero added a commit to dev-zero/nvidia-docker that referenced this pull request Aug 14, 2019
This requires support for openSUSE in the nvidia-container-runtime, see:

  NVIDIA/nvidia-container-runtime#32

Fixes NVIDIA#657

Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
@dev-zero
Copy link
Contributor Author

also in this PR did I now switch from using opensuse_leap to opensuse-leap to match the ID in /etc/os-release for consistency

@@ -0,0 +1,38 @@
ARG BASE
ARG VERSION_ID
FROM nvidia/base/opensuse-leap:${VERSION_ID}
Copy link
Contributor

Choose a reason for hiding this comment

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

You should use the BASE argument here

dev-zero added a commit to dev-zero/nvidia-docker that referenced this pull request Aug 23, 2019
This requires support for openSUSE in the nvidia-container-runtime, see:

  NVIDIA/nvidia-container-runtime#32

Fixes NVIDIA#657

Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
RenaudWasTaken pushed a commit to NVIDIA/nvidia-docker that referenced this pull request Aug 23, 2019
This requires support for openSUSE in the nvidia-container-runtime, see:

  NVIDIA/nvidia-container-runtime#32

Fixes #657

Signed-off-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
@RenaudWasTaken
Copy link
Contributor

I'm not sure what happened with this PR?

@RenaudWasTaken
Copy link
Contributor

Merged: de70c3f !

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.

None yet