Fix style for puppet-lint failures.
Use warning instead of notify when inform about deprecation.

Some modules ex. puppet-nova still use $cluster_disk_nodes, and when
notify is used to inform about deprecation every puppet runs cause
rabbitmq server restart.
Make curl and wget not go via a proxy (if one is configured) for installation of the rabbitmq admin CLI
Remove puppet_module_install in favor of copy_module_to
Optimize check for RHEL GPG key

Reduces the time to check for presence of
gpg pubkey for RabbitMQ from 1.6s to 0.1s.
Output cacertfile value only if it is set

When using verify_none cacertfile can be omitted. Therefore no value should be written if it is unset.
use rpm for RedHat family
passing tests
Add key_content parameter for offline install

* Custom key source/content for Debian (apt module) is
  required in order to allow rabbitmq being installed
  by Puppet for intranet only faced hosts as well as
  internet faced ones.
  It also elliminates a hardcode for URL which is a kind of
  SPOF and 'limitation' for operators' freedom of choose.

* Update docs as well

DocImpact
Implements improvement: #MODULES-1408

Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>

Fix package_gpg_key key_source and docs

* Reuse package_gpg_key for apt as well
* Update/add docs for package_gpg_key, key_source
* Fix key_source usage in repo::apt

Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>
Merge pull request #240 from bogdando/pcs

Add custom key content & service provider support
Use -q flag instead of array slices

Previously we sliced off the informational output lines. This
broke on rabbitmq 3.4.0 because the 'done' was removed. We can use
the -q flag to never print them in the first place.
Update unit tests to handle new -q flag

Updated the rabbitmq_user, rabbitmq_user_permissions, and
rabbitmq_vhost spec tests to correctly mock the rabbitmqctl command
with the -q flag. Also added one missed -q to a list users call.
Merge pull request #1 from cmurphy/fix_fix_340

Update unit tests to handle new -q flag
Merge pull request #251 from nibalizer/fix_340

Use -q flag instead of array slices
Use -m flag on rabbitmq-plugins command for managing rabbitmq_plugin resources

Use the -m flag for calls to the rabbitmq-plugins command, which enables minimal
output mode.  This is much easier and more reliable to parse.

Fixes an issue under RMQ 3.4.0 where rabbitmq_plugin resources were being enabled
on every Puppet run, even though they are already enabled.
Add metadata summary per FM-1523
Strip backslashes from the output of rabbitmqctl

The output of `rabbitmqctl(1)` escapes backslashes. This was documented
in the `rabbitmqctl` man page[1] but was removed when the manpage was
converted to DocBook XML[2].

This change causes extraneous backslashes to be stripped to prevent
this module from misreading escaped characters in regular expressions
and repeatedly applying the same change.

[1]:
https://github.com/rabbitmq/rabbitmq-server/blob/16418a9488e15c4d8ef3bfa9fce69190fb8ec796/docs/rabbitmqctl.1.pod#output-escaping
[2]:
https://github.com/rabbitmq/rabbitmq-server/commit/60c14baa96118e587b12716b49f7d66e487a5939#diff-b96d4e7ccd3d984656b8ab3534e6b460
Use shorter URL for RabbitMQ documentation

As suggested in
https://github.com/puppetlabs/puppetlabs-rabbitmq/pull/225.
Remove trailing whitespace
Use 16char keyid for rabbit repo

The 8char keyids are easier to collide.