Bug 2319897 - Please branch and build koji for EPEL 10
Summary: Please branch and build koji for EPEL 10
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: koji
Version: epel10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mike McLean
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 2325201 (view as bug list)
Depends On: 2325337 2325348
Blocks: EPEL10Tracker 2313928 2313905 2319896 2319898
TreeView+ depends on / blocked
 
Reported: 2024-10-19 10:04 UTC by Xavier Bachelot
Modified: 2024-12-11 01:57 UTC (History)
5 users (show)

Fixed In Version: koji-1.35.1-5.el10_0
Clone Of:
Environment:
Last Closed: 2024-12-11 01:57:56 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Xavier Bachelot 2024-10-19 10:04:14 UTC
Hi,

Could you please branch and build koji for EPEL 10 ?
It is needed to build another package.

Regards,
Xavier

Comment 1 Mike McLean 2024-10-21 16:38:36 UTC
What package requires Koji installed inside the buildroot and why?

Comment 2 Mike McLean 2024-10-21 16:58:00 UTC
To answer my own question, looks like a number of builds such as: centpkg, packit, fedpkg, rpkg, bodhi-client, osh, sigul.

Likely (hopefully), the buildtime dep is simply that these tools import koji and need it installed for their unit tests to work.

Comment 3 Kevin Fenzi 2024-11-03 18:31:31 UTC
So, it builds fine currently... but I have no easy way to test that all the requirements are there, it could be uninstallable. ;( 

Is there some way to get a stream10 container to test in?

Comment 4 Mike McLean 2024-11-08 21:30:02 UTC
centos:stream10-development perhaps?

https://quay.io/repository/centos/centos?tab=tags

Comment 5 Kevin Fenzi 2024-11-09 23:31:45 UTC
ok, some simple tests seem to work fine. 

:) 

So, lets get it in and go from there for any issues.

Comment 6 Fedora Update System 2024-11-09 23:59:44 UTC
FEDORA-EPEL-2024-75c9a716d4 (koji-1.35.1-3.el10_0) has been submitted as an update to Fedora EPEL 10.0.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-75c9a716d4

Comment 7 Fedora Update System 2024-11-11 04:31:30 UTC
FEDORA-EPEL-2024-75c9a716d4 has been pushed to the Fedora EPEL 10.0 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-75c9a716d4

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 8 Carl George 🤠 2024-11-11 20:49:16 UTC
*** Bug 2325201 has been marked as a duplicate of this bug. ***

Comment 9 Kevin Fenzi 2024-11-11 20:53:44 UTC
And my tests were too simple. There's some subpackages with missing deps... ;(

Comment 10 Romain Geissler 2024-11-11 22:48:02 UTC
koji-builder depends on /usr/bin/cvs. Is this really something we would like to keep in EPEL 10 for the next 10 years to come ? Is cvs still really needed nowadays ?

Comment 11 Romain Geissler 2024-11-11 22:55:50 UTC
For python3-koji-hub-plugins which depends on the non existing package python3-qpid-proton it seems that was already the case in EPEL 9: see  https://bugzilla.redhat.com/show_bug.cgi?id=2168181 Not sure if we shall ask for the whole qpid-proton package to be branched or if this is actually not needed for koji in EPEL 10 ?

Comment 12 Carl George 🤠 2024-11-11 23:40:10 UTC
I noticed that too about cvs.  I have no idea how important it is as a koji dependency, but I did want to point out that according to the upstream news page the last release was 16 years ago.  That certainly has the appearance of an unmaintained upstream, and should probably get evaluated for deprecation/retirement in Fedora.

https://savannah.nongnu.org/news/?group=cvs

Comment 13 Kevin Fenzi 2024-11-15 00:32:22 UTC
Yeah, we can drop cvs most likely. I'll try and look at this this weekend if I have time.

Comment 14 Mike McLean 2024-11-15 14:50:35 UTC
> I have no idea how important it is as a koji dependency

It only matters if folks are building from cvs. Should be fine to drop it.

Comment 15 Mike McLean 2024-11-15 14:56:42 UTC
We should probably drop it in the upstream spec too -- https://pagure.io/koji/issue/4265

Comment 16 Kevin Fenzi 2024-11-17 19:01:09 UTC
Perhaps the way to go here is to move those all (cvs/svn/git) to Reccomends? 

python-multilib is tracked in https://bugzilla.redhat.com/show_bug.cgi?id=2325337

qpid-proton is tracked in https://bugzilla.redhat.com/show_bug.cgi?id=2325348

(and is/was a problem in epel9 too): https://bugzilla.redhat.com/show_bug.cgi?id=2168181

Comment 17 Carl George 🤠 2024-11-18 07:00:38 UTC
Recommends is one way to go about it.  Another would be to move the Requires to subpackages that can be conditionally disabled when the SCM tool is not available.  For example, koji-git would require git and would be built everywhere.  koji-cvs would require cvs, and could be disabled on new EPEL branches until/unless cvs is added.

Comment 18 Carl George 🤠 2024-11-19 00:45:55 UTC
Here's a PR for the approach I was describing.  If y'all would rather go another route, such as just dropping the cvs dependency outright, feel free to close it.

https://src.fedoraproject.org/rpms/koji/pull-request/22

Comment 19 Kevin Fenzi 2024-11-19 00:52:00 UTC
FWIW, I'd prefer fedora packages go with whatever upstream prefers here so we don't diverge. :)

Comment 20 Romain Geissler 2024-11-28 14:05:35 UTC
Note that I have submitted https://src.fedoraproject.org/rpms/koji/pull-request/23 and https://pagure.io/koji/pull-request/4271 to remove this strong requirement on the cvs package both upstream and in the Fedora/Epel packaging repo.

Comment 21 Fedora Update System 2024-12-09 22:20:47 UTC
FEDORA-EPEL-2024-9b800ea56b (koji-1.35.1-5.el10_0) has been submitted as an update to Fedora EPEL 10.0.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-9b800ea56b

Comment 22 Fedora Update System 2024-12-10 01:39:01 UTC
FEDORA-EPEL-2024-9b800ea56b has been pushed to the Fedora EPEL 10.0 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-9b800ea56b

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 23 Fedora Update System 2024-12-11 01:57:56 UTC
FEDORA-EPEL-2024-9b800ea56b (koji-1.35.1-5.el10_0) has been pushed to the Fedora EPEL 10.0 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.