Skip to content

[test-util] Pull the MinIO test image from quay.io - #9780

Closed
LuciferYang wants to merge 1 commit into
apache:masterfrom
LuciferYang:fix/minio-image-quay
Closed

[test-util] Pull the MinIO test image from quay.io#9780
LuciferYang wants to merge 1 commit into
apache:masterfrom
LuciferYang:fix/minio-image-quay

Conversation

@LuciferYang

Copy link
Copy Markdown
Contributor

Purpose

Every container test that starts MinioTestContainer now fails before the test body runs:

org.testcontainers.containers.ContainerFetchException: Can't get Docker image:
  RemoteDockerImage(imageName=minio/minio:RELEASE.2022-02-07T08-17-33Z, ...)
Caused by: com.github.dockerjava.api.exception.NotFoundException: Status 404:
  {"message":"pull access denied for minio/minio, repository does not exist or may require 'docker login'"}

MinIO stopped publishing its Community Edition images and the Docker Hub minio/minio repository is gone, so the unqualified name no longer resolves. Re-running does not help. paimon-s3-impl fails in S3FileIOTest, which means any PR whose reactor reaches that module goes red: a paimon-common change now fails CI for a reason that has nothing to do with it.

MinIO's own quay.io registry still serves the same RELEASE.2022-02-07T08-17-33Z tag publicly, so qualifying the name is enough to restore the pull. It is the same image at the same version, so its license is unchanged; only the registry differs. quay.io/minio/minio is MinIO's own organization namespace, and several ASF projects already pull it from there, among them apache/iceberg, apache/hudi, apache/doris and apache/opendal.

DockerImageVersions.MINIO is the only place the image is named. paimon-s3, paimon-s3-impl and paimon-flink-cdc reach it through MinioTestContainer.

Tests

The quay.io tag is anonymously pullable (its registry manifest returns 200) while Docker Hub returns 404 for the same tag. paimon-test-utils and paimon-s3-impl compile, with checkstyle and spotless clean. I have no Docker locally, so CI covers the container startup itself.

The Docker Hub minio/minio repository is no longer available, so every
container test that starts MinioTestContainer now fails at startup with
"Status 404: pull access denied for minio/minio, repository does not exist
or may require 'docker login'". MinIO's own quay.io registry still serves
the same RELEASE.2022-02-07T08-17-33Z tag publicly, so qualifying the image
name restores the pull without changing the version.
@LuciferYang

Copy link
Copy Markdown
Contributor Author

@JingsongLi We should probably fix this issue first.

@XiaoHongbo-Hope

Copy link
Copy Markdown
Contributor

Seems this PR duplicates #9761

@LuciferYang

Copy link
Copy Markdown
Contributor Author

Seems this PR duplicates #9761

ok, close this one.

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