-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
dnf module load/init dnf pkg mgr plugins properly #49278
dnf module load/init dnf pkg mgr plugins properly #49278
Conversation
Hi @maxamillion, thank you for submitting this pull-request! |
Fixes ansible#45940 Signed-off-by: Adam Miller <admiller@redhat.com>
f0c5a4e
to
82916de
Compare
base.pre_configure_plugins() | ||
base.configure_plugins() | ||
except AttributeError: | ||
pass # older versions of dnf didn't require this and don't have these methods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these configuration methods raise dnf.exceptions.RepoError
? If not, they should probably be above the RepoError try: except instead of nested inside of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abadger not to the best of my knowledge, no ... will update 👍
Signed-off-by: Adam Miller <admiller@redhat.com>
Looks good. +1 to merge from me. |
rebuild_merge |
* dnf module load/init dnf pkg mgr plugins properly Fixes ansible#45940 Signed-off-by: Adam Miller <admiller@redhat.com> (cherry picked from commit 378cc96)
* dnf module load/init dnf pkg mgr plugins properly Fixes ansible#45940 Signed-off-by: Adam Miller <admiller@redhat.com>
Signed-off-by: Adam Miller admiller@redhat.com
SUMMARY
Fixes #45940
ISSUE TYPE
COMPONENT NAME
dnf