Skip to content

Commit 672d228

Browse files
committed
ci: force-disable compute-sanitizer on PR matrix (debug only)
Wire the existing SETUP_SANITIZER gate behind a literal `false &&` so every PR test job exports SETUP_SANITIZER=0 and SANITIZER_CMD=, which makes run-tests invoke pytest directly instead of under compute-sanitizer. Pairs with the matrix trim in this branch to test the hypothesis that the TestIpcReexport hang on r595-driver runners is specific to running pytest under compute-sanitizer. Revert before merging.
1 parent 0092172 commit 672d228

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

ci/tools/env-vars

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,10 @@ elif [[ "${1}" == "test" ]]; then
8888
# We don't test compute-sanitizer on CTK<12 because backporting fixes is too much effort
8989
# We only test compute-sanitizer on python 3.12 arbitrarily; we don't need to use sanitizer on the entire matrix
9090
# Only local ctk installs have compute-sanitizer; there is no wheel for it
91-
if [[ "${PY_VER}" == "3.12" && "${CUDA_VER}" != "11.8.0" && "${LOCAL_CTK}" == 1 && "${HOST_PLATFORM}" == linux* ]]; then
91+
# TEMPORARY: force-disable compute-sanitizer to test whether it is the
92+
# cause of the TestIpcReexport hang on r595-driver runners. Revert before
93+
# merging.
94+
if false && [[ "${PY_VER}" == "3.12" && "${CUDA_VER}" != "11.8.0" && "${LOCAL_CTK}" == 1 && "${HOST_PLATFORM}" == linux* ]]; then
9295
echo "LATEST_CUDA_VERSION=$(bash ci/tools/guess_latest.sh $TEST_CUDA_MAJOR)" >> $GITHUB_ENV
9396
SETUP_SANITIZER=1
9497
else

0 commit comments

Comments
 (0)