Skip to content

Make CodeRabbit a required check for openshift/origin#77595

Open
stbenjam wants to merge 1 commit intoopenshift:mainfrom
stbenjam:coderabbit-second-stage
Open

Make CodeRabbit a required check for openshift/origin#77595
stbenjam wants to merge 1 commit intoopenshift:mainfrom
stbenjam:coderabbit-second-stage

Conversation

@stbenjam
Copy link
Copy Markdown
Member

@stbenjam stbenjam commented Apr 9, 2026

Summary

  • Adds branch-protection configuration to require the CodeRabbit status check on the main branch of openshift/origin before merging
  • Uses the same pattern as openshift-eng/ai-helpers for requiring external GitHub Action checks
  • Tide picks this up automatically via from-branch-protection: true

Test plan

  • Verify prow config validation passes
  • Confirm CodeRabbit check appears as required on a test PR to openshift/origin main

🤖 Generated with Claude Code

@stbenjam @claude
Make CodeRabbit a required check for openshift/origin
ee4f140 
Adds branch-protection configuration to require the CodeRabbit
status check on the main branch of openshift/origin before merging.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 9, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stbenjam

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 9, 2026
@openshift-ci openshift-ci bot requested review from deads2k and p0lyn0mial April 9, 2026 13:50
@openshift-merge-bot openshift-merge-bot bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Apr 9, 2026
@openshift-merge-bot
Copy link
Copy Markdown
Contributor

openshift-merge-bot bot commented Apr 9, 2026

[REHEARSALNOTIFIER]
@stbenjam: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 9, 2026

@stbenjam: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@stbenjam
Copy link
Copy Markdown
Member Author

stbenjam commented Apr 9, 2026

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@deads2k deads2k Awaiting requested review from deads2k

@p0lyn0mial p0lyn0mial Awaiting requested review from p0lyn0mial

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@stbenjam