Skip to content
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

parameterize managehome instead of hardcoding to true #21

Merged
merged 1 commit into from
Dec 1, 2015

Conversation

mmckinst
Copy link
Contributor

some people want to preserve deleted user's home dirs

FYI, puppet defaults to false for managehome

@mmckinst
Copy link
Contributor Author

thinking about this some more, you might want to set the default to true if thats what people are expecting from the module but it should at least be a parameter since some people might want to keep old home directories or do something with them before deleting them

@mcanevet
Copy link
Contributor

@mmckinst this will change the default of the module, please use true as default

@mmckinst
Copy link
Contributor Author

mmckinst commented Dec 1, 2015

@mcanevet updated

@mcanevet
Copy link
Contributor

mcanevet commented Dec 1, 2015

@mmckinst thanks

mcanevet added a commit that referenced this pull request Dec 1, 2015
parameterize managehome instead of hardcoding to true
@mcanevet mcanevet merged commit b63a70a into camptocamp:master Dec 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants