Skip to content

ci(fix): update outdated minio client image in integration tests - #358

Merged
deusebio merged 1 commit into
mainfrom
ci-fix-tests-update-minio-client-image
Sep 22, 2026
Merged

deusebio merged 1 commit into
mainfrom
ci-fix-tests-update-minio-client-image

Conversation

@NohaIhab

Copy link
Copy Markdown
Contributor

Summary

The test_workflow_using_artifacts integration test was failing in CI with error: timed out waiting for the condition when creating the artifact bucket. The kubectl run probe pod used the stale minio/mc image, which fails to pull in CI (docker.io/minio/mc:latest no longer resolves / requires auth), so the pod never reached Ready and the command timed out instead of surfacing the real error.

Changes

  • Replaced the stale minio/mc image with quay.io/minio/mc (MINIO_CLIENT_IMAGE constant).
  • Added a run_probe_pod() helper that runs the one-off kubectl run pod and always deletes it in a finally block, replacing the unreliable --rm flag (which only cleans up on a clean attached exit and leaves pods behind on failure, causing name collisions on retries).

Similar to canonical/minio-operator#309.

@ckfbot ckfbot added backport track/3.7 Backport to track/3.7 backport track/3.5 Backport to track/3.5 labels Sep 22, 2026
@ckfbot

ckfbot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

🤖 Backport labels populated

Labels to this pull request were added automatically by the populate-labels.yaml action.

When the PR is merged, backport PRs according to the labels will be automatically created. To skip the backport creation, remove any unneeded labels before merging the PR.

@MattiaSarti
MattiaSarti requested a review from a team September 22, 2026 14:32
@deusebio
deusebio merged commit ac3c88c into main Sep 22, 2026
12 checks passed
@deusebio
deusebio deleted the ci-fix-tests-update-minio-client-image branch September 22, 2026 20:38
@ckfbot

ckfbot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Successfully created backport PR for track/3.7:

@ckfbot

ckfbot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Successfully created backport PR for track/3.5:

NohaIhab added a commit that referenced this pull request Sep 23, 2026
… (#359)

(cherry picked from commit ac3c88c)

Co-authored-by: Noha Ihab <49988746+NohaIhab@users.noreply.github.com>
NohaIhab added a commit that referenced this pull request Sep 23, 2026
… (#360)

(cherry picked from commit ac3c88c)

Co-authored-by: Noha Ihab <49988746+NohaIhab@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport track/3.5 Backport to track/3.5 backport track/3.7 Backport to track/3.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants