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

makedist: update cvmfs package repo urls with fallback #105

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vvolkl
Copy link

@vvolkl vvolkl commented Jan 3, 2025

Adds a fallback URL for the cvmfs package download. makedist currently looks for the latest package in the listing returned by package servers; since the s3 bucket we are mainly using (and which is the most reliable) does not support listings, this mechanism now relies on reading a small file .latest_cvmfs_version_prod (https://cvmrepo.web.cern.ch/cvmrepo/.latest_cvmfs_version_prod) containing a string with the latest version. I'll add this to our testing to ensure it is kept up to date.

Obviously this won't work if you wanted to add mirrors maintained by a third party - one possibility would be to hardcode the latest version in the script.

The downloads for the non-default config packages is kept the same, except for a small fix for the egi download: Fixes #104

The ecsft.cern.ch url is removed alltogether, as this is the most likely one to cause issues.

Tested only on rhel9-x86_64

I'll add a yum repo and support for SUSE (and fedora) packages shortly - it seems like suse anyway needs to have proper testing (#87), so it is currently disabled.

@vvolkl
Copy link
Author

vvolkl commented Jan 3, 2025

@DrDaveD I don't think I touched the ppc64 downloads from copr, but those are failing - could you take a look?

@DrDaveD
Copy link
Collaborator

DrDaveD commented Jan 3, 2025

I think you did break it, because it works for me on an el8 machine using the master branch makedist but I get the same failure as the CI check when I use your PR's makedist.

@vvolkl
Copy link
Author

vvolkl commented Jan 3, 2025

Right, thanks - found it, just stumbled over the CVMFSRPMURLS naming change

Copy link
Collaborator

@DrDaveD DrDaveD left a comment

Choose a reason for hiding this comment

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

I had to make an urgent change to makedist yesterday for an osg yum repo reorganization, so now there's a merge conflict with makedist. Please rebase, and while you're at it, please squash to one commit.

makedist Outdated Show resolved Hide resolved
makedist Outdated Show resolved Hide resolved
makedist Outdated Show resolved Hide resolved
makedist Outdated Show resolved Hide resolved
makedist Outdated Show resolved Hide resolved
Adds a fallback URL for the cvmfs package download.
makedist currently looks for the latest package in the listing returned by package servers;
since the s3 bucket we are mainly using (and which is the most reliable) does not support listings,
this mechanism now relies on reading a small file .latest_cvmfs_version_prod (https://cvmrepo.web.cern.ch/cvmrepo/.latest_cvmfs_version_prod) containing a string with the latest version.
I'll add this to our testing to ensure it is kept up to date.

Obviously this won't work if you wanted to add mirrors maintained by a third party - one possibility would be to hardcode the latest version in the script.

The downloads for the non-default config packages is kept the same, except for a small fix for the egi download.

The ecsft.cern.ch url is removed alltogether, as this is the most likely one to cause issues.
@vvolkl
Copy link
Author

vvolkl commented Jan 9, 2025

Fixed another typo, this should be ready now.

However I did find that our s3 package server supports listings under the alternative url https://cvmrepo.s3-website.cern.ch/cvmrepo - if you prefer I can check if I can make that listing behave the same as the apache IndexDocument , that would let us avoid the .latest_cvmfs_version_prod file

@DrDaveD
Copy link
Collaborator

DrDaveD commented Jan 10, 2025

However I did find that our s3 package server supports listings under the alternative url https://cvmrepo.s3-website.cern.ch/cvmrepo - if you prefer I can check if I can make that listing behave the same as the apache IndexDocument , that would let us avoid the .latest_cvmfs_version_prod file

I don't feel strongly about it, because it is more efficient to just read a single file but it is more standard to do a listing. Is the creation of the version file automated, so it won't be forgotten and won't be any extra work for the CernVM team to maintain? If so then maybe we might as well stick with it.

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.

cvmfs-config-egi rpm searched for only in centos7 repository
2 participants