Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Dockerfile for CUDA 11.8.0 #463

Merged
merged 9 commits into from
Feb 13, 2024
Merged

Add Dockerfile for CUDA 11.8.0 #463

merged 9 commits into from
Feb 13, 2024

Conversation

danpetry
Copy link
Member

@danpetry danpetry commented Jan 31, 2024

This is almost the same as the anaconda-pkg-build container, but derived from Nvidia's CentOS 7 CUDA 11.8.0 image.

See comment in the Dockerfile for info on what it contains and what conda must provide.

This is a copy over of this file which has already been used to successfully build CUDA packages.

This is almost the same as the anaconda-pkg-build container, but derived from
Nvidia's CentOS 7 CUDA 11.8.0 image.

Higher versions of CUDA are supported by conda; see comment in the Dockerfile.
@danpetry danpetry requested review from Callek and dbast January 31, 2024 20:05
@dbast
Copy link
Member

dbast commented Feb 1, 2024

Requires also a workflow file added similar to https://github.com/ContinuumIO/docker-images/blob/main/.github/workflows/anaconda_pkg_build_lts_linux.yml ... and pre-commit findings to be fixed

@danpetry
Copy link
Member Author

danpetry commented Feb 1, 2024

Thanks. Also while I remember, I want to put back in the cuda version argument, so that it's more flexible

Remove concourse stuff as concourse is no more

Adapt paths, image names as required

Leave in multi-platform functionality to be able to add more
architectures later if required
The hadolint error message was, "SC3045 warning: In POSIX sh, ulimit -n is
undefined.". ulimit -n 1024 both works and is necessary for the base container
(tested with nvidia/cuda:11.8.0-devel-centos7).
@danpetry
Copy link
Member Author

danpetry commented Feb 1, 2024

@dbast ready

@dbast dbast requested a review from chenghlee February 2, 2024 15:00
danpetry and others added 2 commits February 2, 2024 15:10
These repos don't exist. We can differentiate from the existing one with a suffix tag.

Co-authored-by: Daniel Bast <2790401+dbast@users.noreply.github.com>
To differentiate from the existing non-cuda container.

Co-authored-by: Daniel Bast <2790401+dbast@users.noreply.github.com>
@danpetry
Copy link
Member Author

danpetry commented Feb 5, 2024

@dbast any further comments?

@dbast dbast merged commit 00a16b5 into main Feb 13, 2024
2 checks passed
@dbast dbast deleted the dpetry-cuda-linux64-image branch February 13, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants