[v1.x] Add new CI pipeline for building and testing with cuda 11.0.#19149
[v1.x] Add new CI pipeline for building and testing with cuda 11.0.#19149samskalicky merged 12 commits intoapache:v1.xfrom
Conversation
|
Hey @josephevans , Thanks for submitting the PR
CI supported jobs: [miscellaneous, sanity, unix-gpu, centos-gpu, clang, website, windows-cpu, unix-cpu, edge, windows-gpu, centos-cpu] Note: |
…ve calls to nvidia/cudnn install scripts.
…anually install cudnn8.
…ee build works. Remove unneeded components of docker containers to reduce size and build time.
|
@leezu @ChaiBapchya Can you guys please review? |
| COPY runtime_functions.sh /work/ | ||
|
|
||
| WORKDIR /work/mxnet | ||
| ENV LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:/usr/local/cuda/compat |
There was a problem hiding this comment.
Pretty sure we need this:
/usr/local/cuda-11.0/compat/libcuda.so.450.51.06
/usr/local/cuda-11.0/compat/libcuda.so
/usr/local/cuda-11.0/compat/libcuda.so.1
There was a problem hiding this comment.
Without this, the cpp-package fails to build (due to unable to find libcuda.so.1, which libmxnet.so is linked against.)
I could also disable the cpp-package portion of the build, since it's actually not being used in the test pipeline steps.
…ctures and will hit the ar limit.
|
@mxnet-bot run ci [windows-gpu] |
|
Jenkins CI successfully triggered : [windows-gpu] |
DickJC123
left a comment
There was a problem hiding this comment.
Although the build configuration files are not my strong suit, this LGTM. The passing CI is a strong indication of correctness here. I'd like to see this merged as an important next step in finalizing the 1.8 release.
|
Thank you very much @josephevans :-) :-) |
Description
This PR adds a new pipeline to CI for testing builds under Cuda 11.0.
The new pipeline ("unix-gpu-cu110") is triggered by the full-build when the sanity build completes.