#1448 Add option to request most recently tagged build from any inherited tag
Opened 3 years ago by sgallagh. Modified 3 years ago

Currently, pungi.phases.pkgset.sources.source_koji.filter_inherited() will show the most-recently tagged version of the top-most tag from the inheritance chain. The ELN project would like the ELN compose to be able to request the most-recently-tagged version that appears in any tag in the inheritence chain.

The reasoning behind this is that ELN is a rebuild of Rawhide content using a RHEL-like configuration (macros, build-flags, etc.). ELN builds are kicked off whenever a package is tagged into Rawhide. If they succeed, all is well, but if they fail and are a dependency for other packages, then those other packages should still be able to build against the functional Rawhide version of the package.

What we would like is for the compose (and buildroot compose) of ELN to allow us to tag in whatever version was most recently tagged into any tag in the inheritance chain. In our case, this will be only two: ELN and Rawhide. That way, we don't break subsequent builds in ELN. It also enables us to rebuild after a side-tag merges into Rawhide without manually needing to plan the build order.

CC @merlinm @tdawson @jkaluza


What do you mean by "buildroot compose"? We do not produce Buildroot compose for Fedora-ELN. If you mean the buildroot in Koji, then this is not done by Pungi, but by Koji itself and needs to be changed there.

The function you pointed out implements the same inheritance logic for modules as Koji does for RPMs (because Koji can't reasonably figure out latest modules).

Even if Pungi allowed changing the logic, it would only affect modules, and not RPMs. I don't think having this discrepancy is desirable.

OK, I was directed at that function by @mohanboddu who may have been confused.

So I gather from this that if I want this to happen, it needs to be an option in Koji instead. I'll move this ticket there.

In that case, it should be added to koji as an option and then it should be added to pungi as well so that we can provide the option in the pungi config.

Login to comment on this ticket.

Metadata