ci: bound apt and wget operations - #130
Conversation
|
@marcusr2ML @Ooolab Do you have time to take a look at this? the changes are very contained and near-trivial -- just some simple timeout guards |
Independent verificationI dug through CI history to confirm the problem and tested the fix in an Ubuntu 24.04 container plus against the real endpoints. The problem is real, and the solution works. Details below, plus two optional suggestions. Problem confirmed from CI history
Solution tested
Two optional suggestions
Neither suggestion blocks this PR — as-is it already converts the observed 6-hour hangs into ≤20-minute failures. 🤖 Generated with Claude Code |
Ooolab
left a comment
There was a problem hiding this comment.
I had the timeout problem previously. Glad to see it addressed by this PR!
The suggestions are just for other potential causes of the similar problem. We can address them in the future in another PR if they do happen.
Problem addressed
Occasionally CI jobs fail due to network timeouts (on apt update and wget). When it happens the CI runs to its maximum of 6 hours before the job gets killed, hung on that step. This PR adds timeout and retry guards, such that these jobs can succeed or fail within minutes with a clear error message.
Summary
Validation
actionlint -ignore shellcheck .github/workflows/build.yml .github/workflows/build_wheels.ymlgit diff --check