Skip to content

Conversation

jgphpc
Copy link
Collaborator

@jgphpc jgphpc commented Aug 27, 2025

UENV=prgenv-gnu/25.06:rc5 ./R -c cuda_nvml.py -r
UENV=gromacs/2024:v2 ./R -c cuda_nvml.py -r
  • You can manually trigger 1 (or more) CI pipelines by adding a cscs-ci run alps-daint-uenv comment in the Pull Request
  • By default all UENVs will be tested, however you can restrict to a single UENV with: cscs-ci run alps-daint-uenv;MY_UENV=cp2k/2025.1:v2

Thank you for taking the time to contribute to cscs-reframe-tests !

@jgphpc jgphpc requested review from Copilot and gppezzi August 27, 2025 13:47
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CUDA NVML test configuration to fix compatibility issues with different programming environments and UENV configurations. The changes align environment features and improve the test's ability to locate CUDA installations across different setups.

  • Added 'cpe' and 'ce' features to programming environments for proper test filtering
  • Enhanced CUDA path detection logic to work with both CUDA_HOME and UENV configurations
  • Updated valid programming environment filters for both CPE and UENV test variants

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
config/systems/daint.py Adds 'cpe' and 'ce' feature tags to programming environments
checks/prgenv/cuda_nvml.py Updates environment filters and improves CUDA path detection logic

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jgphpc jgphpc changed the title Update cuda nvml test Update cuda tests Aug 28, 2025
@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 1, 2025

cscs-ci run alps-daint-uenv;MY_UENV=prgenv-gnu/25.06:rc5

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 1, 2025

cscs-ci run alps-daint-cpe

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-uenv;MY_UENV=prgenv-gnu/25.06:rc5

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-uenv;MY_UENV=prgenv-gnu/25.06:rc5

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-cpe

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-cpe

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-uenv;MY_UENV=prgenv-gnu/25.06:rc5

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-uenv;MY_UENV=prgenv-gnu/25.06:rc5

@jgphpc
Copy link
Collaborator Author

jgphpc commented Sep 2, 2025

cscs-ci run alps-daint-cpe

@gppezzi gppezzi changed the title Update cuda tests Update cuda tests and fix CI Sep 2, 2025
@jgphpc jgphpc merged commit 1a6cddc into eth-cscs:main Sep 2, 2025
2 of 3 checks passed
@jgphpc jgphpc deleted the test/nvml branch September 2, 2025 11:30
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