Skip to content

Commit 5b1766c

Browse files
committed
simplify the approach adding rw
1 parent abc4831 commit 5b1766c

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

.github/workflows/_ci.yaml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -336,14 +336,11 @@ jobs:
336336
EXECUTE: |
337337
set -o pipefail
338338
docker run -i --shm-size=1g --gpus all \
339-
--user $(id -u)$(id -g) \
340-
-e HOME=/tmp \
341-
-e PATH=/tmp/.local/bin:$PATH \
342-
-v $PWD:/opt/output \
339+
-v $PWD:/opt/output:rw \
343340
${{ needs.build-jax.outputs.DOCKER_TAG_FINAL }} \
344341
bash <<"EOF" |& tee test-nsys-jax.log
345342
# nsys-jax is already installed, this is just adding the test dependencies
346-
pip install --user pytest-reportlog nsys-jax[test]
343+
pip install pytest-reportlog nsys-jax[test]
347344
# abuse knowledge that nsys-jax is installed editable, so the tests exist
348345
test_path=$(python -c 'import importlib.resources; print(importlib.resources.files("nsys_jax").joinpath("..", "tests").resolve())')
349346
pytest --basetemp=/opt/output/pytest-tmp --report-log=/opt/output/pytest-report.jsonl "${test_path}"

0 commit comments

Comments
 (0)