Skip to content

Commit 0ea1970

Browse files
committed
chore: drop test config churn
Signed-off-by: Ryan S <267728323+ironcommit@users.noreply.github.com>
1 parent f60e6c1 commit 0ea1970

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

Makefile

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -384,7 +384,7 @@ PYTEST_WORKERS ?= auto
384384
PYTEST_MAX_WORKERS ?= 16
385385
PYTEST_DIST ?= loadscope
386386
PYTEST_MAX_WORKER_RESTART ?= 2
387-
PYTEST_CMD = env PYTHONFAULTHANDLER=1 PYTHONWARNINGS="ignore::UserWarning:pytest_only.version" $(UV) run --frozen \
387+
PYTEST_CMD = env PYTHONWARNINGS="ignore::UserWarning:pytest_only.version" $(UV) run --frozen \
388388
pytest \
389389
-n $(PYTEST_WORKERS) --maxprocesses=$(PYTEST_MAX_WORKERS) \
390390
--max-worker-restart=$(PYTEST_MAX_WORKER_RESTART) \
@@ -411,10 +411,9 @@ PYTEST_CI_CMD = timeout --kill-after=60s $(PYTEST_CI_TIMEOUT)s $(PYTEST_CMD)
411411
# ``loadscope``.
412412
test-integration test-integration-ci: PYTEST_DIST := loadgroup
413413

414-
# In CI, a crashed xdist worker should fail the job immediately. Restarting the
415-
# worker can hide the crashing test and leave pytest waiting until the wrapper
416-
# timeout kills the process.
417-
test-unit-ci test-integration-ci test-gpu-integration-ci: PYTEST_MAX_WORKER_RESTART := 0
414+
# A killed integration worker never tears down its containers and ports, so replacing it re-runs
415+
# the group against resources the dead worker still holds. Unit workers own nothing external.
416+
test-integration test-integration-ci: PYTEST_MAX_WORKER_RESTART := 0
418417

419418
.PHONY: test-unit
420419
test-unit: ## Run Python unit tests across all packages and services

0 commit comments

Comments
 (0)