From 44d0a848da6b98d49444fc867a0d706144acf9f1 Mon Sep 17 00:00:00 2001 From: Xinyue Zhang Date: Wed, 26 Nov 2025 14:52:10 -0800 Subject: [PATCH] [model-gateway] Fix flaky test_circuit_breaker_half_open_failure_reopens (#14019) --- .../integration_mock/test_circuit_breaker.py | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/sgl-router/py_test/integration_mock/test_circuit_breaker.py b/sgl-router/py_test/integration_mock/test_circuit_breaker.py index 01e1213c7..16c619924 100644 --- a/sgl-router/py_test/integration_mock/test_circuit_breaker.py +++ b/sgl-router/py_test/integration_mock/test_circuit_breaker.py @@ -86,6 +86,19 @@ def test_circuit_breaker_half_open_failure_reopens(router_manager, mock_workers) timeout=3, ) + # should see 500 when worker actually starts, before that should see 503 + saw_500 = False + for _ in range(8): + r = post_once() + if r.status_code == 500: + # Worker starts, continue to circuit breaker test + saw_500 = True + break + assert ( + r.status_code == 503 + ), "Should only see 503 when waiting for worker to start" + assert saw_500, "Worker didn't start after 8 requests" + opened = False for _ in range(8): r = post_once()