Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] retries for copy operations #284

Open
gnoling opened this issue Aug 5, 2024 · 2 comments
Open

[feature] retries for copy operations #284

gnoling opened this issue Aug 5, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request priority/review size/L Denotes an issue/PR requiring a relatively large amount of work
Milestone

Comments

@gnoling
Copy link

gnoling commented Aug 5, 2024

Is this RFE related to an Existing Problem? If so, please describe:

  • In unstable networks, there is no retry mechanism when pushing or pulling images.

Describe Proposed Solution(s):

  • They would like the ability to set retries within Hauler (image level retries, not manifest level retries). Something like --retries 3 for when an image fails to pull/push.

Describe Possible Alternatives:

  • Currently the work-around is looping hauler until it completes successfully.

Additional Context:

  • There are times when customers are down range and networks are unstable when pulling/pushing images.
@gnoling gnoling added the enhancement New feature or request label Aug 5, 2024
@github-project-automation github-project-automation bot moved this to Pending Review in Hauler Aug 5, 2024
@amartin120
Copy link
Contributor

amartin120 commented Aug 8, 2024

Currently hauler store sync will retry things up to 3 times. It's just currently not flexible as to the "number" of retries. That's probably something that could be done though. I think the original thought was, if we tried 3 times and it still fails... it's just not going.

hauler store copy is currently the exception as it just passes a directory to cosign load and says... go do it. This will take some effort in our cosign fork if we want to include retries there.

@dweomer dweomer changed the title [feature] Add retries to Hauler for unstable networks [feature] retries for copy operations Oct 8, 2024
@zackbradys zackbradys self-assigned this Nov 21, 2024
@zackbradys zackbradys moved this from To Triage to In Progress in Hauler Nov 21, 2024
@zackbradys zackbradys added this to the Hauler v1.1-next1 milestone Nov 21, 2024
@zackbradys
Copy link
Member

partially similar to Issue #323...

@zackbradys zackbradys pinned this issue Dec 9, 2024
@CamrynCarter CamrynCarter moved this from In Progress to Backlog in Hauler Dec 17, 2024
@zackbradys zackbradys added the size/L Denotes an issue/PR requiring a relatively large amount of work label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority/review size/L Denotes an issue/PR requiring a relatively large amount of work
Projects
Status: Backlog
Development

No branches or pull requests

5 participants