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

Separate 'home' inside user_account resource #77

Closed
hulu1522 opened this issue Oct 2, 2014 · 3 comments
Closed

Separate 'home' inside user_account resource #77

hulu1522 opened this issue Oct 2, 2014 · 3 comments

Comments

@hulu1522
Copy link

hulu1522 commented Oct 2, 2014

Right now home is a combination of home_root and username. It would be nice to separate those so we can pass in our own home_root with the resource.

Example:

user_account "zeus" do
    home_root "/var/lib"
    ssh_keygen false
    action [:create, :manage]
end
@theckman
Copy link
Collaborator

theckman commented Nov 5, 2016

@hulu1522 Thank you for raising this issue. Sorry for the lack of response on this issue. I've just become a collaborator on this project so I'm working through all of the issues.

That's an interesting use-case to consider. Are you able to elaborate on what challenges you have with using the home resource to provide the full path?

@hulu1522
Copy link
Author

hulu1522 commented Nov 5, 2016

@theckman It's been so long I can't actually remember why I requested that. I would assume it was to allow the ability to set it instead of using attributes but I can also see that using the home with the full path does the same thing. I leave the decision on this in your hands as I assume I no longer need it. Thanks for cleaning up the stuff though. 👍

@theckman
Copy link
Collaborator

theckman commented Nov 7, 2016

@hulu1522 yeah, that's my worry with these issues that have been outstanding so long. 😕

I'll close this in favor of using the home resource for now. If someone raises the issue again we can look in to adding it to the LWRP.

Thank you for getting back to me so quickly! 👍

@theckman theckman closed this as completed Nov 7, 2016
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

No branches or pull requests

2 participants