Add instrumentation test to the LTS skip list (#1999)

Fixes #1953.
intel · Aug 23, 2024 · ec62055 · ec62055
1 parent 87e5c06
commit ec62055
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.github/workflows/build-test-reusable.yml b/.github/workflows/build-test-reusable.yml
@@ -192,6 +192,13 @@ jobs:
         run: |
           source ./scripts/pytest-utils.sh
 
+          # FIXME: the "instrumentation" test suite currently contains only one test, when all tests
+          # are skipped pytest reports an error. If the only test is the skip list, then we shouldn't
+          # run pytest at all. This must be changed when there is more than one instrumentation test.
+          if [[ $TEST_UNSKIP = false && -s $TRITON_TEST_SKIPLIST_DIR/instrumentation.txt ]]; then
+            exit 0
+          fi
+
           SHARED_LIB_DIR="${GITHUB_WORKSPACE}/python/build/$(ls python/build | grep -i lib)/triton/_C"
           if [ ! -d "${SHARED_LIB_DIR}" ]; then
             echo "Could not find '${SHARED_LIB_DIR}'" ; exit -1

diff --git a/scripts/skiplist/lts/instrumentation.txt b/scripts/skiplist/lts/instrumentation.txt
@@ -0,0 +1,2 @@
+# https://github.com/intel/intel-xpu-backend-for-triton/issues/1953
+test/unit/instrumentation/test_gpuhello.py::test_op
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		# https://github.com/intel/intel-xpu-backend-for-triton/issues/1953
		test/unit/instrumentation/test_gpuhello.py::test_op