ci: retry backup manager downloads with shell loop#6957
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Follow-up fix pushed in This commit adds explicit retry loops for the e2e image downloads too:
Validation:
I also attempted a local build of |
|
Another follow-up pushed in This commit keeps Validation:
|
|
@tennix: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Summary
rclonedownloads with an explicit shell loopshushdownloads with the same loopMotivation
PR #6919 e2e jobs failed while building
images/tidb-backup-manager/Dockerfile.e2e:wget --triesdid not cover this failure mode reliably in Prow, so this change wraps the download in an explicit retry loop.Test Plan
git diff --checkRUNshell snippets and validated them withsh -nDocker build was not run locally because Docker Desktop daemon is not running in this environment.