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

Test for commit concurrency errors #74

Open
carterm opened this issue Oct 22, 2021 · 2 comments
Open

Test for commit concurrency errors #74

carterm opened this issue Oct 22, 2021 · 2 comments

Comments

@carterm
Copy link
Contributor

carterm commented Oct 22, 2021

Had an auto update run while a human update happened. Got a commit patch error...
Error: 422 error making request patch https://api.github.com/repos/cagov/drought.ca.gov/git/refs/heads/main: "Unprocessable Entity"
Suspect it has something to do with moving the head, after someone else moved the head. Need to account for that.

@carterm
Copy link
Contributor Author

carterm commented Dec 10, 2021

With the use of github-tree-push, this is now only happening in commit-only mode. Would happen if a commit completed, but then during the compare before the commit could be moved, something changed on the branch. Could be testing by putting a breakpoint after the commit happens and then changing something in the branch somewhere else.

This is the API it uses.
https://docs.github.com/en/rest/reference/git#update-a-reference

@carterm
Copy link
Contributor Author

carterm commented Dec 10, 2021

Migrated here...
cagov/github-tree-push#6

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

1 participant