Skip to content

Comments

ci: replace individual fan-in with unified CI summary job#4779

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

ci: replace individual fan-in with unified CI summary job#4779
vdemeester wants to merge 1 commit intotektoncd:mainfrom
vdemeester:ci-summary-check

Conversation

@vdemeester
Copy link
Member

Changes

Add a ci-summary fan-in job to ci.yml that depends on all CI jobs
(build, unit-tests, e2e-tests). This provides a single CI summary
check for branch protection, instead of listing every individual job.

The job runs with if: always() and treats success and skipped
as passing states, failing only on failure or cancelled.

Also adds merge_group trigger for merge queue support and updates the
concurrency group to include merge_group.head_ref.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Docs included if any changes are user facing
  • Tests included if any functionality added or changed
  • Follows the commit message standard
  • Meets the Tekton contributor standards (including
    functionality, content, code)
  • Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
  • Release notes block below has been updated with any user facing changes (new features, significant UI changes, API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
  • Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

Add a ci-summary fan-in job to ci.yml that depends on all CI jobs
(build, unit-tests, e2e-tests). This provides a single 'CI summary'
check for branch protection, instead of listing every individual job.

The job runs with 'if: always()' and treats 'success' and 'skipped'
as passing states, failing only on 'failure' or 'cancelled'.

Also adds merge_group trigger for merge queue support.

Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
@tekton-robot tekton-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 25, 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 assign alangreene after the PR has been reviewed.
You can assign the PR to them by writing /assign @alangreene in a comment when ready.

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

@vdemeester
Copy link
Member Author

/kind cleanup

@tekton-robot tekton-robot added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Feb 25, 2026
vdemeester added a commit to vdemeester/tektoncd-plumbing that referenced this pull request Feb 25, 2026
Replace the 6 individual dashboard checks (Build tests, E2E tests
variants, Unit tests) with the single 'CI summary' fan-in check.

This corresponds to tektoncd/dashboard#4779 which adds the ci-summary
fan-in job to the dashboard CI workflow.

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

kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants