Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add option to assign existing users to the docker privileged user group. #12

Merged
merged 3 commits into from
Dec 22, 2021
Merged

Conversation

philippwaller
Copy link
Contributor

Add option to assign existing users to the privileged docker user group.

Description
This PR provides the ability to assign existing users to the Docker privileged user group (see official docs).

Testing
New test case created.

@robertdebock: If you're okay with this change, I would create the same PR for the "ansible-role-docker" repository.

Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
@robertdebock
Copy link
Owner

That looks good, go a ahead for the docker role as well.

I like how you include well defined test scenario's, thanks!

Regards, Robert de Bock.

…encies.

Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
Signed-off-by: Philipp Waller <1090452+philippwaller@users.noreply.github.com>
@robertdebock robertdebock merged commit f72e664 into robertdebock:master Dec 22, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants