-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
chore: un-cliwrap and use dnf5 instead of rpm-ostree #1954
base: main
Are you sure you want to change the base?
Conversation
We also need to explicitly test on both F40 and F41. We were bitten pretty hard enabling bootc in F40 with bad ISOs. Additionally, we need to make sure that bootc label is present on our images so that we can integrate with the rest of the bootc ecosystem (F41 is good already) |
+1 for F41, it is working perfectly fine indeed |
Just realized that this will let us use https://github.com/osbuild/bootc-image-builder/ for the ISOs |
1d6edb5
to
1b6f462
Compare
I just force pushed an update, and some keen observers may note that I ask people NOT to force-push their PRs... this is true, but in this case there were no comments/requests for changes on any files, so no context was lost. |
One known problem with this...
This is... undesirable. I think we need to decide:
|
We can just keep this open right? Is there an open upstream issue we can track? |
I have found a few possible issues in the dnf tracker, but I haven't nailed down the exact problem either, so I'm not sure which to track, if any truly relate. |
I think we hold off while we sort out which issues to track. I've seen some of the issues with dnf5 working with fedora 41 containers and it's very bizarre to get an error message with a 0 exit code. |
@bsherman should we open an issue of our own for this in the DNF bug tracker? This would be a pretty serious bug that the DNF team and image mode for RHEL team would want to address. Is there any way we can force a DNF command to fail in order to replicate it? Like an intentionally bad mirror or something as a test? |
Possibly upstream issues I've found: |
I've been tinkering with and testing dnf5 for a while. Key items are: - ensuring cliwrap is properly unwound (checked rpm-ostree Rust code) - using normal (not wrapped) binaries, eg for dracut - and the more obvious, replacing "rpm-ostree install" with "dnf5 install", etc Relates: #1946
1de8955
to
4c604fc
Compare
* retrofit unwrap PR from bluefin upstream ublue-os/bluefin#1954 * add tig * fix os-release for bluefin-dx build * update readme
I've been tinkering with and testing dnf5 for a while.
Key items are:
Note this does NOT switch to using dnf copr plugin for ublue-os/akmods COPR since doing so removes the RPM managed repo file. That repo file needs to be renamed in the
ublue-os-akmods-addons
package to match the COPR naming convention.Relates: #1946