[router] Fix flaky router e2e tests (#13306)
This commit is contained in:
@@ -112,6 +112,9 @@ def test_circuit_breaker_disable_flag(router_manager, mock_workers):
|
|||||||
"disable_retries": True,
|
"disable_retries": True,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
|
||||||
|
saw_500 = False
|
||||||
|
for _ in range(8):
|
||||||
r = requests.post(
|
r = requests.post(
|
||||||
f"{rh.url}/v1/completions",
|
f"{rh.url}/v1/completions",
|
||||||
json={
|
json={
|
||||||
@@ -122,7 +125,15 @@ def test_circuit_breaker_disable_flag(router_manager, mock_workers):
|
|||||||
},
|
},
|
||||||
timeout=3,
|
timeout=3,
|
||||||
)
|
)
|
||||||
assert r.status_code == 500
|
if r.status_code == 500:
|
||||||
|
# Worker starts, continue to check
|
||||||
|
saw_500 = True
|
||||||
|
break
|
||||||
|
assert (
|
||||||
|
r.status_code == 503
|
||||||
|
), "Should only see 503 when waiting for worker to start"
|
||||||
|
|
||||||
|
assert saw_500
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.integration
|
@pytest.mark.integration
|
||||||
|
|||||||
@@ -47,6 +47,8 @@ def test_disable_retries_surfaces_failure(router_manager, mock_workers):
|
|||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
|
||||||
|
saw_500 = False
|
||||||
|
for _ in range(8):
|
||||||
r = requests.post(
|
r = requests.post(
|
||||||
f"{rh.url}/v1/completions",
|
f"{rh.url}/v1/completions",
|
||||||
json={
|
json={
|
||||||
@@ -57,5 +59,13 @@ def test_disable_retries_surfaces_failure(router_manager, mock_workers):
|
|||||||
},
|
},
|
||||||
timeout=5,
|
timeout=5,
|
||||||
)
|
)
|
||||||
assert r.status_code == 500
|
if r.status_code == 500:
|
||||||
|
# Worker starts, continue to check
|
||||||
|
saw_500 = True
|
||||||
|
break
|
||||||
|
assert (
|
||||||
|
r.status_code == 503
|
||||||
|
), "Should only see 503 when waiting for worker to start"
|
||||||
|
|
||||||
|
assert saw_500
|
||||||
# mock_workers fixture handles cleanup
|
# mock_workers fixture handles cleanup
|
||||||
|
|||||||
Reference in New Issue
Block a user