You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "LuciferYang (via GitHub)" <gi...@apache.org> on 2026/09/24 16:28:36 UTC
[PR] [test] Replace the removed MinIO test image with the pgsty/silo drop-in [paimon]
LuciferYang opened a new pull request, #10167:
URL: https://github.com/apache/paimon/pull/10167
### Purpose
`quay.io/minio/minio:RELEASE.2022-02-07T08-17-33Z`, pinned in `DockerImageVersions.MINIO`, is no longer pullable. MinIO archived its community edition and removed the images from Docker Hub (Sep 2026) and quay.io, so `quay.io/minio/minio` now returns 401 for an anonymous pull. Every test that starts the MinIO container (`S3FileIOTest` and the other `MinioTestContainer` users in `paimon-s3` and `paimon-s3-impl`) fails in CI with `org.testcontainers.containers.ContainerFetchException: Can't get Docker image`.
This repoints the pin at `pgsty/silo`, a maintained community MinIO fork that keeps the S3 API, the `MINIO_*` env vars, the `/minio/*` routes, and the `.minio.sys` storage layout, and still accepts the legacy `server <dir>` argv. Only `DockerImageVersions.MINIO` changes; `MinioTestContainer` (its command, env vars, and health probe) is untouched. This follows grafana/mimir#16604, which swapped to the same image and tag. `pgsty/silo` is published on Docker Hub only, so the pin uses `docker.io`.
### Tests
No new tests. The existing MinIO-backed tests (`S3FileIOTest` and the other `MinioTestContainer` users) now pull an image that exists; the S3 CI jobs exercise the swap.
### API and Format
no
### Documentation
no
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [test] Replace the removed MinIO test image with the pgsty/silo drop-in [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #10167:
URL: https://github.com/apache/paimon/pull/10167#issuecomment-5831838340
Reviewed the exact image-pin change and both `MinioTestContainer` fixtures. This has end-to-end value: the old quay.io image is unavailable, so the existing S3 integration suites cannot start; the new image keeps the `server /data` command, root-credential environment variables, port 9000, readiness path, and S3/STS behavior those fixtures require. There are no production API or file-format changes.
The PR head's CI is green across the Java matrix, including Core and integrations on JDK 8/11, both Flink E2E jobs, and the S3-backed test suites. I also ran the focused local `S3FileIOTest` Maven command: compilation succeeded and the test reached container startup, but my local Docker Hub image pull did not complete after more than four minutes, so I stopped that local attempt without a test assertion result. I restored Colima to its initial stopped state. The green CI provides the integration verification for this head. No code blocker found.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [test] Replace the removed MinIO test image with the pgsty/silo drop-in [paimon]
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #10167:
URL: https://github.com/apache/paimon/pull/10167#issuecomment-5834000298
Thank you @JingsongLi
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [test] Replace the removed MinIO test image with the pgsty/silo drop-in [paimon]
Posted by "LuciferYang (via GitHub)" <gi...@apache.org>.
LuciferYang commented on PR #10167:
URL: https://github.com/apache/paimon/pull/10167#issuecomment-5825245976
cc @JingsongLi FYI
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [test] Replace the removed MinIO test image with the pgsty/silo drop-in [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #10167:
URL: https://github.com/apache/paimon/pull/10167
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org