Skip to content

Commit ae2f943

Browse files
committed
TSAN CI, make jax buid/test step fail if missing deps wheels
1 parent a0af34f commit ae2f943

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/tsan.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,7 @@ jobs:
116116
- name: Build TSAN Numpy wheel
117117
if: steps.cache-numpy-tsan-restore.outputs.cache-hit != 'true'
118118
run: |
119+
set -eux
119120
cd numpy
120121
121122
# If we restored cpython from cache, we need to get python interpreter from python-tsan.tgz
@@ -131,7 +132,6 @@ jobs:
131132
export PATH=${GITHUB_WORKSPACE}/cpython-tsan/bin/:$PATH
132133
133134
python3 -m pip install uv~=0.5.30
134-
135135
python3 -m uv pip install -r requirements/build_requirements.txt
136136
137137
CC=clang-18 CXX=clang++-18 python3 -m pip wheel --wheel-dir dist -v . --no-build-isolation -Csetup-args=-Db_sanitize=thread -Csetup-args=-Dbuildtype=debugoptimized
@@ -268,11 +268,15 @@ jobs:
268268
--bazel_options=--copt=-g \
269269
--clang_path=/usr/bin/clang-18
270270
271-
272271
mkdir -p dist
272+
# Check whether we have numpy wheel or exit with error
273+
ls ${GITHUB_WORKSPACE}/wheelhouse/numpy/*.whl || exit 1
273274
cp -v ${GITHUB_WORKSPACE}/wheelhouse/numpy/*.whl dist/
274-
cp -v ${GITHUB_WORKSPACE}/wheelhouse/scipy/*.whl dist/
275275
if [ "${{ matrix.python-version }}" == "3.14" ]; then
276+
# Check whether we have scipy wheel or exit with error
277+
ls ${GITHUB_WORKSPACE}/wheelhouse/scipy/*.whl || exit 1
278+
cp -v ${GITHUB_WORKSPACE}/wheelhouse/scipy/*.whl dist/
279+
276280
# Patch build/requirements_lock_3_14_ft.txt to use TSAN instrumented NumPy and Scipy
277281
sed -i "s|--extra-index-url.*|--extra-index-url file://${GITHUB_WORKSPACE}/wheelhouse/|" build/${{ matrix.requirements_lock_name }}.txt
278282

0 commit comments

Comments
 (0)