Skip to content

Commit f8e1633

Browse files
committed
Revert "[CI][Diag] Move dep-manifest dump to after pytest"
This reverts commit 51cde76.
1 parent c2d7833 commit f8e1633

1 file changed

Lines changed: 2 additions & 11 deletions

File tree

.github/actions/run-tests/action.yml

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -228,16 +228,6 @@ runs:
228228
;;
229229
esac
230230
fi
231-
echo 'Starting pytest with path: $test_path'
232-
# Run pytest under a saved exit code so the dep-manifest diagnostic
233-
# below always emits, even on test failure. The diagnostic must run
234-
# AFTER pytest: importing numpy in this shell beforehand registers
235-
# numpy's vendored OpenBLAS pthread_atfork handler in the wrong
236-
# process, which trips Kit's libomni.platforminfo fork() and SIGSEGVs.
237-
set +e
238-
./isaaclab.sh -p -m pytest --ignore=tools/conftest.py --ignore=source/isaaclab/test/install_ci $test_path $pytest_options -v --junitxml=tests/$result_file
239-
PYTEST_EXIT=\$?
240-
set -e
241231
# Diagnostic: dump numpy/scipy/openblas state so we can verify the
242232
# numpy!=2.3.5 pin landed in IsaacLab's setup.py prevents the broken
243233
# vendored OpenBLAS (libscipy_openblas64_-fdde5778.so) from being
@@ -248,7 +238,8 @@ runs:
248238
./isaaclab.sh -p -m pip show numpy scipy 2>/dev/null | grep -E '^(Name|Version|Location):' || true
249239
./isaaclab.sh -p -c \"import numpy, scipy, os; print('numpy', numpy.__version__); print('scipy', scipy.__version__); [print('bundled openblas:', os.path.join(d, f)) for d in [os.path.dirname(numpy.__file__) + '/../numpy.libs', os.path.dirname(scipy.__file__) + '/../scipy.libs'] if os.path.isdir(d) for f in os.listdir(d) if 'openblas' in f.lower()]\" 2>/dev/null || true
250240
echo '=== /Dep manifest ==='
251-
exit \$PYTEST_EXIT
241+
echo 'Starting pytest with path: $test_path'
242+
./isaaclab.sh -p -m pytest --ignore=tools/conftest.py --ignore=source/isaaclab/test/install_ci $test_path $pytest_options -v --junitxml=tests/$result_file
252243
"
253244
254245
# Stream container logs in background.

0 commit comments

Comments
 (0)