-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Move images used in tests off of Docker Hub #136
Comments
A workaround on the |
@adamnovak Can you try the following for the
I did look at moving to quay.io hosted images, but I had difficulties finding equivalents |
I added a list of the 4 containers we use to the original comment. |
@mr-c We're looking at getting a Docker Hub account we can use on CI for Toil. Is there a place in Those particular images all look to be
|
Looks like |
Hey @adamnovak I agree that finding a free resource would be better. I'm not against using these mirror URLs, but I'm having trouble finding public documentation or an commitments from Google. While the hashes match docker.io/library, it would be nice to know who is providing these images. Anyhow, feel free to send a PR against https://github.com/common-workflow-language/cwl-v1.2/tree/main and https://github.com/common-workflow-language/cwl-v1.2/tree/1.2.1_proposed |
This has been less of a problem for the Toil team lately, since our cache has been working properly. |
Docker Hub no longer allows the images used in the conformance tests to be fetched an arbitrary number of times by our CI for Toil. This causes conformance tests to fail because they need to fetch copies of the images and are denied access by Docker Hub. THis is especially a problem for Toil because it can in the worst case fetch a copy of each image every time it executes a CWL job that uses the image.
The images used in the tests should be hosted somewhere where they are freely accessible.
dockerPull: docker.io/alpine:latest
dockerPull: docker.io/bash:4.4
dockerPull: docker.io/debian:stable-slim
dockerPull: docker.io/python:3-slim
The text was updated successfully, but these errors were encountered: