Bug 313472
| Summary: | Make CleanGitRepo robust against stale git lock files left by interrupted builds | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> |
| Component: | New Bugs | Assignee: | Aakash Jain <aakash_jain> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Other | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Aakash Jain
The CleanGitRepo step in ews-build can get permanently stuck when a previous git process crashes or is killed, leaving behind files like .git/packed-refs.lock, .git/packed-refs.new, or .git/index.lock. Since the cleanup step doesn't remove these files before running git operations, every subsequent build fails with errors like:
Unable to create '.git/packed-refs.lock': File exists.
Another git process seems to be running in this repository
The step already removes .git/gc.log as a stale artifact, but doesn't handle git lock/temp files. This means a single interrupted git operation can cause a worker to fail indefinitely until manual intervention.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/175701473>
Aakash Jain
Pull request: https://github.com/WebKit/WebKit/pull/63731
EWS
Committed 312216@main (7e49a67a5c31): <https://commits.webkit.org/312216@main>
Reviewed commits have been landed. Closing PR #63731 and removing active labels.