Repository navigation
Build: Pull MinIO images from Quay (cherry-pick apache/iceberg#18071) - #11
Merged
Merged
Conversation
dcagney
approved these changes
Sep 22, 2026
nickwu241
added a commit
that referenced
this pull request
Sep 22, 2026
Records the Docker Hub -> Quay MinIO fix as part of 1.11.0-affirm-patch.1 now that it's merged onto 1.11.0-affirm-patch. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
nickwu241
added a commit
that referenced
this pull request
Sep 22, 2026
…licate commits in DynamicCommitter when Flink jobId changes on restart (#10) * Flink: Fix duplicate commits in DynamicCommitter when Flink jobId changes on restart (apache#16011) Cherry-pick of apache#16011. Upstream commit: apache/iceberg@c11404c * Flink: Backport DynamicCommitter jobId fix to v1.20 and v2.0 Backport commit apache#16011. * Pin version.txt to 1.11.0-affirm-patch.1 and add AFFIRM_VERSIONS.md version.txt was previously untracked, so build.gradle's getProjectVersion() silently falls back to a git-describe-derived version if the file is ever missing from the working tree (e.g. after a checkout/clean) instead of failing loudly. This has already caused one unintended publish under 1.10.0-SNAPSHOT. AFFIRM_VERSIONS.md records which Affirm-local patches are carried by each published version. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * Remove published commit tracking from AFFIRM_VERSIONS.md Git tags are the source of truth for the commit published to Artifactory, not this file. * Document PR #11 (MinIO/Quay CI fix) in AFFIRM_VERSIONS.md Records the Docker Hub -> Quay MinIO fix as part of 1.11.0-affirm-patch.1 now that it's merged onto 1.11.0-affirm-patch. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: lrsb <github@lrsb.xyz> Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
nickwu241
added a commit
that referenced
this pull request
Sep 25, 2026
nickwu241
added a commit
that referenced
this pull request
Sep 25, 2026
nickwu241
added a commit
that referenced
this pull request
Sep 25, 2026
* Document and stage 1.11.0-affirm-patch.2 Bumps version.txt and adds the AFFIRM_VERSIONS.md changelog entry for the next patch release, covering the 3 review-triggered fixes from all-the-things#222721 (#12/#13/#14). Those PRs are open, not yet merged — rebase this branch onto 1.11.0-affirm-patch once they land, then tag 1.11.0-affirm-patch.2 at the resulting commit and publish per RUNBOOK_ICEBERG_PATCH_PUBLISH.md. * Update AFFIRM_VERSIONS.md: base off 1.11.0-affirm-patch.1, drop reviewer names * Drop #10/#11 from .2 table (already covered by .1 base) * Note #12/#13/#14 merged and branch rebased; ready to tag * Drop 'not yet published' framing now that .2 contents are final * Drop trailing note from .2 section to match .1 format
This was referenced Sep 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-picks apache#18071 (apache/iceberg@cff60bc18e) onto
1.11.0-affirm-patch.Why this is needed
MinIO deleted the
minio/minioandminio/mcimages from Docker Hub on 2026-09-11 (minio/minioandminio/mcnow return "repository does not exist"). This fork still references those Docker Hub images in two places:aws/src/integration/java/org/apache/iceberg/aws/s3/MinioUtil.java— used byiceberg-aws'score-testsintegration tests (TestFlakyS3InputStream,TestMinioUtil,TestS3FileIO*)kafka-connect/kafka-connect-runtime/docker/docker-compose.yml— used bykafka-connect-tests'TestIntegration*suite viaComposeContainerSince the deletion, both
core-testsandkafka-connect-testsfail deterministically on every PR against this branch (verified via 3 rerun attempts against #10, all producing the identicalcom.github.dockerjava.api.exception.NotFoundExceptionpulling these images) — this is not flaky/rate-limited CI, the images are permanently gone.Upstream
apache/iceberghit the same break and fixed it same-day with apache#18071, repointing both files atquay.io/minio/minio/quay.io/minio/mc(MinIO's images remain available there).mainhas been green since; this fork never picked up that commit because1.11.0-affirm-patchwas cut before it landed.Commit
Direct cherry-pick of apache/iceberg@cff60bc18e ("Build: Pull MinIO images from Quay", apache#18071), no conflicts, no adaptation needed — same 2 files this fork actually uses (
docker/iceberg-flink-quickstart/docker-compose.ymlandsite/docs/spark-quickstart.mdare also in the upstream commit but don't exist / aren't relevant on this branch's build).Verification
origin/1.11.0-affirm-patch; diff matches upstream's change exactly.core-testsandkafka-connect-testsgo green here the same way they already are on upstreammain.🤖 Generated with Claude Code