Skip to content

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 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>
@pratap0007
Copy link
Contributor

/retest

@pratap0007
Copy link
Contributor

/approve

@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pratap0007

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

The pull request process is described 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 the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 26, 2026
@pratap0007
Copy link
Contributor

cc: @divyansh42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. 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.

3 participants