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

Workaround for yum v3.4.3 downgrade test #66

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

dhodovsk
Copy link
Contributor

RHEL 7 has yum v3.4.3. As far as I understood, a downgrade on that version is failing to successfully resolve dependencies available in multiple repos by itself, which results in an error, e.g:

Error: Package: freeradius-utils-3.0.13-15.el7.x86_64 (rhel-opt)
           Requires: freeradius = 3.0.13-15.el7
           Installed: freeradius-3.0.13-15.el7_9.1.x86_64 (@brew-62705510)
               freeradius = 3.0.13-15.el7_9.1
           Available: freeradius-3.0.13-15.el7.x86_64 (rhel)
               freeradius = 3.0.13-15.el7

This MR workarounds the issue by adding list of all installed content from the tested task repo to the downgrade command.

mtps-pkg-test Fixed Show fixed Hide fixed
mtps-pkg-test Fixed Show fixed Hide fixed
@jankaluza
Copy link
Contributor

I'm actually checking that there is no way to fully test it since the test packages are not built for epel7. Have you tried this code manually on rhel-7, or is it just a guess for now?

@dhodovsk
Copy link
Contributor Author

I debugged and tried the command on centos:7 (before it EOLed) and then again on a random rcm rhel 7 image.

mtps-pkg-test Dismissed Show dismissed Hide dismissed
Copy link
Member

@jpopelka jpopelka left a comment

Choose a reason for hiding this comment

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

I'm fine with this, It can't be worse than the current situation. :-)

@jpopelka
Copy link
Member

Can you also bump the Release in the spec file?

@jpopelka jpopelka merged commit a3d9b08 into fedora-ci:main Aug 14, 2024
7 of 8 checks passed
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.

3 participants