Skip to content

Commit 3e066ef

Browse files
author
Caspar van Leeuwen
committed
Use an EESSI_SITE_SOFTWARE_PREFIX for one of the CPU targets to check that this correctly becomes the starting prefix of EESSI_SITE_SOFTWARE_PATH
1 parent 2a3ace9 commit 3e066ef

2 files changed

Lines changed: 12 additions & 0 deletions

File tree

.github/workflows/scripts/verify_eessi_environment.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,11 @@ def check_env_endswith(var1, var2):
3232
if not val1.endswith(val2):
3333
raise EnvVarError(f"'{var1}' must end with '{var2}':\n{var1}='{val1}'\n{var2}='{val2}'")
3434

35+
def check_env_startswith(var1, var2):
36+
val1, val2 = get_env_vars(var1, var2)
37+
if not val1.startswith(val2):
38+
raise EnvVarError(f"'{var1}' must start with '{var2}':\n{var1}='{val1}'\n{var2}='{val2}'")
39+
3540
if __name__ == "__main__":
3641
try:
3742
# accelerator stuff is not guaranteed to exist
@@ -58,6 +63,10 @@ def check_env_endswith(var1, var2):
5863
check_env_contains("EESSI_SITE_MODULEPATH_ACCEL", "EESSI_SOFTWARE_SUBDIR")
5964
check_env_contains("EESSI_MODULEPATH_ACCEL", "EESSI_ACCELERATOR_TARGET")
6065
check_env_contains("EESSI_SITE_MODULEPATH_ACCEL", "EESSI_ACCELERATOR_TARGET")
66+
# Verify that configuring an EESSI_SITE_SOFTWARE_PREFIX results in this prefix being
67+
# the first part of EESSI_SITE_SOFTWARE_PATH
68+
if os.getenv("EESSI_SITE_SOFTWARE_PREFIX"):
69+
check_env_startswith("EESSI_SITE_SOFTWARE_PATH", "EESSI_SITE_SOFTWARE_PREFIX")
6170
# Finally, verify that all the expected module path are included
6271
check_env_contains("MODULEPATH", "EESSI_MODULEPATH")
6372
check_env_contains("MODULEPATH", "EESSI_SITE_MODULEPATH")

.github/workflows/tests_eessi_module.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,9 @@ jobs:
171171
FINAL_ACCELERATOR_TARGET_EXPECTED: accel/nvidia/cc80
172172
- EESSI_ACCELERATOR_TARGET_OVERRIDE: accel/nvidia/cc77 # deliberately chose a non-existent CUDA capability
173173
FINAL_ACCELERATOR_TARGET_EXPECTED: accel/nvidia/cc70 # this reverts to the fallback case (which does exist)
174+
# Test if setting an EESSI_SITE_SOFTWARE_PREFIX works by setting that for the zen4 target
175+
- EESSI_SOFTWARE_SUBDIR_OVERRIDE: x86_64/amd/zen4
176+
EESSI_SITE_SOFTWARE_PREFIX: /tmp/site_prefix
174177

175178
steps:
176179
- name: Check out software-layer repository

0 commit comments

Comments
 (0)