Problem/Motivation
https://www.drupal.org/pift-ci-job/153442
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 2620104-php54-12.patch | 13.11 KB | twistor |
https://www.drupal.org/pift-ci-job/153442
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 2620104-php54-12.patch | 13.11 KB | twistor |
Comments
Comment #2
catchComment #3
David_Rothstein commentedThe testbot results linked from the issue summary no longer exist for some reason, so here's a no-op patch that we can use to get direct test results in this issue.
Also I think this should be major rather than critical unless we identity something that's actually seriously broken on PHP 5.4 (but as far as I know there isn't; the test failures are likely to be more minor issues). It would definitely be nice to be able to reliably use the Drupal 7 testbot on PHP 5.4 though.
Comment #4
twistor commentedComment #5
twistor commentedThis should fix everything except UpgradePathTaxonomyTestCase.
Comment #6
twistor commentedComment #7
defconjuan commentedComment #8
twistor commentedComment #9
twistor commentedI've created 4 sub-issues for the related failures:
The testbot seems to be having random failures for unrelated things making this more difficult than necessary.
Comment #10
twistor commentedI've managed to convince the testbot to pass all 4 issues above.
There were 4 failures in total. Each issue brings it down to 3 failures. Here's a test to prove all failures are fixed.
Comment #12
twistor commentedLast patch contained some other nonsense.
Comment #14
twistor commentedComment #16
twistor commentedComment #17
twistor commented5.3 and 5.4 are both passing with the related issues compiled into a single patch. I added 5.5 just for fun.
Comment #18
isntall commentedI just ran the patch from #12
So this looks to be an issue with the concurrency causing tables to be deleted before they should.
On the old QA system we set the concurrency of Drupal 7 to 8, now we set the concurrency of 31. There were some issues with Drupal 8 and concurrency a while ago, though I don't remember the node IDs.
Comment #19
fabianx commentedComment #20
fabianx commentedComment #21
fabianx commentedFixed PHP 5.4 passes now as far as I can see.