Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

workflows: add stamp file in openshift/os to ensure we push rebases #1776

Merged
merged 1 commit into from Jun 9, 2022
Merged

workflows: add stamp file in openshift/os to ensure we push rebases #1776

merged 1 commit into from Jun 9, 2022

Conversation

bgilbert
Copy link
Contributor

@bgilbert bgilbert commented Jun 9, 2022

The create-pull-request action currently only force-pushes branches if their diff has changed. However, if we've meanwhile landed a parallel fedora-coreos-config update PR via a different branch, the diff might be the same but the base commit will be different. We need to push the branch in this case too, or else the PR will be left unmergeable due to conflicts. As a workaround, write the base commit ID into a stamp file in the root of openshift/os, so the diff will change whenever the base does.

Works around peter-evans/create-pull-request#1191.

The create-pull-request action currently only force-pushes branches if
their diff has changed.  However, if we've meanwhile landed a parallel
fedora-coreos-config update PR via a different branch, the diff might
be the same but the base commit will be different.  We need to push the
branch in this case too, or else the PR will be left unmergeable due to
conflicts.  As a workaround, write the base commit ID into a stamp file
in the root of openshift/os, so the diff will change whenever the base
does.
@bgilbert bgilbert merged commit 2dfcd0f into coreos:testing-devel Jun 9, 2022
@bgilbert bgilbert deleted the os/testing-devel branch June 9, 2022 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants