Skip to content

Commit

Permalink
CI: skip tests marked "cufile" on cuda 11.8 and arm64
Browse files Browse the repository at this point in the history
  • Loading branch information
madsbk committed Oct 25, 2024
1 parent 4b8181c commit 2ad401f
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 2 deletions.
9 changes: 8 additions & 1 deletion ci/run_pytests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,11 @@ set -euo pipefail
# Support invoking run_pytests.sh outside the script directory
cd "$(dirname "$(realpath "${BASH_SOURCE[0]}")")"/../python/kvikio

pytest --cache-clear --verbose "$@" tests
# If running CUDA 11.8 on arm64, we skip tests marked "cufile".
# cuFile didn't support arm until 12.4
PYTEST_MARK=$(
[[ "${CUDA_VERSION}" == "11.8.0" && "${NVARCH}" == "sbsa" ]] \
&& echo "-m 'not cufile'" || echo " "
)

python -m pytest ${PYTEST_MARK} --cache-clear --verbose "$@" tests
9 changes: 8 additions & 1 deletion ci/test_wheel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,11 @@ RAPIDS_PY_WHEEL_NAME="kvikio_${RAPIDS_PY_CUDA_SUFFIX}" rapids-download-wheels-fr

python -m pip install "$(echo ${WHEELHOUSE}/kvikio_${RAPIDS_PY_CUDA_SUFFIX}*.whl)[test]"

python -m pytest ./python/kvikio/tests
# If running CUDA 11.8 on arm64, we skip tests marked "cufile".
# cuFile didn't support arm until 12.4
PYTEST_MARK=$(
[[ "${CUDA_VERSION}" == "11.8.0" && "${NVARCH}" == "sbsa" ]] \
&& echo "-m 'not cufile'" || echo " "
)

python -m pytest ${PYTEST_MARK} ./python/kvikio/tests
1 change: 1 addition & 0 deletions cpp/include/kvikio/shim/libcurl.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -224,6 +224,7 @@ class CurlHandle {
} else {
ss << "(" << msg << ")";
}
std::cout << "perform() error: " << ss.str() << " (ERROR END)" << std::endl;
throw std::runtime_error(ss.str());
}
}
Expand Down

0 comments on commit 2ad401f

Please sign in to comment.