Skip to content

Commit 596660d

Browse files
committed
fixup! Add 502/503/504 to the default request retry list
1 parent 084097b commit 596660d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/rest/test_job.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -353,8 +353,8 @@ def test_start_and_wait_with_error_require_success(dummy_backend, require_succes
353353
pytest.raises(OpenEoApiPlainError, match=re.escape("[500] Internal Server Error")),
354354
[23],
355355
),
356-
( # Default config with a 503 error (skipped by soft error feature of execute_batch poll loop)
357-
None,
356+
( # Only retry by default on 429, but still handle a 503 error with soft error skipping feature of execute_batch poll loop
357+
{"status_forcelist": [HTTP_429_TOO_MANY_REQUESTS]},
358358
[httpretty.Response(status=HTTP_503_SERVICE_UNAVAILABLE, body="Service Unavailable")],
359359
contextlib.nullcontext(),
360360
[23, 12.34, 34],

0 commit comments

Comments
 (0)