Skip to content

feat: use resuming downloads for file downloader#4415

Merged
NikolaMilosa merged 31 commits intomasterfrom
nm-cont-downloads
Mar 26, 2025
Merged

feat: use resuming downloads for file downloader#4415
NikolaMilosa merged 31 commits intomasterfrom
nm-cont-downloads

Conversation

@NikolaMilosa
Copy link
Contributor

To tackle problems with low bandwidth networks and connectivity issues we can try using resuming downloads.

@NikolaMilosa NikolaMilosa marked this pull request as ready for review March 20, 2025 11:58
@NikolaMilosa NikolaMilosa requested review from a team as code owners March 20, 2025 11:58
Copy link
Contributor

@kpop-dfinity kpop-dfinity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left some comments, mostly nits/style

@kpop-dfinity
Copy link
Contributor

About the PR title: I don't think refactor is an appropriate tag here, maybe we should use feat instead?

@github-actions github-actions bot added the feat label Mar 26, 2025
@NikolaMilosa NikolaMilosa added this pull request to the merge queue Mar 26, 2025
Merged via the queue into master with commit 3cdab83 Mar 26, 2025
20 checks passed
@NikolaMilosa NikolaMilosa deleted the nm-cont-downloads branch March 26, 2025 16:38
github-merge-queue bot pushed a commit that referenced this pull request Apr 15, 2025
This chore builds on top of #4415.

Previously we had two ways of downloading ic os images. This PR enforces
the use of `FileDownloader` utility that has support for resuming
downloads.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 feat

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants