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

Clean yum metadata after versionlock file update #102

Merged
merged 1 commit into from
Jul 4, 2018

Conversation

traylenator
Copy link

Currently when a versionlock entry is updated it has the unfortunate
problem that the cached yum repository metadata only contains
the previous locked files.

The class yum::plugin::versionlock now has a new option clean
defaulting to false. If true a yum clean all will exectuted
after the versionlock file is updated.

A new class yum::clean is added as it may be sensible
for some other operations to do the same.

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Currently when a versionlock entry is updated it has the unfortunate
problem that the cached yum repository metadata only contains
the previous locked files.

The class `yum::plugin::versionlock` now has a new option `clean`
defaulting to `false`. If true a `yum clean all` will exectuted
after the versionlock file is updated.

A new class `yum::clean` is added as it may be sensible
for some other operations to do the same.
@traylenator traylenator added the enhancement New feature or request label May 29, 2018
class yum::clean {

exec{'yum_clean_all':
command => '/usr/bin/yum clean all',

Choose a reason for hiding this comment

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

is yum in /usr/bin/ on el6 and el7?

Copy link
Author

Choose a reason for hiding this comment

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

Yes it's in /usr/bin on both. Acceptance tests should have picked it up anyway.

Choose a reason for hiding this comment

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

Cool, that's just something that bites me more than it should.

@bastelfreak
Copy link
Member

thanks @traylenator !

@bastelfreak bastelfreak merged commit 684baa5 into voxpupuli:master Jul 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants