-
Notifications
You must be signed in to change notification settings - Fork 158
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
dnf5: set releasever in the base object's vars #1379
Conversation
With Fedora 40 now branched, openQA installer build tests started failing with an error: RuntimeError: cannot open file: (2) - No such file or directory [/etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-$releasever-x86_64] it seems like setting releasever via set_substitutions just does not fully work as expected, and libdnf goes looking for a file with a literal `$releasever` in it based on the gpgkey line in the repo file. Though it seems like the releasever substitution must work to *some* extent because we at least manage to resolve and download packages before the failure. This seems to fix it, anyhow, and is based on code from libdnf5 itself in dnf5/main.cpp RootCommand::set_argument_parser() . Signed-off-by: Adam Williamson <awilliam@redhat.com>
I really don't know all the details behind what's going on here, or if this is the correct long-term fix. But it at least fixes it for now and I need to get the openQA tests working again, so I'm going to do a downstream build with this patch, and I submitted this PR so we have everything nicely referenced. I kinda suspect this may ultimately be a libdnf bug? It seems like setting this via |
Oh, forgot to note - this happened only after Branching because Rawhide repo config files don't use |
Pull Request Test Coverage Report for Build 7898100212Details
💛 - Coveralls |
Full
|
Fixed in #1380 |
With Fedora 40 now branched, openQA installer build tests started failing with an error:
RuntimeError: cannot open file: (2) - No such file or directory [/etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-$releasever-x86_64]
it seems like setting releasever via set_substitutions just does not fully work as expected, and libdnf goes looking for a file with a literal
$releasever
in it based on the gpgkey line in the repo file. Though it seems like the releasever substitution must work to some extent because we at least manage to resolve and download packages before the failure.This seems to fix it, anyhow, and is based on code from libdnf5 itself in dnf5/main.cpp RootCommand::set_argument_parser() .