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

Run statgen/alignment:1.0.0 without being root #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Run statgen/alignment:1.0.0 without being root #66

wants to merge 1 commit into from

Conversation

cmball1
Copy link
Contributor

@cmball1 cmball1 commented Jul 31, 2018

Add a user to the Dockerfile to be able to run the CWL workflow without --no-match-user and --non-strict options with CWLTool.

@wshands
Copy link
Member

wshands commented Jul 31, 2018

Hi Chris, we have been using the Docker image at Dockerhub https://hub.docker.com/r/statgen/alignment/ where the Dockerfile is maintained by the U of Michigan, who wrote the original workflows for processing TopMed data. The Dockerfile at https://github.com/DataBiosphere/topmed-workflows/tree/master/aligner/docker is not being built or used yet. I can merge this and we can consider switching to this repo for the Dockerfile and or you can submit a pull request to https://github.com/statgen/docker-alignment to get the change in there. Thoughts?

@cmball1
Copy link
Contributor Author

cmball1 commented Jul 31, 2018

Hi @wshands I can submit a PR with https://github.com/statgen/docker-alignment and see what happens.

@cmball1
Copy link
Contributor Author

cmball1 commented Jul 31, 2018

PR: statgen/docker-alignment#1

@DailyDreaming
Copy link
Member

LGTM.

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.

3 participants