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

Create user to run as non-root #1

Closed
wants to merge 1 commit into from
Closed

Conversation

cmball1
Copy link

@cmball1 cmball1 commented Jul 31, 2018

In reference to this comment: DataBiosphere/topmed-workflows#66 (comment)
In this PR: DataBiosphere/topmed-workflows#66

I wanted to create a PR here to see if we can get the Dockerfile changes integrated into the statgen/alignment image. Running the CWL Workflow in Toil with a Docker container where the user is root causes problems. I modified the Dockerfile to create a user to run as when the Docker Image is launched.

CWL Workflow: https://github.com/DataBiosphere/topmed-workflows/blob/master/aligner/sbg-alignment-cwl/topmed-alignment.cwl

@jonathonl
Copy link
Collaborator

This PR seems pretty specific to your environment. Can you achieve the same outcome by running containers with one of the following?

docker run --user `id nobody` statgen/docker-alignment whoami

or

docker run --user $(id `whoami`) statgen/docker-alignment whoami

@cmball1
Copy link
Author

cmball1 commented Aug 22, 2018

@jonathonl It looks like I can make it run as is with a few more input options. Sorry for the delayed response. I'll close this PR. Thanks!

@cmball1 cmball1 closed this Aug 22, 2018
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