Skip to content

Check merged upstream PR is included in source branch before dropping#64

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift-eng:mainfrom
RadekManak:check_pr_included
Jul 23, 2025
Merged

Check merged upstream PR is included in source branch before dropping#64
openshift-merge-bot[bot] merged 1 commit intoopenshift-eng:mainfrom
RadekManak:check_pr_included

Conversation

@RadekManak
Copy link
Contributor

@RadekManak RadekManak commented Jun 10, 2025

We were blindly dropping commits with merged upstream PR regardless of the branch that it merged into.

This caused commit being erroneously dropped when rebasing from release tag and the PR was only in main.

Now merge_commit_sha of the PR is checked if it is present it the source branch before dropping it.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 10, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 10, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 10, 2025
Copy link
Contributor

@damdo damdo left a comment

Choose a reason for hiding this comment

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

Nice!

/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 16, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: damdo, RadekManak

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

@RadekManak RadekManak marked this pull request as ready for review July 23, 2025 10:15
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 23, 2025
@openshift-ci openshift-ci bot requested review from JoelSpeed and elmiko July 23, 2025 10:15
Copy link
Contributor

@damdo damdo left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2025
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2025
Copy link
Contributor

@damdo damdo left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 12ad5eb into openshift-eng:main Jul 23, 2025
4 checks passed
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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants