search

sous-chefs / consul

Development repository for the consul cookbook
https://supermarket.chef.io/cookbooks/consul
Apache License 2.0
192 stars 244 forks source link

Use "service_user" and "service_group" attributes as defaults for resource params #445

Closed legal90 closed 7 years ago

legal90 commented 7 years ago

It looks like a mistake existing for a long time already. Regardless of their names, attributes ['consul']['service_user'] and ['consul']['service_group'] don't actually affect the user running Consul as well as permissions for ['consul']['config']['data_dir'] directory.

This PR fixes that. Now user and group params of consul_service resource have default values as ['consul']['service_user'] and ['consul']['service_group'] appropriately.

legal90 commented 7 years ago

I've added the same for consul_watch resource as well, so now it behaves similar to consul_definitions (in terms of owner&group of the file generated)

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.