Skip to content

Make EESSI-extend support accelerator installations #27

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.

Sign up for GitHub

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

Merged
merged 19 commits into from
Jul 7, 2025
Merged

Make EESSI-extend support accelerator installations #27

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
47290c5
Make EESSI-extend support accelerator installations
ocaisa Jul 3, 2025
f551ffe
Explicitly export environment variable
ocaisa Jul 3, 2025
889480a
Missing quote
ocaisa Jul 3, 2025
c5e8830
Separate our the GPU checks
ocaisa Jul 3, 2025
132ab4d
Make sure to source shell functions
ocaisa Jul 3, 2025
a72a5e1
Match to accel/nvidia/ccXX in EESSI-extend
ocaisa Jul 3, 2025
bb5faa1
EasyBuild expects . in CUDA compute capabilities
ocaisa Jul 3, 2025
dd4238d
accel is included in EESSI_ACCELERATOR_TARGET
ocaisa Jul 3, 2025
5ed221d
typo
ocaisa Jul 3, 2025
540e088
Extra subdirectory removed for accelerator subdir
ocaisa Jul 3, 2025
2416487
MAke sure the bot sets the right environment variables
ocaisa Jul 3, 2025
635ff46
Ensure build scripts are aware of new EESSI-extend behaviour
ocaisa Jul 4, 2025
8a6ce0f
Merge remote-tracking branch 'upstream/main' into eessi-extend-cuda
ocaisa Jul 4, 2025
996f065
Update EESSI-install-software.sh
ocaisa Jul 4, 2025
b9cdb1a
Update EESSI-install-software.sh
ocaisa Jul 4, 2025
d4ceacc
Update EESSI-install-software.sh
ocaisa Jul 4, 2025
baae956
Be more careful when setting EESSI_ACCELERATOR_TARGET_OVERRIDE based …
ocaisa Jul 4, 2025
787dd5e
Merge branch 'eessi-extend-cuda' of github.com:ocaisa/software-layer-…
ocaisa Jul 4, 2025
545f5d9
Update bot/build.sh
ocaisa Jul 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
accel is included in EESSI_ACCELERATOR_TARGET
  • Loading branch information
@ocaisa
ocaisa committed Jul 3, 2025
commit dd4238d386144e988c21dd4ab836f8d7b705296d
4 changes: 2 additions & 2 deletions .github/workflows/tests_eessi_extend_module.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ jobs:
check_env_var "EASYBUILD_CUDA_COMPUTE_CAPABILITIES" "$STORED_CUDA_CC"
export EESSI_ACCELERATOR_INSTALL=1
Copy link
Member Author

Choose a reason for hiding this comment

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

It is this variable (EESSI_ACCELERATOR_INSTALL) and EESSI_ACCELERATOR_TARGET_OVERRIDE that need to be set by the bot in order to configure EESSI-extend correctly for a GPU installation

module load EESSI-extend/${{matrix.EESSI_VERSION}}-easybuild # reload for an actual GPU installation
check_env_var "EASYBUILD_INSTALLPATH" "${EESSI_SOFTWARE_PATH}/accel/${EESSI_ACCELERATOR_TARGET_OVERRIDE}"
check_env_var "EASYBUILD_INSTALLPATH" "${EESSI_SOFTWARE_PATH}/${EESSI_ACCELERATOR_TARGET_OVERRIDE}"
# unload and make sure the environment is clean again
module unload EESSI-extend
check_disallowed_env_prefix EASYBUILD_
Expand All @@ -169,7 +169,7 @@ jobs:
check_env_var "EASYBUILD_CUDA_COMPUTE_CAPABILITIES" "$STORED_CUDA_CC"
export EESSI_ACCELERATOR_INSTALL=1
module load EESSI-extend/${{matrix.EESSI_VERSION}}-easybuild # reload for an actual GPU installation
check_env_var "EASYBUILD_INSTALLPATH" "${EESSI_SITE_SOFTWARE_PATH}/accel/${EESSI_ACCELERATOR_TARGET_OVERRIDE}"
check_env_var "EASYBUILD_INSTALLPATH" "${EESSI_SITE_SOFTWARE_PATH}/${EESSI_ACCELERATOR_TARGET_OVERRIDE}"
# unload and make sure the environment is clean again
module unload EESSI-extend
check_disallowed_env_prefix EASYBUILD_
Expand Down