traderepublic / Cilicon

🛠️ Self-Hosted ephemeral macOS CI on Apple Silicon
MIT License
950 stars 28 forks source link

Improved Downloads #32

Closed ast3150 closed 9 months ago

ast3150 commented 1 year ago

Summary

Reason

Solution

Context

Closes #31

Work In Progress

Marcocanc commented 9 months ago

Hi @ast3150, first of all I really appreciate the contribution! A few things have changed since the latest tart release though, which we might be able to leverage to implement a cleaner solution to accelerating download speeds. (Blog post). I will look into the detailed changes and see if we can reuse this solution or find a simpler one. In any case, I would like to separate the download and unzipping logic into a separate class.

Marcocanc commented 9 months ago

Tart's implementation even supports resuming and we will have to implement support for this anyway, as the Sonoma images are only using V2 disk layers.

https://github.com/cirruslabs/tart/blob/4bf9bdd531e955f1f49794a62701714a09952a4d/Sources/tart/OCI/Layerizer/DiskV2.swift#L35

Marcocanc commented 9 months ago

Update: I started working on splitting the download/unzipping logic out and building a tart-inspired resumable concurrent download solution. Will close this ticket for now.