ci(pacman-mirrors): switch chaotic mirror sources to CDN mirror and enable curl transfers for CI ISO builds #16
Conversation
…nable curl transfers This change improves CI ISO build reliability and speed by switching chaotic mirrors to the CDN mirror and enabling curl-based transfers in pacman to handle slow connections more gracefully. - Update root pacman.conf to use CDN chaotic.cx for chaotic-aur and garuda - Propagate mirror changes into airootfs/pacman.conf used during ISO build - Enable XferCommand = /usr/bin/curl -L -C - -f -o %o %u to bypass slow transfers - Improves CI build stability with faster, more reliable downloads
WalkthroughThe pull request modifies two pacman.conf files to uncomment the XferCommand option, enabling curl-based package downloads, and updates mirror URLs for chaotic-aur and garuda repositories from geo-mirror.chaotic.cx to cdn-mirror.chaotic.cx. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
pacman.conf (1)
18-20: Ensurecurlis available everywhere this pacman.conf is used, and consider retry flags.Uncommenting this XferCommand is fine, but pacman will now hard‑depend on
/usr/bin/curlin every environment (host, CI container, chroot, etc.) that uses this config. Please verifycurlis installed in all those contexts, otherwise downloads will fail instead of falling back to the internal downloader. For CI robustness, you may also want to add basic retry/timeout flags (e.g.--retry,--retry-delay,--connect-timeout).-XferCommand = /usr/bin/curl -L -C - -f -o %o %u +XferCommand = /usr/bin/curl -L -C - --retry 3 --retry-delay 5 --connect-timeout 15 -f -o %o %u
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
airootfs/etc/pacman.conf(2 hunks)pacman.conf(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build-and-upload
🔇 Additional comments (3)
pacman.conf (1)
102-108: Mirror host switch tocdn-mirror.chaotic.cxlooks good; just confirm it’s the desired CI endpoint.Updating both
chaotic-aurandgarudatohttps://cdn-mirror.chaotic.cx/$repo/$archmatches the stated CI goals and is consistent with the airootfs pacman.conf. Please double‑check that this CDN host is the one currently recommended by Chaotic/Garuda for automated builds, and keep in mind that any future mirror/XferCommand changes will need to be applied in both this file andairootfs/etc/pacman.confto avoid drift.airootfs/etc/pacman.conf (2)
18-20: Verifycurlis included in the ISO environment that uses this config.Since the live/airootfs pacman now relies on
/usr/bin/curlvia XferCommand, ensurecurlis part of the ISO’s package list (and any build chroots that read this file). If it’s missing, pacman inside the ISO will fail to download packages.
102-108: Runtime mirror configuration matches root pacman.conf – good for consistency.The switch to
cdn-mirror.chaotic.cxforchaotic-aurandgarudahere mirrors the top‑level pacman.conf, which should keep CI builds and the live system behavior aligned. Nothing else to change from my side as long as this CDN endpoint is confirmed as the intended one for CI.
Summary
This change stabilizes the CI ISO build by switching package mirrors to a CDN-based mirror and enabling curl-based transfers to improve download reliability and speed.
Details
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.