Terrimonster fix 37 prov

[hunner]

Rebase and readme update for #45. We still need to solve the following issue:

This adds pe_puppetserver_gem to the provider for PE 3.7 in params, for hiera-eyaml installation. I do have a bit of a conundrum as a fresh install doesn't work until pe-puppetserver is restarted. BUT restarting pe-puppetserver in code could cause cascading failures for end users due to JVM startup time. Does anyone have thoughts on this? Should we just leave this out for now and just add documentation that JVM needs to be restarted? Or do we wait for things down the line that will fix this issue?

[binford2k]

@hunner this is the horrible way we mitigated it in the classroom. https://github.com/puppetlabs/puppetlabs-training-bootstrap/blob/master/modules/classroom/manifests/master/wait_for_startup.pp

[elyscape]

@binford2k Well that's nice and horrifying.

[igalic]

@binford2k how come this

File <| |> -> Service['pe-puppetserver']

doesn't leak?