Skip to content

Unit tests no longer pull PostgreSQL anonymously from Docker Hub - #2473

Open
Eric Curtin (ericcurtin) wants to merge 5 commits into
agent-substrate:mainfrom
ericcurtin:pg-image-mirror
Open

Eric Curtin (ericcurtin) wants to merge 5 commits into
agent-substrate:mainfrom
ericcurtin:pg-image-mirror

Conversation

@ericcurtin

@ericcurtin Eric Curtin (ericcurtin) commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Configure mirror.gcr.io as the Docker daemon registry mirror in CI and share one Postgres image constant across the four call sites. Drops a copy of dockerenv.Configure in authz tests.

Fixes #2471

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@ericcurtin

Copy link
Copy Markdown
Contributor Author

Zoe Zhao (@zoez7) Haven Xia (@HavenXia) PTAL when you get a chance. Thank you!

@ericcurtin

Copy link
Copy Markdown
Contributor Author

Benjamin Elder (@BenTheElder) Reworked, PTAL when you get a chance. Thank you!

@ericcurtin

Copy link
Copy Markdown
Contributor Author

Benjamin Elder (@BenTheElder) Merged main and resolved the conflicts. PTAL.

@bowei Bowei Du (bowei) added area/dev-infra area/tests Enhancing / fixing test coverage. kind/bug Something isn't working / bugfixes labels Oct 19, 2026
@ericcurtin

Copy link
Copy Markdown
Contributor Author

Benjamin Elder (@BenTheElder) Taahir Ahmed (@ahmedtd) Digest is pinned and the mirror step moved into a script. PTAL when you get a chance. Thank you!

Signed-off-by: Eric Curtin <eric.curtin@docker.com>
Signed-off-by: Eric Curtin <eric.curtin@docker.com>
Signed-off-by: Eric Curtin <eric.curtin@docker.com>
Signed-off-by: Eric Curtin <eric.curtin@docker.com>
@ericcurtin

Copy link
Copy Markdown
Contributor Author

Benjamin Elder (@BenTheElder) Tim Hockin (@thockin) PTAL when you get a chance. Thank you!

Signed-off-by: Eric Curtin <eric.curtin@docker.com>
@ericcurtin

Copy link
Copy Markdown
Contributor Author

Benjamin Elder (@BenTheElder) DCO sign-off fixed, no code changes.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dev-infra area/tests Enhancing / fixing test coverage. kind/bug Something isn't working / bugfixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: Docker Hub rate limit (toomanyrequests) fails every PostgreSQL-backed unit test package

3 participants