Skip to content

Commit e898be6

Browse files
committed
Mark telemetry e2e tests as serial - must not run in parallel
Root cause: Telemetry tests share host-level client across pytest-xdist workers, causing test isolation issues with patches. Tests pass serially but fail with -n auto. Solution: Add @pytest.mark.serial marker. CI needs to run these separately without -n auto.
1 parent fb8e86d commit e898be6

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

pyproject.toml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,10 @@ exclude = ['ttypes\.py$', 'TCLIService\.py$']
6262
exclude = '/(\.eggs|\.git|\.hg|\.mypy_cache|\.nox|\.tox|\.venv|\.svn|_build|buck-out|build|dist|thrift_api)/'
6363

6464
[tool.pytest.ini_options]
65-
markers = {"reviewed" = "Test case has been reviewed by Databricks"}
65+
markers = [
66+
"reviewed: Test case has been reviewed by Databricks",
67+
"serial: Tests that must run serially (not parallelized)"
68+
]
6669
minversion = "6.0"
6770
log_cli = "false"
6871
log_cli_level = "INFO"

tests/e2e/test_telemetry_e2e.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,9 @@ def connection(self, extra_params=()):
4343
conn.close()
4444

4545

46+
@pytest.mark.serial
4647
class TestTelemetryE2E(TelemetryTestBase):
47-
"""E2E tests for telemetry scenarios"""
48+
"""E2E tests for telemetry scenarios - must run serially due to shared host-level telemetry client"""
4849

4950
@pytest.fixture(autouse=True)
5051
def telemetry_setup_teardown(self):

0 commit comments

Comments
 (0)