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

Use cvmrepo.s3.cern.ch as cvmfs package download url #43

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

vvolkl
Copy link
Contributor

@vvolkl vvolkl commented Jan 4, 2025

This is strictly better and removes one point of failure. Packages are downloaded anyway from s3.cern.ch, so it's better to rely on this url than to introduce one more host.

Next step will be to update the cvmfs-release package with a failover to cvmrepo.web.cern.ch, then this should be really robust.

A note on Amazonlinux 2023: this didn't work when I tested it with the amazonlinux docker image as 2023 uses (micro-)dnf instead of yum. Even s/yum/dnf/ didn't work as micro-dnf doesn't seem to support installing local packages (?). For the release package, also the redhat-release dependency was not supported.
Worked fine in amazonlinux:2

I'll take a look into adding a yum repo for amazonlinux, if it can really just link to the centos7/9 packages that should be very easy to do and let you remove this workaround.

This is strictly better and removes one point of failure.
Packages are downloaded anyway from s3.cern.ch, so it's better to
rely on this url than to introduce one more host.

Next step will be to update the cvmfs-release package with a failover to cvmrepo.web.cern.ch, then
this should be really robust.

A note on Amazonlinux 2023: this didn't work when I tested it with the amazonlinux docker image
as 2023 uses (micro-)dnf instead of yum. Even s/yum/dnf/ didn't work as micro-dnf doesn't seem to support
installing local packages (?). For the release package, also the redhat-release dependency was not supported.

I'll take a look into adding a yum repo for amazonlinux, if it can really just link to the centos7/9 packages that should
be very easy to do and let you remove this workaround.
@boegel boegel merged commit 2c6699c into EESSI:main Jan 6, 2025
7 checks passed
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 this pull request may close these issues.

2 participants