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

Implement home directory creation for consul user #404

Closed sun77 closed 6 years ago

sun77 commented 7 years ago

Poise-service doesn't create home directory with poise_service_user resource Also allow us to change the home directory with an attribute

legal90 commented 7 years ago

@Ginja @johnbellone What do you think about this? Should we manage the home dir in this cookbook by default? I'm not sure about using directory resource for that. I would roll poise_service_user back to user + group resources with setting manage_home true. Then the home dir will be created by "useradd" with system-defined attributes and permissions.

Ginja commented 7 years ago

https://github.com/poise/poise-service/issues/9

Why does the Consul user need a home? I'm in agreeance with @coderanger on this one. Generally service users don't have homes for security reasons. If one is required, I would stuff it in a wrapper cookbook.

kamaradclimber commented 7 years ago

consul user is the one used when launching health checks of type script. Many scripts/binary assume the existence of a $HOME

johnbellone commented 7 years ago

@kamaradclimber What would be an example?

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.