Skip to content

Commit 643b55a

Browse files
committed
Fix compilation tests find commands
Signed-off-by: ilmarkov <markovilya197@gmail.com>
1 parent 2627832 commit 643b55a

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.buildkite/test-pipeline.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -468,7 +468,9 @@ steps:
468468
# tests covered elsewhere.
469469
# Use `find` to launch multiple instances of pytest so that
470470
# they do not suffer from https://github.com/vllm-project/vllm/issues/28965
471-
- "find compile/ -maxdepth 1 -name 'test_*.py' -exec pytest -s -v {} \\\\;"
471+
# However, find does not normally propagate error codes, so we combine it with xargs
472+
# (using -0 for proper path handling)
473+
- "find compile/ -maxdepth 1 -name 'test_*.py' -print0 | xargs -0 -n1 -I{} pytest -s -v '{}'"
472474

473475
- label: PyTorch Fullgraph Smoke Test # 15min
474476
timeout_in_minutes: 30
@@ -482,7 +484,9 @@ steps:
482484
# as it is a heavy test that is covered in other steps.
483485
# Use `find` to launch multiple instances of pytest so that
484486
# they do not suffer from https://github.com/vllm-project/vllm/issues/28965
485-
- "find compile/fullgraph/ -name 'test_*.py' -not -name 'test_full_graph.py' -exec pytest -s -v {} \\\\;"
487+
# However, find does not normally propagate error codes, so we combine it with xargs
488+
# (using -0 for proper path handling)
489+
- "find compile/fullgraph -maxdepth 1 -name 'test_*.py' -not -name 'test_full_graph.py' -print0 | xargs -0 -n1 -I{} pytest -s -v '{}'"
486490

487491
- label: PyTorch Fullgraph Test # 27min
488492
timeout_in_minutes: 40

0 commit comments

Comments
 (0)