Skip to content

fix(bulker): pull the minio test container from quay, pinned - #1520

Merged
sahiltyagi-jitsu merged 1 commit into
newjitsufrom
fix/minio-testcontainer-quay
Sep 15, 2026
Merged

sahiltyagi-jitsu merged 1 commit into
newjitsufrom
fix/minio-testcontainer-quay

Conversation

@sahiltyagi-jitsu

Copy link
Copy Markdown
Contributor

Bulker Test (app) fails on every branch that actually runs it, and newjitsu is currently red for the same reason.

panic: create container: Error response from daemon: pull access denied for
minio/minio, repository does not exist or may require 'docker login'

Cause

docker.io/minio/minio is no longer publicly pullable. An anonymous pull token from auth.docker.io is rejected with UNAUTHORIZED, and Docker Hub's API has no public record of the repository. Docker's "repository does not exist or may require 'docker login'" is its generic message for an unresolvable reference — so this is not rate limiting, and adding registry credentials would not have fixed it.

Reproduced locally with the identical error:

$ docker pull minio/minio:latest
Error response from daemon: pull access denied for minio/minio, repository does
not exist or may require 'docker login'

Fix

MinIO also publish to quay.io, which is public. One line in minio_container.go.

Pinned to RELEASE.2025-09-07T16-13-09Z rather than a floating tag. That tag is multi-arch (amd64 / arm64 / ppc64le) and its manifest list is currently byte-identical to quay's :latest, so pinning costs nothing today and keeps the tests runnable on Apple Silicon. Both verified by pulling:

$ docker pull quay.io/minio/minio:RELEASE.2025-09-07T16-13-09Z     # arm64, native
Status: Downloaded newer image
$ docker pull --platform linux/amd64 quay.io/minio/minio:RELEASE.2025-09-07T16-13-09Z
Status: Downloaded newer image

There is precedent for quay in the tree — docker/docker-compose.yml:288 pulls Keycloak from it.

Why this went unnoticed for six days

The unpinned :latest meant the break needed no commit in this repo. And the bulker shard skips whenever a PR does not touch bulker/, reporting pass in ~11s — a real run takes 4-5 minutes. Recent merges (#1502, #1507, #1509, #1510) all show Bulker Test (app): pass at 11s; none of them touched bulker/.

The last real run was #1503 on 9 September. The JITSU-163 PRs were the first to exercise the suite since, which is why they surfaced it — and merging #1503 turned newjitsu red.

Worth considering separately: a scheduled run of the full suite on newjitsu would have caught this within a day instead of six.

Verification

CI on this PR is the real test — it runs the exact job that is failing. Locally the image now pulls natively on both architectures, and the previous reference reproduces the CI failure exactly.

Unblocks #1504, #1506, #1508 and the default branch.

🤖 Generated with Claude Code

`Bulker Test (app)` fails on every branch that actually runs it:

    panic: create container: Error response from daemon: pull access denied for
    minio/minio, repository does not exist or may require 'docker login'

`docker.io/minio/minio` is no longer publicly pullable — an anonymous pull token
is rejected with UNAUTHORIZED and Docker Hub has no public record of the repo.
Docker's "repository does not exist or may require 'docker login'" is its generic
message for an unresolvable reference, so this is not rate limiting and adding
registry credentials would not have fixed it. Reproduced locally with the same
error.

MinIO also publish to quay.io, which is public. Pinned to
RELEASE.2025-09-07T16-13-09Z rather than a floating tag: it is multi-arch
(amd64/arm64/ppc64le) and its manifest list is currently byte-identical to quay's
:latest, so pinning costs nothing today and keeps the tests runnable on Apple
Silicon.

The unpinned `:latest` is why this broke with no commit in the repo and stayed
hidden for six days: the bulker shard skips whenever a PR does not touch
`bulker/`, reporting "pass" in 11s, and nothing between 9 and 15 Sep touched it.
The JITSU-163 PRs were the first to run the suite again, and merging #1503 turned
newjitsu red.

Unblocks #1504, #1506, #1508 and the default branch.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

@jitsu-code-review jitsu-code-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewed the MinIO test-container image update and the surrounding S3 test setup. The pinned Quay image is publicly available and multi-architecture; the MinIO-backed file-storage test path also ran successfully with it. No actionable findings.

@sahiltyagi-jitsu
sahiltyagi-jitsu merged commit 8ec374f into newjitsu Sep 15, 2026
11 checks passed
@sahiltyagi-jitsu
sahiltyagi-jitsu deleted the fix/minio-testcontainer-quay branch September 15, 2026 08:39
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