search

voxpupuli / puppet-yum

Puppet module for Yum
https://forge.puppet.com/puppet/yum
MIT License
16 stars 99 forks source link

add dnf as package_provider #242

Closed cemyetil closed 1 year ago

cemyetil commented 2 years ago

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

traylenator commented 2 years ago

I think this is a close won't fix.

https://github.com/voxpupuli/puppet-yum#centos-7-and-older

The idea here was that on EL7 it was still fine to do.

yum::versionlock { '0:bash-4.1.2-9.el6_2.*':
  ensure => present,
}

however for EL8 and newer you should switch to

yum::versionlock{'bash':
  ensure => present,
  version => '4.1.2',
  release => '9.el8.2.*',
  epoch   => 0,
  arch    => 'x86_64',
}

this new syntax will work on EL7 as well so its an easy migration.

Once EL7 is dead the old yum::versionlock { '0:bash-4.1.2-9.el6_2.*': will be removed.

This should be closed.

jhoblitt commented 1 year ago

I am sympathetic to the pain this change has caused... and I'm dealing with that pain myself. However, the versionlock syntax has slightly changed under EL8 and I agree with the changes in this module. I'm going to close this PR as there is no path forward.