#2001 F29 Final: dnf rebase requested post-freeze
Closed: Accepted 5 years ago Opened 5 years ago by adamwill.

Hi, FESCo. Today at the F29 Final blocker review meeting, we noted that the DNF team has submitted this update to fix several blocker and FE bugs. The update is a rebase of both DNF and libdnf to new versions, including multiple changes and new features that do not relate to blocker or FE bugs.

This is against the usual policy that only blocker and FE fixes should be accepted through the milestone freezes. As DNF is a major component of Fedora and a significant part of some key F29 Changes, we would like to ask FESCo to consider this and decide on the best course of action. Should we accept the DNF 4.x / libdnf 0.22.x rebase into F29 Final, or request that they create something like DNF 3.6 / libdnf 0.20 feature branches with only F29 blocker/FE fixes instead?

@sgallagh @mattdm @dmach @kparal @sdharane


I think it's too late to accept a backwards incompatible release (I assume that's why it's changed to 4.0?) into Fedora 29. I'm of the persuasion that 4.0 should wait for Fedora 30 at this point.

DNF version has changed to get parity with yum versioning (DNF 4 == YUM 4 > YUM 3).
It doesn't come with any major changes, you can consider it another
3.x release but with a different version.

It contains mainly bugfixes incl. those related to modularity and backward compatibility.

Thanks @dmach for explaining, at the same time, do we have impact assessment of this change? If the impact only means waiting to see how QA would find issues and then taking a call, I'm afraid its too late given the final release date is just around the corner.

While I think this is not ideal, I'm +1 to accepting the other changes in as well (provided we resolve this ticket in a timely manner leaving testing time).

We could also ask releng for a compose with this included to see if it passes basic compose tests.

Could other FESCo members please vote in ticket? or you need more information?

Before voting, I would like to know what patches are being included that aren't for fixing FE's or blockers. I'd also like to know what the risks/benefits are if we accept those patched. In general, it seems unwise to include patches in such a core component that weren't approved by the FE/blocker process.

Metadata Update from @jforbes:
- Issue tagged with: meeting

5 years ago
  • 6fdf0e8cc7 (tag: 4.0.4) [module] Fix issue with incorrect type detected by swig
  • d539db7879 Release 4.0.4
  • cd78abb947 zanata update
  • 7ee18b4716 Document filterm method for query
  • 63db76e514 Improve a documentation of Selector
  • ee9cbcd14c Allow to use goal in dnf base
  • fedf2c3980 Improve query documentation
  • baf1965987 Allow to enable module dependencies (RhBug:1622566)
  • f4c9e64428 [conf] Use VectorString directly rather than converting it to a list.
  • c5a9238422 Revert "[config] Return vector options as tuple"
  • 9b27f5c039 Not quiet errors (RhBug:1508649)
  • 5e13b250f4 Quiet scriptlets information (RhBug:1590690)
  • dc277b657d Not report and log scriptlets error twice (RhBug:1619677)
  • 724d2072fd [module] Remove unused option
  • 954093ef5f [module] Add context information to module info (RhBug:1636091)
  • 496cf35e9e Calculate sack version from all installed packages (RhBug:1624291)
  • 3032624399 [module] Allow install pkgs from nonmodular content (RhBug:1631217)
  • 5323ab01d3 Remove logging function (RhBug:1489308)
  • 0085c56fd6 [doc] Add example how to use --advisory with install (RhBug:1625879)
  • 51542d15f3 docs: remove repoquery man page
  • e9f362facb [shell] Do not keep a copy of all demands
  • 4fb76364d3 Replace _additional_metadata config option with property
  • 5296843ae8 Rename extra_metadata config option to _additional_metadata
  • 24a286d4fa Fixes to work with the latest dnf/libdnf
  • 949b954222 Reflect changes in libdnf
  • 0cd70a0902 Repoquery --changelog documentation
  • a463d1cc65 Repoquery command accepts --changelogs option (RhBug:1483458)
  • 3b98e3df19 New demand "changelogs" to reqire changelogs metadata
  • 2de14efc0c Fix problem with upgrade in dnf
  • 55ea100835 Solves problem with "dnf upgrade" if base.conf.obsoletes=True
  • aa71348c83 automatic: set a timeout for EmailEmitter SMTP connections
  • b3ae0f721c Set termforce to AUTO to automatically detect if stdout is terminal
  • 9ca9907b1b Add missing newline after table header (RhBug:1629702)
  • c55fd6194e [tests] Ignore generated tests configurations in git
  • c90c00c1de use conditional name for python enum package
  • e75aff599f Introduce new dependencies to the spec file
  • e85ce27574 fix testing data for DNSSEC
  • 0f7b9d6f4b copyright note and future imports
  • 609e873d1c documentation for the dnssec extension
  • 9fc1e56067 dnssec extension
  • adc3247e9b Fix accessing exception messages (RhBug:1634676)
  • f9595bdd7a Report only unique module information (RhBug:1633151)
  • ceb5bbae1d Remove incorrect information from doc (RhBug:1497171)
  • 0062392a8a Fix traceback with repoquery --unsatisfied (RhBug:1620242)
  • f8e8301526 Do not handle defaults profiles if they are absent (RhBug:1628156)
  • 197901eeaa (tag: 3.6.1) Release 3.6.1

This is the list of commits in 4.0.4. Those are pretty much all bug fixes, with the exception of the --changelog stuff (a463d1cc65) and dnssec stuff (adc3247e9b..c55fd6194e). The changelog stuff could be considered almost a bugfix, but the second one is more worrying — it brings in new deps and is quite a bit of new code.

I guess we can accept the update to 4.0.4 for F29, but OTOH, it wouldn't be that much work to make a branch without the dnssec changes.

agreed F29 Final: dnf rebase requested post-freeze is approved (+6,1,-0)

Metadata Update from @jforbes:
- Issue close_status updated to: Accepted
- Issue status updated to: Closed (was: Open)

5 years ago

Login to comment on this ticket.

Metadata