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

Use DNF instead of YUM. #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

FaramosCZ
Copy link
Contributor

Yum is deprecated and since Fedora 30 it is removed entirely (its binary).

Yum is deprecated and since Fedora 30 it is removed entirely (its binary).
@FaramosCZ
Copy link
Contributor Author

I could've pushed it without PR, however I'm not confident enough it won't break things.

Since Fedora 30 the /usr/bin/yum no longer exists.
And we can either use this patch, or set an alias at the beginning which would overwrite "yum" to "dnf".
By commenting this alias out, we would be able to run this tests on a systems without dnf too, even though they are not really suited for them.

I'd also like Jakub's opinion, since he's in charge of some tests automation which use those tests.

@praiskup
Copy link
Member

As an alternative, we can use 'dnf-yum' package.

@hhorak
Copy link
Collaborator

hhorak commented Oct 23, 2018

Using dnf will make the scripts not work in RHEL-7 for example, so I'd rather go with 'dnf-yum' as @praiskup proposes. I guess installing it on Fedora only could work well.

@kubco2
Copy link

kubco2 commented Oct 23, 2018

I am more for dnf-yum too.

@FaramosCZ
Copy link
Contributor Author

'dnf-yum' can't be installed inside beaker :(
It conflicts with 'yum' on which beaker client depends

@praiskup
Copy link
Member

@FaramosCZ that sounds like a bug either in dnf-yum or yum or snake or beaker client (that it depends on yum).

@praiskup
Copy link
Member

I've reported this https://bugzilla.redhat.com/show_bug.cgi?id=1669061 (this seems to be problem only on Fedora 30 (rawhide)).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants