[andrewpark-scaleai]

Why

The goal is a gateway tier that a network hiccup can't fail. #575 made the gateway fixtures pull an image that main has published when its sources are unchanged. In PR #580's run 42716509774, gateway shard 1 failed all five of its tests during fixture setup, because GHCR's blob download broke mid-pull:

docker pull --platform linux/amd64 ghcr.io/scaleapi/agentenv-db-web:src-f993…@sha256:d379… failed:
failed to copy: httpReadSeeker: failed open: failed to do request: Get "https://pkg-containers.githubusercontent.com/…"

The fixture raised there. Before #575 it would have built the image.

What changes

A failed registry lookup, pull or tag now logs a warning, and the fixture builds the image from its Dockerfile as it did before #575. A failed build still raises.

The helper moves to tst/util/published_image.py, so tst/unit/test_published_image.py can force each failure without Docker or the network.

Verification

• With docker pull stubbed to fail with the error above, the helper logged Pulling db-web's published image failed, so it's built: … and then ran docker build.
• tst/unit/test_published_image.py covers the pull, a failed pull or tag, a failed lookup, unpublished sources, and a failed build. Making the helper stop catching failures fails 3 of its 6 tests.
• gateway_test.py still collects all 844 items, and ruff check passes.

Confidence Score: 5/5

The PR appears safe to merge; no actionable issues remain.

Summary

Gateway fixtures now build locally when a published-image lookup, pull, or tag fails.

• Gateway fixtures build images locally when published-image setup fails.

Tests were inspected, not run.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Find published image] --> B{Available?}
  B -->|Yes| C[Pull amd64 image and tag it]
  C -->|Success| D[Return]
  A -->|Lookup fails| E[Build locally]
  B -->|No| E
  C -->|Pull or tag fails| E
  E -->|Build fails| F[Raise error]
  E -->|Success| D

Loading

_{Reviews (2) · Last reviewed commit: "test: unit-test the gateway fixtures' pu..." · Reviewed by Greptile}