Merge branch 'fix/gitlab_404_local_mirror' into 'master'

Handle gitlab 404 error

See merge request espressif/esp-idf!16818
This commit is contained in:
Fu Hanxi
2022-01-20 01:17:16 +00:00

View File

@@ -25,9 +25,16 @@ def retry(func: TR) -> TR:
try: try:
res = func(self, *args, **kwargs) res = func(self, *args, **kwargs)
except (IOError, EOFError, gitlab.exceptions.GitlabError) as e: except (IOError, EOFError, gitlab.exceptions.GitlabError) as e:
if isinstance(e, gitlab.exceptions.GitlabError) and e.response_code != 500: if isinstance(e, gitlab.exceptions.GitlabError):
# Only retry on error 500 if e.response_code == 500:
raise e # retry on this error
pass
elif e.response_code == 404:
# remove the environment variable "LOCAL_GITLAB_HTTPS_HOST" and retry
os.environ.pop('LOCAL_GITLAB_HTTPS_HOST', None)
else:
# other GitlabErrors aren't retried
raise e
retried += 1 retried += 1
if retried > self.DOWNLOAD_ERROR_MAX_RETRIES: if retried > self.DOWNLOAD_ERROR_MAX_RETRIES:
raise e # get out of the loop raise e # get out of the loop