refactor: Moving RETRYABLE_TRANSIENT_ERROR_CODES to const

Liora Milbaum · nejch · commit 887852d7ef02 · 2022-12-06T09:53:48.000+01:00
diff --git a/gitlab/client.py b/gitlab/client.py
@@ -22,7 +22,6 @@
     "{source!r} to {target!r}"
 )
 
-RETRYABLE_TRANSIENT_ERROR_CODES = [500, 502, 503, 504] + list(range(520, 531))
 
 # https://docs.gitlab.com/ee/api/#offset-based-pagination
 _PAGINATION_URL = (
@@ -781,7 +780,7 @@ def http_request(
                 return result
 
             if (429 == result.status_code and obey_rate_limit) or (
-                result.status_code in RETRYABLE_TRANSIENT_ERROR_CODES
+                result.status_code in gitlab.const.RETRYABLE_TRANSIENT_ERROR_CODES
                 and retry_transient_errors
             ):
                 # Response headers documentation:
diff --git a/gitlab/const.py b/gitlab/const.py
@@ -131,6 +131,8 @@ class SearchScope(GitlabEnum):
 
 USER_AGENT: str = f"{__title__}/{__version__}"
 
+RETRYABLE_TRANSIENT_ERROR_CODES = [500, 502, 503, 504] + list(range(520, 531))
+
 __all__ = [
     "AccessLevel",
     "Visibility",
diff --git a/tests/unit/test_gitlab_http_methods.py b/tests/unit/test_gitlab_http_methods.py
@@ -6,7 +6,7 @@
 import responses
 
 from gitlab import GitlabHttpError, GitlabList, GitlabParsingError, RedirectError
-from gitlab.client import RETRYABLE_TRANSIENT_ERROR_CODES
+from gitlab.const import RETRYABLE_TRANSIENT_ERROR_CODES
 from tests.unit import helpers
 
 
