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

422 Update is not a fast forward #75

Open
ehuss opened this issue Oct 30, 2019 · 5 comments
Open

422 Update is not a fast forward #75

ehuss opened this issue Oct 30, 2019 · 5 comments

Comments

@ehuss
Copy link

ehuss commented Oct 30, 2019

I've been lucky enough to hit "422 Update is not a fast forward" twice in the past two weeks. Not sure if it makes sense to file an issue here (feel free to close). Cross referencing:

My recent encounters:

Doesn't seem to happen too often: https://github.com/rust-lang/rust/pulls?utf8=%E2%9C%93&q=is%3Apr+%22update+is+not+a+fast+forward%22+

From what little I've read, I see other projects having similar problems, and some people concluding that the GitHub API can take a while to update (up to 30 seconds?) due to caching. I'm not too familiar with how homu works, but maybe a longer retry might help?

@RalfJung
Copy link
Member

It happened again.

@RalfJung
Copy link
Member

And again.

@RalfJung
Copy link
Member

It now happened in Miri for the first time as well.

@RalfJung
Copy link
Member

Looking at the commit it was trying to push to Miri, the error is completely right -- this is not a fast-forward. When bors created the merge commit here, it picked the wrong parent to merge in. It failed to update to the fact that a previous PR had just been merged.

@Mark-Simulacrum
Copy link
Member

Interesting! That seems like a potential across the board cause...

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

No branches or pull requests

3 participants