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

cicd/control service: set dockerhub account for default imagePullSecrets #550

Merged
merged 2 commits into from
Nov 23, 2021

Conversation

antoniivanov
Copy link
Collaborator

using dockerhub account we are hitting docker limits less.
https://www.docker.com/increase-rate-limit

I had set the account for gitlab produced pods/containers but did not
for pods/containers produced in other ways (e.g control service).

This way it is much less likely to hit docker limit

Testing Done: ran the command and saw the secrets appear and to be
configured. Immediately after that docker limits errors stopped.

Signed-off-by: Antoni Ivanov [email protected]

using dockerhub account we are hitting docker limits less.
https://www.docker.com/increase-rate-limit

I had set the account for gitlab produced pods/containers but did not
for pods/containers produced in other ways (e.g control service).

This way it is much less likely to hit docker limit

Testing Done: ran the command and saw the secrets appear and to be
configured. Immediately after that docker limits errors stopped.

Signed-off-by: Antoni Ivanov <[email protected]>
@antoniivanov antoniivanov enabled auto-merge (squash) November 23, 2021 16:24
@antoniivanov antoniivanov merged commit 0f5b186 into main Nov 23, 2021
@antoniivanov antoniivanov deleted the person/aivanov/control-service-docker branch November 23, 2021 16:39
ivakoleva pushed a commit that referenced this pull request Nov 29, 2021
…ets (#550)

using dockerhub account we are hitting docker limits less.
https://www.docker.com/increase-rate-limit

I had set the account for gitlab produced pods/containers but did not
for pods/containers produced in other ways (e.g control service).

This way it is much less likely to hit docker limit

Testing Done: ran the command and saw the secrets appear and to be
configured. Immediately after that docker limits errors stopped.

Signed-off-by: Antoni Ivanov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants