Skip to content

docker image tests failed due to dockerhub limitation #1380

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.

Sign up for GitHub

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

Jump to bottom
Closed
wyli opened this issue Dec 17, 2020 · 3 comments · Fixed by #1406
Closed

docker image tests failed due to dockerhub limitation #1380

wyli opened this issue Dec 17, 2020 · 3 comments · Fixed by #1406
Labels
CI/CD

Comments

@wyli
Copy link
Contributor

wyli commented Dec 17, 2020

cron-docker-test couldn't pull from dockerhub:

  /usr/bin/docker pull projectmonai/monai:latest
  Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
  Warning: Docker pull failed with exit code 1, back off 4.538 seconds before retry.
  /usr/bin/docker pull projectmonai/monai:latest
  Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
  Warning: Docker pull failed with exit code 1, back off 3.538 seconds before retry.
  /usr/bin/docker pull projectmonai/monai:latest
  Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
  Error: Docker pull failed with exit code 1

https://github.com/Project-MONAI/MONAI/runs/1567795300?check_suite_focus=true#step:2:14

cc @IsaacYangSLA @YuanTingHsieh @behxyz @yanchengnv
@wyli wyli added the CI/CD label Dec 17, 2020
@wyli
Copy link
Contributor Author

wyli commented Dec 17, 2020

I've submitted an OSS project application to get dockerhub's support
https://www.docker.com/community/open-source/application

@danielschulz
Copy link
Contributor

May I fix this issue?

@Nic-Ma
Copy link
Contributor

Nic-Ma commented Dec 30, 2020 •
edited
Loading

May I fix this issue?

Of course, welcome to contribute if you know how to fix it.

Thanks in advance.

danielschulz added a commit to danielschulz/MONAI that referenced this issue Jan 5, 2021
@danielschulz
added SHA256 check before unpacking downloaded artifacts
a749026 
added list of Nvidia maintainers
resolves: "docker image tests failed due to dockerhub limitation Project-MONAI#1380"

Signed-off-by: Daniel Schulz [email protected] <[email protected]>
@danielschulz danielschulz mentioned this issue Jan 5, 2021
Merged
3 tasks
@wyli wyli closed this as completed in #1406 Jan 6, 2021
wyli pushed a commit that referenced this issue Jan 6, 2021
@danielschulz
Fix failures from many pulls from hub.docker.com (Done) (#1406)
db831ae 
* changed year 2020 in copyright to "2020 - 2021" for both occurences

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* added SHA256 check before unpacking downloaded artifacts
added list of Nvidia maintainers
resolves: "docker image tests failed due to dockerhub limitation #1380"

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* added danielschulz-specific changes for GitHu Actions to work properly

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* added test

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* revoked test line

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* migrated code to local registry instead of hub.docker.com to circumvent rate limiting on it
remark: do not use it as pull-through cache as this would not work with static labels and docker images would diverge from one another

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* reverted back local-only changes

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* improved checking integrity of NGC CLI

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* removed typo in GH repository variable

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* added remark

Signed-off-by: Daniel Schulz [email protected] <[email protected]>

* improved MAINTAINER label
got rid of "as base" extension in FROM -- seems not be be pickup downwards (is no multi-stage Docker build -- hence: removed)

Signed-off-by: Daniel Schulz [email protected] <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CI/CD
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix failures from many pulls from hub.docker.com (Done) danielschulz/MONAI
3 participants
@wyli @danielschulz @Nic-Ma