-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add non-privileged user #5
Comments
For my own workflows, I use these images just to test the basic functionality of the Ansible roles and playbooks I'm using, and don't test out things like permissions/become/etc., so I don't have a need for it... However, I would be willing to add this on (since it doesn't seem to break anything by default), but only if I were able to get it added to all my various docker-*-ansible images so they're on feature parity. But if you work on a PR for this one, I may be able to just copy and paste across the rest. |
Thanks for the quick answer. That sounds good to me, I should be able to get to this next week. |
Let me know whether I should open the other PRs as well after you had time to review the changes here or if it's easier for you to do the changes yourself. |
Would you consider adding the possibility of running as a non-privileged user to the image as described in the molecule docs?
https://molecule.readthedocs.io/en/latest/examples.html#docker-with-non-privileged-user
I'd be happy to contribute this change instead of forking, if you think it would be a good addition.
The text was updated successfully, but these errors were encountered: