Skip to content

[GH-3355] Pull the MinIO test image from quay.io - #3356

Merged
jiayuasu merged 1 commit into
apache:masterfrom
jiayuasu:ci-minio-quay
Sep 12, 2026
Merged

[GH-3355] Pull the MinIO test image from quay.io#3356
jiayuasu merged 1 commit into
apache:masterfrom
jiayuasu:ci-minio-quay

Conversation

@jiayuasu

Copy link
Copy Markdown
Member

Did you read the Contributor Guide?

Is this PR related to a ticket?

What changes were proposed in this PR?

Docker Hub no longer serves the minio/minio repository, so every Testcontainers-based MinIO test fails to start and, with fail-fast, takes the whole Scala and Java build matrix down. This switches the four call sites (OsmReaderTest and the three GeoPackageReaderTest copies) to the same image on quay.io, pinned to a release tag instead of latest:

new MinIOContainer(
  DockerImageName
    .parse("quay.io/minio/minio:RELEASE.2025-09-07T16-13-09Z")
    .asCompatibleSubstituteFor("minio/minio"))

asCompatibleSubstituteFor is required because MinIOContainer asserts that the image is compatible with minio/minio. RELEASE.2025-09-07T16-13-09Z is the build quay's latest tag points at today, so this is the image the tests were already using.

How was this patch tested?

  • Test sources of spark/common and spark/spark-3.5 compile with the change.
  • The container tests need Docker, which is not available on my machine, so the real verification is this PR's CI run: the previously failing OsmReaderTest and GeoPackageReaderTest S3 cases should now start their MinIO container and pass.

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

Docker Hub no longer serves the minio/minio repository, so every
Testcontainers-based MinIO test fails to start and fail-fast takes the
whole Scala and Java build matrix down with it. Point the four call
sites at the same image on quay.io, pinned to the release that quay's
latest tag currently resolves to, and declare it a compatible
substitute so MinIOContainer accepts it.
@jiayuasu jiayuasu added this to the sedona-2.0.0 milestone Sep 12, 2026
@jiayuasu
jiayuasu merged commit e3b8254 into apache:master Sep 12, 2026
41 checks passed
@jiayuasu
jiayuasu deleted the ci-minio-quay branch September 12, 2026 03:48
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.

CI: MinIO-backed tests fail because Docker Hub no longer serves minio/minio

1 participant