Skip to content

ci: use Quay and GitHub releases for MinIO - #933

Merged
kevinjqliu merged 2 commits into
apache:mainfrom
smaheshwar-pltr:ci/minio-quay-images
Sep 11, 2026
Merged

ci: use Quay and GitHub releases for MinIO#933
kevinjqliu merged 2 commits into
apache:mainfrom
smaheshwar-pltr:ci/minio-quay-images

Conversation

@smaheshwar-pltr

@smaheshwar-pltr smaheshwar-pltr commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Related:

Docker Hub is no longer serving minio/minio (https://hub.docker.com/r/minio/minio), preventing the Docker-based S3 test environment from starting on clean machines, including Linux CI and open PRs. The native binary downloads from dl.min.io also return HTTP 410, stopping bucket setup on Linux and macOS.

This PR switches to the corresponding Quay image, as the related PRs above do - those other projects are encountering the same issue as us. It also downloads the MinIO client and Windows server from pinned GitHub releases and fails immediately on HTTP download errors.

We should track moving away from MinIO as a follow-up, like PyIceberg did - but that's a larger discussion; this fix is temporary to unblock CI, PRs, etc.

AI Disclosure: Assisted by Codex (GPT-6).

Docker Hub no longer serves the MinIO server repository. Use the corresponding Quay image so Docker-based S3 test setup can start on clean machines.

Generated-by: Codex
@smaheshwar-pltr
smaheshwar-pltr marked this pull request as ready for review September 11, 2026 22:50
@smaheshwar-pltr
smaheshwar-pltr marked this pull request as draft September 11, 2026 22:52
The MinIO download host returns HTTP 410 for native binaries. Use pinned official GitHub release assets for the client and Windows server, and reject HTTP errors before executing downloads.

Generated-by: Codex
@smaheshwar-pltr smaheshwar-pltr changed the title ci: pull MinIO image from Quay ci: use Quay and GitHub releases for MinIO Sep 11, 2026
@smaheshwar-pltr
smaheshwar-pltr marked this pull request as ready for review September 11, 2026 22:57

@kevinjqliu kevinjqliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ty!

@kevinjqliu
kevinjqliu merged commit a00387f into apache:main Sep 11, 2026
27 checks passed
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