Skip to content

Commit 7c560eb

Browse files
authored
Merge pull request #36 from casparvl/fix_test_accelerator_override
Fix test accelerator override
2 parents 66b8ffc + 0a03f18 commit 7c560eb

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

bot/test.sh

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,14 @@ export EESSI_SOFTWARE_SUBDIR_OVERRIDE
175175
echo "bot/test.sh: EESSI_SOFTWARE_SUBDIR_OVERRIDE='${EESSI_SOFTWARE_SUBDIR_OVERRIDE}'"
176176

177177
# determine accelerator target (if any) from .architecture in ${JOB_CFG_FILE}
178-
export EESSI_ACCELERATOR_TARGET=$(cfg_get_value "architecture" "accelerator")
179-
echo "bot/test.sh: EESSI_ACCELERATOR_TARGET='${EESSI_ACCELERATOR_TARGET}'"
178+
ACCEL_OVERRIDE=$(cfg_get_value "architecture" "accelerator")
179+
if [[ -n "$ACCEL_OVERRIDE" ]]; then
180+
# bot job config does not include accel subdirectory
181+
export EESSI_ACCELERATOR_TARGET_OVERRIDE="accel/$ACCEL_OVERRIDE"
182+
else
183+
export EESSI_ACCELERATOR_TARGET_OVERRIDE=""
184+
fi
185+
echo "bot/test.sh: EESSI_ACCELERATOR_TARGET_OVERRIDE='${EESSI_ACCELERATOR_TARGET_OVERRIDE}'"
180186

181187
# get EESSI_OS_TYPE from .architecture.os_type in ${JOB_CFG_FILE} (default: linux)
182188
EESSI_OS_TYPE=$(cfg_get_value "architecture" "os_type")

test_suite.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,8 +155,8 @@ export RFM_PREFIX=$PWD/reframe_runs
155155

156156
# Get the correct partition name
157157
REFRAME_PARTITION_NAME=${EESSI_SOFTWARE_SUBDIR//\//_}
158-
if [ ! -z "$EESSI_ACCELERATOR_TARGET" ]; then
159-
REFRAME_PARTITION_NAME=${REFRAME_PARTITION_NAME}_${EESSI_ACCELERATOR_TARGET//\//_}
158+
if [ ! -z "$EESSI_ACCELERATOR_TARGET_OVERRIDE" ]; then
159+
REFRAME_PARTITION_NAME=${REFRAME_PARTITION_NAME}_${EESSI_ACCELERATOR_TARGET_OVERRIDE//\//_}
160160
fi
161161
echo "Constructed partition name based on EESSI_SOFTWARE_SUBDIR and EESSI_ACCELERATOR_TARGET: ${REFRAME_PARTITION_NAME}"
162162

0 commit comments

Comments
 (0)