Skip to content

Comments

Fix crio package repository, allow data adjustment#125

Merged
jorhett merged 1 commit intovoxpupuli:masterfrom
jorhett:fix_crio_package_repo
Jun 20, 2025
Merged

Fix crio package repository, allow data adjustment#125
jorhett merged 1 commit intovoxpupuli:masterfrom
jorhett:fix_crio_package_repo

Conversation

@jorhett
Copy link
Contributor

@jorhett jorhett commented Apr 16, 2025

Pull Request (PR) description

The current package repository for CRIO is incorrect according to https://github.com/cri-o/packaging. The current value does not work on RedHat nodes:

Status code: 403 for https://prod-cdn.packages.k8s.io/repositories/isv:/kubernetes:/core:/stable:/v1.32/rpmrepodata/repomd.xml.key (IP: 18.244.214.76)

Additional benefits

By moving the package base to a parameter, this would allow someone to

  • fix it should the group change it again
  • choose to run another variant such as an unstable build

@jorhett jorhett requested a review from ananace April 16, 2025 20:48
@jorhett jorhett force-pushed the fix_crio_package_repo branch 3 times, most recently from 256a321 to f5525b7 Compare April 16, 2025 22:54
@jorhett
Copy link
Contributor Author

jorhett commented Apr 22, 2025

@ananace any thoughts?

Copy link
Member

@ananace ananace left a comment

Choose a reason for hiding this comment

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

Have thrown some quick looks at things, then got thoroughly distracted with other work.
I would prefer for this to use the same template setup as all the other URLs, but not going to make that a requirement for the PR.

@jorhett jorhett force-pushed the fix_crio_package_repo branch 3 times, most recently from bccb451 to 3f41879 Compare April 23, 2025 07:05
Add slashes after variable for consistency
@jorhett jorhett force-pushed the fix_crio_package_repo branch from 3f41879 to 7259410 Compare April 23, 2025 07:06
@jorhett jorhett requested a review from ananace June 14, 2025 03:24
@ananace ananace added the bug Something isn't working label Jun 20, 2025
@jorhett jorhett merged commit b880b20 into voxpupuli:master Jun 20, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants