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

Pause Automatic FF when the release stage runs #2807

Closed
Tracked by #2120
puerco opened this issue Dec 9, 2022 · 3 comments · Fixed by #2814
Closed
Tracked by #2120

Pause Automatic FF when the release stage runs #2807

puerco opened this issue Dec 9, 2022 · 3 comments · Fixed by #2814
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/feature Categorizes issue or PR as related to a new feature. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@puerco
Copy link
Member

puerco commented Dec 9, 2022

What would you like to be added:

We should have a way to pause the auto FF process. Or even better, make it aware of a release stage running on the branches it forwards.

Why is this needed:

When running the release stage of a new minor release, the FF process can pick up new commits and forward the branch, leaving the clone in GCB behind. This has dangerous implications as it fails late in the release process, while it is pushing the git artifacts.

We think this actually happened during the 1.26.0 cut and it broke the release as krel could not push the branch cleanly:

push git objects: pushing branches to the remote repository: pushing release-1.26 branch: 
   merge remote if required: merging remote branch origin/release-1.26 to local repo:
   run git merge: command /usr/bin/git merge -X ours origin/release-1.26 did not succeed:  

(unfortunately, we did not get the rest of the error)

@puerco puerco added kind/feature Categorizes issue or PR as related to a new feature. sig/release Categorizes an issue or PR as relevant to SIG Release. area/release-eng Issues or PRs related to the Release Engineering subproject labels Dec 9, 2022
@leonardpahlke
Copy link
Member

Is this a bug / flake?

@saschagrunert
Copy link
Member

saschagrunert commented Dec 9, 2022

This is unfortunate, and I have not considered that the fast forward can cause conflicts when cutting the minor together with the patches in parallel. So it's not a flake, but an uncovered corner case.

Two example for breaking fast forwards are:

Means we could check the author of the commits, and if we find the Kubernetes Release Robot <[email protected]>, then we do not do the fast forward.


I'm thinking about other indicators when to stop the fast forward. Maybe we can use the release day or check for the release cut issue?

@saschagrunert
Copy link
Member

The release cut issue may be a good indicator, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/feature Categorizes issue or PR as related to a new feature. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants