Skip to content

Comments

Add CI summary fan-in job to presubmit CI#2741

Open
vdemeester wants to merge 1 commit intotektoncd:mainfrom
vdemeester:ci-summary-check
Open

Add CI summary fan-in job to presubmit CI#2741
vdemeester wants to merge 1 commit intotektoncd:mainfrom
vdemeester:ci-summary-check

Conversation

@vdemeester
Copy link
Member

@vdemeester vdemeester commented Feb 23, 2026

Changes

Add a ci-summary fan-in job that depends on all CI jobs and acts as a
single required check for branch protection. This replaces having
to configure individual job names as required checks.

The job treats both success and skipped as passing states,
only failing when a job returns failure or cancelled.

This follows the same pattern used in tektoncd/pipeline and tektoncd/triggers.

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Run the code checkers with make check
  • Regenerate the manpages, docs and go formatting with make generated
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

Release Notes

NONE

Add a ci-summary job that depends on all CI jobs and acts as a
single required check for branch protection. This replaces having
to configure individual job names as required checks.

The job treats both 'success' and 'skipped' as passing states,
only failing when a job returns 'failure' or 'cancelled'.

Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
@tekton-robot tekton-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Feb 23, 2026
@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from vdemeester after the PR has been reviewed.

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

@tekton-robot tekton-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. release-note-none Denotes a PR that doesnt merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Feb 23, 2026
vdemeester added a commit to vdemeester/tektoncd-plumbing that referenced this pull request Feb 23, 2026
Replace the 7 individual required checks for tektoncd/cli with
the single 'CI summary' fan-in check, matching the pattern already
used by tektoncd/pipeline, tektoncd/plumbing and tektoncd/pruner.

Corresponding PR: tektoncd/cli#2741

Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note-none Denotes a PR that doesnt merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants