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

Cannot create Rawhide side tag update when the package has no %dist tag #5725

Open
hroncok opened this issue Aug 6, 2024 · 1 comment
Open
Milestone

Comments

@hroncok
Copy link
Contributor

hroncok commented Aug 6, 2024

I built fedora-obsolete-packages-41-5 in a rawhide side tag.

I proceed to create a Bodhi update from that side tag.

Bodhi told me:

Builds: Can't create update from tag for release 'EL-5' composed by Bodhi.

This is incorrect. The build was tagged to f41-build-side-93643 which is a Fedora 41 side tag. Bodhi should not attempt to parse the -5 release as EL5.

I fixed this by koji tag f41-updates-candidate fedora-obsolete-packages-41-5 which successfully created a Fedora 41 update: https://bodhi.fedoraproject.org/updates/FEDORA-2024-26e04543f7

@mattiaverga mattiaverga added this to the 8.2 milestone Oct 5, 2024
@mattiaverga
Copy link
Contributor

mattiaverga commented Oct 5, 2024

cache_nvrs(request, build)
if request.validated.get('from_tag'):
n, v, r = request.buildinfo[build]['nvr']
release = request.db.query(Release).filter(or_(Release.name == r,
Release.name == r.upper(),
Release.version == r)).first()
if release and release.composed_by_bodhi:
request.errors.add(
'body', 'builds',
f"Can't create update from tag for release"
f" '{release.name}' composed by Bodhi.")

This validator has no sense to me.
First of all, in n, v, r = request.buildinfo[build]['nvr'] the r refers to the release of the RPM, not the Release which the build refers to. So this, at least, has never worked as planned.

Then, why creating an update from tag should not be allowed for releases composed by Bodhi? Maybe this is a leftover of the time when sidetags were only allowed in Rawhide?

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

No branches or pull requests

2 participants