-
Notifications
You must be signed in to change notification settings - Fork 62
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
Failed git push does not fail release #385
Comments
@mauritsvanrees Did you have time to check on this issue ? |
@LvffY Sorry, no not yet. Let me have a look now.
@reinout We do not actually use this The easiest solution to the issue here would be to change the |
I think the problem here would be: how far do you roll back? Do you revert the postrelease commit? The tag? The release commit? The prerelease commit? Commits by plugins? All of these? And if you have used this successfully to upload a new release on PyPI, then I would not want to undo any of this. I would want to keep the exact commit and tag that were used to create this release. I can then manually push the changes to a new branch and create a PR, and that will likely solve the problem. |
@mauritsvanrees on the rollback feature I think that we want to rollback everything. You probably want to rollback the upload on the pipy repository as well. But if the upload on pypi is not "rollbackable", may be you're right and we don't want to rollback anything ? In this case, may be it could just be a documentation on how to do such a rollback if customer want to do so. |
Some quick thoughts:
The actual error that you show is caused by insufficient rights due to a protected main branch. That's actually a nasty problem. The ones making releases have to be admins that are allowed to circumvent the protection. Anyway, such a protection isn't something that a retry is going to help with. Though... you could get someone to give you the rights and then retry it. So: the retry seems like a good idea. |
Should help with the documentation part of #385
I have a PR #430 that (mostly) fixes this, I hope. |
Allow for 'git push' retry + a bit of documentation, fixes #385
First of all : thanks for your tool :)
First, my issue
During a release, if a push fails, the release does not fail. This can make false positive CI/CD builds (i.e the pipeline return as green but the push did not went well, so it's in fact, a red pipeline).
Reproduce steps
fullrelease --no-input
Expected result
The release should me marked as failed
Actual result
The release is marked as succeeded.
Here is the logs that I get :
fullrelease --no-input
git tag test
then try to push withgit push origin my-branch --tags
we can see that my command return a non-zero return codeSecond, a feature request
Based on this case, it could be great to have a rollback feature, that could allow people to run some code in case of error.
I think that a default implementation could be :
if something went wrong, try to remove the supposed created tag and, why not, the uploaded artifact
The text was updated successfully, but these errors were encountered: