-
Notifications
You must be signed in to change notification settings - Fork 56
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
(RE-14819) Retry failed curls before trying to gunzip them #203
Conversation
Needs testing. |
} | ||
fetch_tarball_command = "cd #{host['working_dir']}; #{command_file_push}#{filename}#{extension}" | ||
retry_on(host, fetch_tarball_command, retry_requirements) | ||
on host, "cd #{host['working_dir']}; tar -xvf #{filename}#{extension}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only thing that comes to mind is the question of whether or not we can rely on tar being able to gunzip automagically on all platforms this method would execute on. But since I think this would only be used on pe master platforms, I don't think that's an issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went through exactly that same series of steps. I'm willing to fix it if I got it wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Looks like this is now blocked by #207 Will wait on 🟢 before merge. |
Please delete any headings that don't apply to this Pull Request (PR).