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

URL redirect for centos stream does not work correctly #3334

Closed
thrix opened this issue Jul 19, 2024 · 7 comments · Fixed by #3341
Closed

URL redirect for centos stream does not work correctly #3334

thrix opened this issue Jul 19, 2024 · 7 comments · Fixed by #3341

Comments

@thrix
Copy link

thrix commented Jul 19, 2024

It would be great if this:

❯ curl https://copr.fedorainfracloud.org/coprs/rhcontainerbot/podman-next/repo/centos-stream-$releasever/rhcontainerbot-podman-next-centos-stream-$releasever.repo
[copr:copr.fedorainfracloud.org:rhcontainerbot:podman-next]
name=Copr repo for podman-next owned by rhcontainerbot
baseurl=https://download.copr.fedorainfracloud.org/results/rhcontainerbot/podman-next/centos-stream-10-$basearch/
type=rpm-md
skip_if_unavailable=True
gpgcheck=1
gpgkey=https://download.copr.fedorainfracloud.org/results/rhcontainerbot/podman-next/pubkey.gpg
repo_gpgcheck=0
enabled=1
enabled_metadata=1

Would return a generic copr repo with $releasever (as for Fedora):

❯ curl https://copr.fedorainfracloud.org/coprs/rhcontainerbot/podman-next/repo/centos-stream-$releasever/rhcontainerbot-podman-next-centos-stream-$releasever.repo
[copr:copr.fedorainfracloud.org:rhcontainerbot:podman-next]
name=Copr repo for podman-next owned by rhcontainerbot
baseurl=https://download.copr.fedorainfracloud.org/results/rhcontainerbot/podman-next/centos-stream-$releasever-$basearch/
type=rpm-md
skip_if_unavailable=True
gpgcheck=1
gpgkey=https://download.copr.fedorainfracloud.org/results/rhcontainerbot/podman-next/pubkey.gpg
repo_gpgcheck=0
enabled=1
enabled_metadata=1
@lsm5
Copy link

lsm5 commented Jul 22, 2024

@praiskup any ETA for this?

@praiskup
Copy link
Member

@xsuchy
Copy link
Member

xsuchy commented Jul 24, 2024

@thrix, Can you please summarize the motivation and use case for why you need/want this? It will help us prioritize this issue.

@nikromen nikromen moved this from Needs triage to In 3 months in CPT Kanban Jul 24, 2024
@nikromen
Copy link
Member

Patches are welcome for this!

@praiskup
Copy link
Member

Ad ETA, we release like every few months, and this should be an easy fix somewhere here. If you want to be 100% we have this in a new release, patch would be helpful.

@lsm5
Copy link

lsm5 commented Jul 24, 2024

Can you please summarize the motivation and use case for why you need/want this? It will help us prioritize this issue.

@xsuchy I need to use centos-stream copr repos from the podman-next copr on centos-stream envs and if I use generic centos-stream repos with $releasever in the URL, releasever resolves to 10. So, currently I can't mention a repo for c9s using releasever.

Sure, there are ugly workarounds, but rather have centos-stream urls behave the same way as fedora.

Hopefully fix is as simple as the linked frontend PR :)

@nforro
Copy link
Contributor

nforro commented Jul 24, 2024

The thing is, $releasever in the URL expands to an empty string, but that's still a valid repo URL, but the baseurl in it points to the first chroot in the list, so in this case centos-stream-10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
6 participants