Skip to content

ci: wait for active image producers - #13974

Open
jeffspahr wants to merge 2 commits into
kubeflow:masterfrom
jeffspahr:codex/wait-for-image-producers
Open

ci: wait for active image producers#13974
jeffspahr wants to merge 2 commits into
kubeflow:masterfrom
jeffspahr:codex/wait-for-image-producers

Conversation

@jeffspahr

@jeffspahr jeffspahr commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • extend the image-artifact barrier while a required producer is queued or running
  • fail early when a producer completes unsuccessfully without publishing its artifact
  • allow a short publication grace after successful producers finish
  • bound the fallback when GitHub producer-state queries remain unavailable

Why

Image-consuming test matrices currently start alongside the reusable image-build workflow. The shared deploy action gives producers a fixed 20-minute artifact window. A producer delayed by hosted-runner scheduling or a slow external package mirror can remain healthy beyond that window, causing every consumer lane to fail before tests begin.

This preserves the intentional cluster/build overlap while making the barrier depend on the state of the specific missing producer. One slow producer therefore no longer fans out into dozens of premature deployment failures, while genuine producer failures still terminate promptly with a useful diagnostic.

Verification

  • python3 .github/resources/scripts/wait_for_image_artifacts_test.py -v: 12 tests passed
  • CI scripts unit suite: 157 tests passed
  • bash -n .github/resources/scripts/wait-for-image-artifacts.sh
  • git diff --check

Signed-off-by: Jeff Spahr <spahrj@gmail.com>
@google-oss-prow

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from jeffspahr. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Jeff Spahr <spahrj@gmail.com>
@jeffspahr
jeffspahr marked this pull request as ready for review August 4, 2026 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant