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

Puppet RVM looks for presence of gpg2, but RVM installer is agostic #82

Closed
rwilcox opened this issue Feb 16, 2015 · 6 comments
Closed

Comments

@rwilcox
Copy link

rwilcox commented Feb 16, 2015

The RVM install script will check if you have either gpg2 or gpg installed, and use either to validate the GPG signature.

However, currently Puppet RVM only checks for gpg2 (and if it's found install the public key). This could lead to a confusing situation / failed provision if you have gpg installed: the installation will fail because puppet RVM hasn't downloaded the key (no gpg2 in the path!) but RVM's installed script wants to check the validity anyway (because it sees gpg in the path).

While this can be worked around (require the rvm GPG resource before Puppet installs RVM), it was.. rather surprising to see it fail and resulted in a lot of head scratching on my front.

TL; DR: can we check for gpg in addition to gpg2 ? And if either is there use the appropriate one to download the public key?

@domcleal
Copy link

Indeed, on Debian 7 (wheezy, stable), the binary is called "gpg" so this causes failures.

@TJM
Copy link

TJM commented Feb 18, 2015

For what its worth, gpg generally should be a symlink to gpg2, for example:

  • On CentOS 6.x:
# ls -l /usr/bin/gpg
lrwxrwxrwx. 1 root root 4 Oct 29 21:00 /usr/bin/gpg -> gpg2
  • On CentOS 5.x:
# ls -l /usr/bin/gpg
-rwxr-xr-x 1 root root 924152 Jan  8  2014 /usr/bin/gpg

@domcleal
Copy link

For completeness, ditto for CentOS 7.0:

# ls -l /bin/gpg
lrwxrwxrwx. 1 root root 4 Aug 27 00:07 /bin/gpg -> gpg2

@TJM
Copy link

TJM commented Feb 18, 2015

Thanks Dominic, As a result of that, maybe we can just check for gpg instead of gpg2 as opposed to "in addition to" since that is going to start to get ugly, logic wise.

@carlossg
Copy link
Member

If only we had tests to run in actual servers,... beaker to the rescue

@domcleal
Copy link

Thanks @carlossg, seems to work well in production!

janschumann pushed a commit to janschumann/puppet-rvm that referenced this issue Jun 26, 2019
Fix the installation on debian
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

No branches or pull requests

4 participants