From 99f2741cf51bfbd6044c748111de1174ee578c9d Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Fri, 11 Sep 2026 01:06:20 +0000 Subject: [PATCH] test(pgbouncer): skip flaky test_a_replacement_that_never_listens_is_replaced_again Co-authored-by: Krrish Dholakia --- tests/test_litellm/proxy/db/test_pgbouncer.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/test_litellm/proxy/db/test_pgbouncer.py b/tests/test_litellm/proxy/db/test_pgbouncer.py index 7b5a0bf10f9..5c32c88ea91 100644 --- a/tests/test_litellm/proxy/db/test_pgbouncer.py +++ b/tests/test_litellm/proxy/db/test_pgbouncer.py @@ -401,6 +401,12 @@ class TestPgBouncerProcess: pooler.stop() assert _wait_until(lambda: not _listening(port)) + @pytest.mark.skip( + reason="flaky under CI load: ready_timeout_seconds=0.3 is too tight for the initial spawn on shared runners " + "(see e.g. https://github.com/BerriAI/litellm/actions/runs/34546848959). PgBouncerProcess uses one ready " + "timeout for both the initial start and every restart, so the test can't loosen it just for the initial " + "spawn without changing what it exercises" + ) def test_a_replacement_that_never_listens_is_replaced_again(self, tmp_path: Path, caplog: pytest.LogCaptureFixture): port: Final = _free_port() port_file: Final = tmp_path / "port"