Currently, we're having a number of problems on test setup, primarily with 'failure to create dependency directory' and 'failed to clear checkout directory' issues.
The first is invariably due to "Unable to create .git/index.lock: File exists" issues, and the second is typically associated with a "rm -rf failed: Permission denied" response.
Because these are testbot setup issues, and not related to the file under test, I think the directory clearing should occur before the 'next' test request is made ... this will allow us to validate that the directory is indeed clear, take remedial action if not, and not pick up a test if the bot is in a bad state.
The '.git/index.lock: File exists' issue is particularly interesting ... I've seen this occur on a bot 2 seconds after a successful 'rm -rf sites/default/files/checkout'. This seems to indicate that we may actually have two git processes running in parallel on that bot.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 1903118.patch | 2.6 KB | jthorson |
Comments
Comment #1
jthorson commentedThe setup step should also verify that there are no active git processes currently running ... I suspect that a test cancellation in the middle of a git checkout doesn't actually stop git from creating additional files.
Comment #2
jthorson commentedSomething like this ... I know that this approach ends up with two 'clear directory' attempts per test run ... but given the number of current issues this has been causing, I don't really see that as a bad thing!
Comment #3
jthorson commentedCommitted to 6.x-2.x (89828de).
Comment #4
jthorson commentedReverted in 6.x-2.x (e440b6a).