Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Switch to use Artifactory to avoid rate limit #87

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

Conversation

sellers
Copy link

@sellers sellers commented Jan 23, 2023

Please use docker-virtual for all image pulls from Dockerhub or related cached repositories. Failure to do so will cause service outage, build failures, or other problems with your system and it will happen randomly and unexpectedly.

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    This fixes a hazard with rate limit to pull image which will result in an incident if not fixed.

  • What is the current behavior? (You can also link to an open issue here)
    Image pulls will fail if not cached on the given node or on the system, experience will be random and non-deterministic. Remediation when incident occurs will be this exact PR, causing a new build, and deploy.

  • What is the new behavior (if this is a feature change)?
    It will pull from Artifactory, and be successful 99.9% of the time.

  • Other information:

Please use docker-virtual for all image pulls from Dockerhub or related cached repositories.  Failure to do so will cause service outage, build failures, or other problems with your system and it will happen randomly and unexpectedly.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant