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

configs: BuildWithDNF5 for ELN #1291

Merged
merged 1 commit into from
May 14, 2024

Conversation

praiskup
Copy link
Member

@sgallagher
Copy link
Contributor

Let's hold off on this for a few more days; I've got two related MRs out that we should land first:

  1. Add DNF 5 to RHEL 11 minimization/content-resolver-input#1105 - makes sure ELN has dnf5 and dnf5-plugins explicitly included. They're present right now, but only as part of ELN Extras.
  2. https://pagure.io/fedora-kickstarts/pull-request/1025 - Once the above is merged, we will make dnf5[-plugins] part of the bootstrap image.

Then we can avoid needing to turn off the bootstrap_ready option.

@yselkowitz
Copy link
Contributor

Can this proceed now?

@praiskup
Copy link
Member Author

Yeah, we have to... this is breaking Copr builds.

# Used to be True per https://github.com/fedora-eln/eln/issues/166 but with
# the 'dnf5' we are switching to False, reobserve after
# https://fedoraproject.org/wiki/Changes/ReplaceDnfWithDnf5 applied to ELN!
config_opts['bootstrap_image_ready'] = False
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, we made this change in ELN stack in the wrong order -> the ELN image now contains dnf5 and not dnf4, so it stays "ready". Going to update..

@praiskup praiskup force-pushed the praiskup-eln-dnf5 branch from 725827a to b771d95 Compare May 14, 2024 09:09
@praiskup praiskup removed the blocked label May 14, 2024
@praiskup praiskup marked this pull request as ready for review May 14, 2024 09:10
@praiskup praiskup force-pushed the praiskup-eln-dnf5 branch from b771d95 to a4471a1 Compare May 14, 2024 09:13
@praiskup praiskup force-pushed the praiskup-eln-dnf5 branch from a4471a1 to f1c5a46 Compare May 14, 2024 09:14
@praiskup praiskup added the release-blocker This needs to be shipped in the next release label May 14, 2024
@FrostyX
Copy link
Member

FrostyX commented May 14, 2024

I just tested the proposed config and it works well.
+1

@FrostyX FrostyX merged commit 02842f9 into rpm-software-management:main May 14, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-blocker This needs to be shipped in the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants