Add fact that indicates if gpg is installed or not. about golja-gnupg HOT 4 CLOSED

Hey Chris,

thank you for the commit. To be honest I am not a big fun of adding custom facts, if they are not really necessary.

In a big puppet installation more custom facts you have, more time it takes the puppet agent to start/finish. Hopefully cfacter will fix that, but is still few months away.

Usually if I really need some custom facts I add them per environment in the common module. Also if I install gpg keys on a node, I know that I need them there, and if so I include the gnupg class.

What are your thoughts on that ?

from golja-gnupg.

RVM uses the which command to determine if gpg is installed. If it is, it looks for a gpg key and fails if one is not present.
https://github.com/wayneeseguin/rvm/blob/480fbc35211940dc2d5ac09ce630e1a161090f70/binscripts/rvm-installer#L386
We are simply trying to replicate this in the RVM module and only add a key if needed to avoid failure.

The RVM module should not install gnupg if it is not present. That is what this module is for.
So we need to determine if in fact gpg is present. We are currently using this exec, which I find sloppy.
https://github.com/maestrodev/puppet-rvm/blob/master/manifests/system.pp#L38
I think this is a good use of a fact and I would much rather use this modules gnupg_key type.

I suppose we could add the fact to the RVM module, but I feel it makes more sense for it to go in this module, as it is more related to gpg.

More info here:
voxpupuli/puppet-rvm#86

from golja-gnupg.

HaHa I like your obsession to make the code less sloppy 👍

I guess if https://github.com/carlossg want to make his module depending on mine we can add this new feature.

from golja-gnupg.

done ... sorry for the delay and thank you again for the PR.

from golja-gnupg.