Skip to content

chore: replace live r8.im registry with local test registry#2816

Merged
michaeldwan merged 2 commits intomainfrom
md/use-test-registry
Mar 5, 2026
Merged

chore: replace live r8.im registry with local test registry#2816
michaeldwan merged 2 commits intomainfrom
md/use-test-registry

Conversation

@michaeldwan
Copy link
Member

Summary

  • Replace flaky integration tests that hit the live r8.im registry with a local test registry seeded via crane.Copy
  • Add registry-seed harness command that supports both local refs (alpine:latest) and external refs (docker.io/library/python:3.12-slim)
  • Fix missing name.Insecure on ParseReference in build.go (consistent with pkg/registry/)

Details

Four integration tests (build_base_image_sha, build_cog_version_match, build_python313_base_image, torch_cuda_baseimage) previously depended on the live r8.im registry, causing intermittent CI failures. They now use a local test registry:

  • 3 tests seed python:3.12-slim / python:3.13-slim from Docker Hub into the local registry and do full cog build + predict
  • 1 test (torch_cuda_baseimage) seeds alpine:latest as a dummy CUDA base image and verifies Dockerfile generation only via cog debug (avoids downloading multi-GB CUDA images)

The registry-seed command detects absolute vs relative refs by checking if the host portion before the first / contains a . or :.

michaeldwan and others added 2 commits March 5, 2026 09:57
…tion tests

Use registry-seed with crane.Copy to seed base images into a local test
registry instead of hitting the live r8.im registry, which is flaky in CI.

- Add registry-seed harness command supporting both local and external sources
- Convert build_base_image_sha, build_cog_version_match, build_python313_base_image
  to seed python:3.12/3.13-slim from Docker Hub via crane
- Convert torch_cuda_baseimage to use cog debug with a dummy seeded image
- Fix name.Insecure missing from ParseReference in build.go (consistent with
  pkg/registry/)
@michaeldwan michaeldwan marked this pull request as ready for review March 5, 2026 16:12
@michaeldwan michaeldwan requested a review from a team as a code owner March 5, 2026 16:12
@michaeldwan michaeldwan requested a review from markphelps March 5, 2026 16:27
@michaeldwan michaeldwan changed the title test: replace live r8.im registry with local test registry chore: replace live r8.im registry with local test registry Mar 5, 2026
@michaeldwan michaeldwan merged commit 32b8e9b into main Mar 5, 2026
31 checks passed
@michaeldwan michaeldwan deleted the md/use-test-registry branch March 5, 2026 17:52
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.

2 participants