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

[CI] CI cost saving #7407

Merged
merged 7 commits into from
Nov 18, 2021
Merged

[CI] CI cost saving #7407

merged 7 commits into from
Nov 18, 2021

Conversation

hcho3
Copy link
Collaborator

@hcho3 hcho3 commented Nov 9, 2021

  • Drop CUDA 10.x and use CUDA 11.0 in CI
  • Drop JDK 11 and JDK 12 tests

@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 9, 2021

I'll need to rebuild the machine image for the Windows worker to use CUDA 11.0.

@hcho3 hcho3 changed the title [CI] Drop CUDA 10.x and require CUDA 11.0 [CI] CI cost saving Nov 16, 2021
@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 16, 2021

I'll keep #7366 open, since clang-tidy fails when using CUDA 11.0

@trivialfis
Copy link
Member

Is it caused by def dockerArgs = "--build-arg CUDA_VERSION_ARG=10.1" in the ClangTidy groovy function?

@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 16, 2021

When I changed 10.1 to 11.0, clang-tidy fails to pass.

@trivialfis
Copy link
Member

Got it. I will take a look into the error later.

@trivialfis
Copy link
Member

Hi, the errored CI tests for clang-tidy seems to be cmake failing to find appropriate CTK:

[2021-11-16T09:29:31.963Z] -- Detecting CUDA compiler ABI info
[2021-11-16T09:29:32.615Z] -- Detecting CUDA compiler ABI info - done
[2021-11-16T09:29:32.615Z] CMake Error at CMakeLists.txt:134 (message):
[2021-11-16T09:29:32.615Z]   CUDA version must be at least 11.0!
[2021-11-16T09:29:32.615Z] 
[2021-11-16T09:29:32.615Z] 
[2021-11-16T09:29:32.615Z] -- Configuring incomplete, errors occurred!
[2021-11-16T09:29:32.615Z] See also "/workspace/cdb/CMakeFiles/CMakeOutput.log".

@trivialfis
Copy link
Member

Other than clang-tidy, is there anything else we need to do for this PR?

@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 17, 2021

@trivialfis We can merge this PR once the Windows test completes running. I had to re-build the Windows machine's image from scratch.

@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 17, 2021

@trivialfis Urr... One of the gtest is now failing:

[ RUN      ] MultiClassesSerializationTest.GpuHist
tests/cpp/test_serialization.cc(49): error: Expected equality of these values:
   l
     Which is: false
   r
     Which is: true

@hcho3
Copy link
Collaborator Author

hcho3 commented Nov 17, 2021

@trivialfis Are you okay with disabling MultiClassesSerializationTest.GpuHist for now? Reducing CI cost is a higher priority.

@trivialfis
Copy link
Member

Are you okay with disabling MultiClassesSerializationTest.GpuHist for now?

Go ahead. I will try to reproduce it later.

@hcho3 hcho3 merged commit 2adf222 into dmlc:master Nov 18, 2021
@hcho3 hcho3 deleted the remove_cuda10 branch November 18, 2021 05:02
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.

2 participants