Skip to content

test: use pgsty/silo image for MinIO test fixture - #783

Merged
kylebarron merged 1 commit into
mainfrom
kyle/silo-test-image
Sep 29, 2026
Merged

kylebarron merged 1 commit into
mainfrom
kyle/silo-test-image

Conversation

@kylebarron

@kylebarron kylebarron commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

should fix CI errors in existing PRs like #782


Note

This PR was written by Claude (Claude Code), not by @kylebarron.

The Python test jobs currently fail on every PR (e.g. #782) because the minio_store / minio_bucket fixtures can't pull quay.io/minio/minio:

docker.errors.APIError: 500 Server Error for http+docker://localhost/v1.48/images/create?tag=latest&fromImage=quay.io%2Fminio%2Fminio:
Internal Server Error ("unauthorized: access to the requested resource is not authorized")

MinIO has stopped publishing a public community image. quay.io/minio/minio now returns 401 to anonymous requests, and minio/minio on Docker Hub returns 404. main last passed on 2026-09-01, before the image was pulled.

This PR switches the fixture to pgsty/silo, a community-maintained fork of the MinIO server (formerly pgsty/minio). It keeps the same server /data --console-address :9001 command (the image entrypoint maps it to silo server …) and the same MINIO_ROOT_USER / MINIO_ROOT_PASSWORD env vars, so nothing else in the fixture changes. The image is pinned to a release tag rather than latest, so a future upstream change can't break CI silently.

Not included: docs/examples/minio.md and examples/minio/README.md still tell users to run quay.io/minio/minio. That can be a separate docs PR.

🤖 Written by Claude Code

The quay.io/minio/minio image now requires authentication (and
minio/minio is gone from Docker Hub), so every test using the
minio_store/minio_bucket fixtures fails with "unauthorized" when
pulling the image. Silo is a community-maintained fork of the MinIO
server that keeps the same CLI, MINIO_* env vars and S3 behavior.

Pin a release tag so the fixture doesn't break silently again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ds-release-bot ds-release-bot Bot added the test label Sep 29, 2026
@kylebarron
kylebarron enabled auto-merge (squash) September 29, 2026 15:15
@kylebarron
kylebarron disabled auto-merge September 29, 2026 15:16
@kylebarron
kylebarron merged commit 48cde12 into main Sep 29, 2026
10 checks passed
@kylebarron
kylebarron deleted the kyle/silo-test-image branch September 29, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant