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
Does this Orb require any changes to work with DockerHub auth? #193
Comments
Hello @apaleslimghost, Thank you for checking this out! There will be more communications coming out shortly to discuss this further, but I can confirm this orb will not be affected by the changes at docker. That said, it is still best practice to attempt to login if the user provides authentication. I am currently in communications with folks about that pattern, that is likely the pattern we will use. I have not yet tested but I also want to ensure that if they are blank, there is no negative repercussions. I can confirm though that this will not be a time-sensitive issue for us in this particular use-case. |
https://github.com/CircleCI-Public/slack-orb/blob/master/src/jobs/on-hold.yml#L37 -- uses an image from dockerhub. Without authenticating to dockerhub this will certainly be impacted by rate limits. |
More information will be coming but I can 100% guarantee this will not be affected. |
Some more information has been released. I believe we will be doing a longer and more detailed announcement soon as well. The main point from this post is, the namespace under which CircleCI published our own Docker images We do however still encourage users to add their own Docker credentials, especially in instances of private or non-circleci images. |
Describe Request:
When DockerHub starts rate limiting anonymous requests on Nov 1st, it looks like this Orb will be impacted, since there's no
auth
key in thedocker
config. It looks like a standard pattern is to expectDOCKERHUB_USERNAME
/DOCKERHUB_PASSWORD
environment variables (see example), will this (or something equivalent) be required for this Orb?Examples:
The text was updated successfully, but these errors were encountered: