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

tests: use DockerHub token in tests' kind clusters #679

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pmalek
Copy link
Member

@pmalek pmalek commented Oct 1, 2024

What this PR does / why we need it:

Which issue this PR fixes

Fixes #

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect significant changes

@pmalek pmalek self-assigned this Oct 1, 2024
@pmalek pmalek force-pushed the use-dockerhub-creds-in-kind-clusters branch 7 times, most recently from ca05dda to af70fc8 Compare October 3, 2024 13:07
@pmalek pmalek force-pushed the use-dockerhub-creds-in-kind-clusters branch from af70fc8 to 139d855 Compare October 7, 2024 11:19
@pmalek pmalek linked an issue Oct 7, 2024 that may be closed by this pull request
@pmalek
Copy link
Member Author

pmalek commented Oct 7, 2024

After a fix for e2e tests has been merged in #691 let's observe if we're still encountering the problem of exhausting DockerHub image pull quota.

We can reevaluate this at some later time when more data about test flakiness is available (e.g. through https://app.buildpulse.io/@Kong/gateway-operator)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration & E2E tests fail due to exceeding DockerHub pull quota
1 participant