Skip to content
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

FR: Support alternate URLs for package_index #113

Open
rohit-giridharan-ai opened this issue Nov 15, 2024 · 2 comments · May be fixed by #134
Open

FR: Support alternate URLs for package_index #113

rohit-giridharan-ai opened this issue Nov 15, 2024 · 2 comments · May be fixed by #134

Comments

@rohit-giridharan-ai
Copy link

We are using apt in apt.bzl to create a Debian package index. Our CI went down because the snapshot repository we were using, https://snapshot-cloudflare.debian.org, went down. https://snapshot.debian.org was still up - it would be nice to have the ability to add it as a mirror URL. I think it would require updating _fetch_package_index in package_index.bzl to support multiple URLs.

@BillyBlaze
Copy link

snapshot.debian.org was also not working for us (429 errors). We changed it to another mirror (utwente.nl) so it would be great if we could configure more beside debian.org.

@jjmaestro
Copy link
Contributor

+1, I've found the Debian mirror failures quire annoying. I've also seen the Cloudflare mirror have issues (e.g. lagging replication for months, at some point I think it was 6+ months behind!) but it's much better overall.

I've had to manually change this a few times while working on some project so I'm going to fix it once and for all 😄

jjmaestro added a commit to jjmaestro/rules_distroless that referenced this issue Dec 11, 2024
Fixes GoogleContainerTools#113

The Debian snapshot mirror fails quite often and, while the Cloudflare
mirror has had some issues in the past (e.g. it was lagging replication
for months) it's much more reliable.

This commit adds support for multiple URLs in the manifest. See
examples/debian_snapshot/bullseye.yaml for a sample usage.
@jjmaestro jjmaestro linked a pull request Dec 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants