Skip to content

convertor: Retry on docker registry upload failures#12

Merged
gautam-rl merged 2 commits into
mainfrom
gautam-retry-upload
Aug 8, 2025
Merged

convertor: Retry on docker registry upload failures#12
gautam-rl merged 2 commits into
mainfrom
gautam-retry-upload

Conversation

@gautam-rl

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:
Adds a --retry parameter to the convertor tool, default 5. This does exponential backoff with a factor of 2 i.e. 1, 2, 4... with random jitter.
This will help us deal with registry downtime/timeouts

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Please check the following list:

  • Does the affected code have corresponding tests, e.g. unit test, E2E test?
  • Does this change require a documentation update?
  • Does this introduce breaking changes that would require an announcement or bumping the major version?
  • Do all new files have an appropriate license header?

@gautam-rl
gautam-rl requested review from adam-rl and dines-rl August 8, 2025 20:58
@gautam-rl
gautam-rl merged commit 095e48d into main Aug 8, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants