puppetlabs-operations/puppet-puppet — pull request #158 by danieldreier (2014-12-14 01:55:12+00:00)
Title: Add option to provide puppet agent install script

This is a bit experimental, and I'd like feedback on whether this feels like something in-scope for this module.

This wraps another module ([danieldreier/puppet_installer](https://forge.puppetlabs.com/danieldreier/puppet_installer) that I just published which installs a bash-language puppet install script and creates an apache or nginx vhost to provide it to new client nodes.

The intended use case is that when you add a new node to your infrastructure, you can do something like the following to install puppet and connect it to your master:

```bash
curl https://puppetmaster.example.com/deploy | sudo bash
```

The reason it's convenient to do it from inside the puppet-puppet module is that it simplifies the process of bootstrapping an entirely new infrastructure. The downside is that many of the options (using your own purchased SSL cert that isn't the puppet master's) are blocked unless you use hiera automatic parameter substitution.

Please push back on including this if it feels like unwarranted scope creep. I'm not quite sure where to draw the line in terms of whether this module is purely about puppetizing the puppet master itself vs a more opinionated puppetizing of the puppet infrastructure. If people would rather keep this more narrowly defined we could create an additional module to hold the opinionated stuff and just wrap this.