puppet-redis icon indicating copy to clipboard operation
puppet-redis copied to clipboard

Published 20 hours ago • verified publisher icon fsalum

Fix CentOS/RHEL fact

Open pall-valmundsson opened this issue 9 years ago • 2 comments

The version fact currently supplies the available redis version regardless of whether it's installed or not. The result is that all nodes contain a fact that implies that the latest available redis version seems to be installed.

This change switches from using yum to rpm to query the locally installed package database. It has the added benefit of not querying yum, which can be costly.

Addresses #39 in part.

pall-valmundsson avatar Mar 29 '15 23:03 pall-valmundsson

We use the version fact to use the correct redis.conf template. This change would break that.

fsalum avatar Jun 16 '15 04:06 fsalum

I think using rpm instead of yum makes a lot of sense, since a broken yum install or network connection could cause this fact to hang and timeout. I think running rpm instead of yum would be more efficient. 👍

bplunkert avatar Jan 23 '17 17:01 bplunkert