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

Regenie Docker image #23

Closed
akotlar opened this issue Aug 6, 2020 · 11 comments
Closed

Regenie Docker image #23

akotlar opened this issue Aug 6, 2020 · 11 comments

Comments

@akotlar
Copy link

akotlar commented Aug 6, 2020

Hey all, would it be possible add building and pushing a version-tagged regenie docker image. We (Hail team), are building a regenie module, and would like to rely on an official image. Happy to PR a Makefile change if that would help (just need the repo to push to).

Also, would you accept a PR that modified the Dockerfile to exclude ENTRYPOINT? This would make our lives slightly easier, and would make regenie invocation via docker container a bit more uniform with that in a normal shell environment (could copy paste the regenie examples, and when running from docker would make it possible to use environment variables within the regenie command)

@akotlar akotlar changed the title Regenie image Regenie Docker image Aug 6, 2020
@joellembatchou
Copy link
Collaborator

joellembatchou commented Aug 9, 2020

Hi,

Docker images are available in here. Thank you for suggesting the removal of ENTRYPOINT (indeed this was not necessary); the Dockerfile has been changed appropriately.

The Makefile has also been edited so that running make docker-build will build the docker image and make docker-test will run a test script (using the exemplar commands from the Regenie webpage) to check the image build.

Cheers,
Joelle

@akotlar
Copy link
Author

akotlar commented Aug 9, 2020

Awesome thanks Joelle!

@akotlar
Copy link
Author

akotlar commented Aug 9, 2020

Sorry, to be clear about the image: you aren't actually publishing it to Docker Hub. Would it be possible to do so? This has some implications on integrating the project into an existing build system, and the licensing issues that may entail, even with a compatible license (namely we need to get approval from our legal department to add the entire regenie codebase to our github repo, which is needed to build the image, but referencing a built docker image , published to Docker Hub or some other repository like gcr.io, carries no such complexities). It would be helpful, but I would also understand if you didn't want to add the CI step of docker build/docker push each time you release.

Thanks again.

@joellembatchou
Copy link
Collaborator

Hi,

We are currently only planning on publishing Docker images through Github.

Cheers,
Joelle

@akotlar
Copy link
Author

akotlar commented Aug 11, 2020

Ah, I had checked your link on mobile, and the GitHub app just redirected me to the repo, so I though you were pointing out that you make the Dockerfile available Didn't know GitHub had this feature, cool. Thanks!

@akotlar
Copy link
Author

akotlar commented Aug 11, 2020

This looks like it has a fairly important restriction. One needs to authenticate with Github before pulling the image. This makes docker pull ... break in a somewhat surprising way (authenticating to pull public image). Same issue is present when using as base image (FROM:)

edit: https://github.community/t/docker-pull-from-public-github-package-registry-fail-with-no-basic-auth-credentials-error/16358/18

Here's an example of a fairly large project running into issues with GitHub's image registry: containerd/containerd#3291 , even after providing credentials in some contexts.

@saphir746
Copy link

It's also not very compatible for use via Singularity

@matuskosut
Copy link

Hi @akotlar

I was working a little on publishing regenie in bioconda, which also includes docker images published to Biocontainers (https://biocontainers.pro/tools/regenie). It is an open community, so anyone can help out.

@saphir746 would Biocontainers work better with singularity?

@Shicheng-Guo
Copy link

Dear Joelle, any plan to update docker image to 3.0.3 ?

@joellembatchou
Copy link
Collaborator

@nathanweeks
Copy link
Contributor

Hi @akotlar

I was working a little on publishing regenie in bioconda, which also includes docker images published to Biocontainers (https://biocontainers.pro/tools/regenie). It is an open community, so anyone can help out.

@saphir746 would Biocontainers work better with singularity?

@matuskosut @saphir746 A Singularity image created from the regenie Biocontainers image seems to work OK (#292)

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

6 participants