Add 502/503/504 to the default request retry list

refs: #835, #441

Author: soxofaan

CHANGELOG.md

@@ -14,6 +14,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Internal reorganization of `openeo.extra.job_management` submodule to ease future development ([#741](https://github.com/Open-EO/openeo-python-client/issues/741))
+- `openeo.Connection`: add some more HTTP error codes to the default retry list: `502 Bad Gateway`, `503 Service Unavailable` and `504 Gateway Timeout` ([#835](https://github.com/Open-EO/openeo-python-client/issues/835))
+
 
 ### Removed
 

openeo/extra/job_management/_manager.py

@@ -231,7 +231,13 @@ def add_backend(
         parallel_jobs: int = 2,
     ):
         """
-        Register a backend with a name and a Connection getter.
+        Register a backend with a name and a :py:class:`Connection` getter.
+
+        .. note::
+           For optimal throughput and responsiveness, it is recommended
+           to provide a :py:class:`Connection` instance without its own (blocking) retry behavior,
+           since the job manager will do retries in a non-blocking way,
+           allowing to take care of other tasks before retrying failed requests.
 
         :param name:
             Name of the backend.

openeo/utils/http.py

@@ -43,6 +43,9 @@
 DEFAULT_RETRY_FORCELIST = frozenset(
     [
         HTTP_429_TOO_MANY_REQUESTS,
+        HTTP_502_BAD_GATEWAY,
+        HTTP_503_SERVICE_UNAVAILABLE,
+        HTTP_504_GATEWAY_TIMEOUT,
     ]
 )