Skip to content

Comments

ci: Enable Tide from-branch-protection context policy#3148

Merged
tekton-robot merged 1 commit intotektoncd:mainfrom
vdemeester:tide-from-branch-protection
Feb 18, 2026
Merged

ci: Enable Tide from-branch-protection context policy#3148
tekton-robot merged 1 commit intotektoncd:mainfrom
vdemeester:tide-from-branch-protection

Conversation

@vdemeester
Copy link
Member

Changes

Enable from-branch-protection: true in the Tide context_options
configuration so that Tide derives required/optional status contexts
from GitHub branch protection rules (managed via Terraform in
terraform/branch-protection/).

This fixes a problem where conditional/skipped GitHub Actions jobs
(e.g. jobs that only run when certain files change) block Tide from
merging PRs, because Tide treats unknown contexts as required by
default.

With this change, only the checks listed in branch protection
(e.g. tide, EasyCLA, CI summary for tektoncd/pipeline) are
required — any other status context (including skipped conditional
jobs) is treated as optional.

See for example tektoncd/pipeline#9390.

/kind feature

Submitter Checklist

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

- Allow conditional/skipped GitHub Actions jobs to not block merges
- Derive required contexts from branch protection rules (Terraform)

Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
@tekton-robot tekton-robot added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 18, 2026
@tekton-robot tekton-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 18, 2026
Copy link
Member

@afrittoli afrittoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve
/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 18, 2026
@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: afrittoli

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 18, 2026
@tekton-robot tekton-robot merged commit 50bc706 into tektoncd:main Feb 18, 2026
27 checks passed
@vdemeester vdemeester deleted the tide-from-branch-protection branch February 18, 2026 15:55
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. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants