Only run OSU test for now in test step. #571
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… this would take too long otherwise. In the (near) future, we should make some mapping to determine which tests to run for which software installations
|
Instance
|
|
Instance
|
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/intel/skylake_avx512 |
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
|
Test step from the SLURM log: That's exactly what I intended it to look like. It's also much faster now (30s), which is good for a default test step. Maybe in the long run we should be able to tell the bot to test, or not to test. That way, if we are still debugging builds, we don't have to run the test step (yet). |
|
Something is going wrong, I see a CUDA and PSM2 as missing installations |
|
PSM2 is coming from easybuilders/easybuild-easyconfigs#20501 which has a build dependency on CUDA (https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/p/PSM2/PSM2-12.0.1-GCCcore-12.2.0.eb#L26) |
|
@casparvl This now requires a sync with the default branch for CI to pass |
|
I'm not convinced we should go ahead and merge this, seems like something we'll easily forget to revert later, and the time needed to run the test suite currently isn't limiting at all imho... |
|
No longer needed, we now have test selection possibilities in #673 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the test suite is growing, the test step would start taking longer and longer. This is particularly annoying if you want to do small tweaks during a build.
In the (near) future, we should make some mapping to determine which tests to run for which software installations. E.g. if your tarball contains a new TensorFlow module, you probably want to run the TensorFlow test (
-n TensorFlow). If your tarball contains OpenMPI, you probably want to runOSUand maybe one MPI-based application (i.e.-n OSU -n GROMACS.*fossfor example).