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

Can we get rid of the need to run as root? #12

Open
bhuvy2 opened this issue Feb 2, 2019 · 1 comment
Open

Can we get rid of the need to run as root? #12

bhuvy2 opened this issue Feb 2, 2019 · 1 comment

Comments

@bhuvy2
Copy link
Contributor

bhuvy2 commented Feb 2, 2019

As usual, running a piece of code as root unless absolutely necessary is just good security practice. Can we have a script that creates a new user and sets up that user to be a part of the docker group instead of root?

@jasonB221
Copy link

I tried running as a user with access to docker, and the main issue I ran into is most docker containers run as root, which will make it hard to clean up the job directory after the run finishes. Even if the containers aren't running as root, it is unlikely that the docker container user and whatever service user is created will have the same UID, making cleanup annoying.

@zhengyao-lin zhengyao-lin transferred this issue from illinois-cs241/broadway-grader Sep 29, 2019
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