search

claranet / puppet-consul_template

A Puppet module to manage the config and jobs of Consul Template from Hashicorp
Apache License 2.0
30 stars 89 forks source link

Fixing a bit of an overzealous change #15

Closed joshsouza closed 9 years ago

joshsouza commented 9 years ago

The PR I put in earlier can cause issues with unrelated groups that require users to exist first. Since the only thing that matters here is that users and groups must be established before files, this change is less error-prone. My apologies for any issues the first one may have caused.

gdhbashton commented 9 years ago

Josh, thanks for your commitment on this - I had it on my long TODO list to revisit, tho real life was really interfering too much for that to have happened any time soon!

joshsouza commented 9 years ago

No problem. Sorry if my first change caused you any headaches. On Jun 25, 2015 1:33 AM, "Gavin Hamill" notifications@github.com wrote:

Josh, thanks for your commitment on this - I had it on my long TODO list to revisit, tho real life was really interfering too much for that to have happened any time soon!

— Reply to this email directly or view it on GitHub https://github.com/gdhbashton/puppet-consul_template/pull/15#issuecomment-115145685 .