New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make home directory location setting optional #575
Conversation
@bastelfreak I am not sure I picked the right label here - feel free to change if I goofed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did some chatting with @genebean on IRC and I think this is the best approach to move forward. It worka for new and existing environments and in my opinion that is important.
@bastelfreak how do you feel about merging this and cutting a release? |
manifests/init.pp
Outdated
@@ -117,6 +117,12 @@ | |||
# [*manage_user*] | |||
# Whether to create/manage the user that should own consul's configuration files. | |||
# | |||
# [*manage_user_home_location*] | |||
# Whether to explicitly set the location of teh consul user's home directory when this modules | |||
# manages the creation of the user. If the consul user already exists and this is enabled, puppet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also note that this setting only takes effect if manage_user
is true ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How does that look @solarkennedy ?
Fixes #559