Add cert_name parameter to letsencrypt::certonly

[saimonn]

Pull Request (PR) description

Add the possibility to choose a $cert_name parameter different from $title.

This Pull Request (PR) fixes the following issues

When the $cert_name vary with the $title, this prevents to use two resources to manage the same certificate, as in the following example:

      # First standalone certificate is needed to create the key/certs, _before_ we can start apache with ssl enabled.
      ::letsencrypt::certonly {"${le_domains[0]}-standalone":
          domains     => $le_domains,
          manage_cron => false,
      }
      # this is the long-term cert, managing the cron renewal task
      -> ::letsencrypt::certonly {"${le_domains[0]}-webroot":
          domains              => $le_domains,
          manage_cron          => true,
          cron_success_command => 'apachectl -t && apachectl graceful',
          plugin               => 'webroot',
          webroot_paths        => [
            "/var/www/vhosts/${name}/htdocs",
          ],
      }

[alexjfisher]

voxpupuli/puppet-epel#116 should fix the build failures. I've restarted the build to see. Same with your other PR.

[saimonn]

Thanks !
I've fixed a trailing space in my spec test, and squashed my commits. it should pass now.

[alexjfisher]

Optional? I think this should just be String[1]. There's no way to set it to undef and that wouldn't be valid.