Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
base system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 Nov 2015 at 18:18 UTC
Updated:
17 Jul 2016 at 15:04 UTC
Jump to comment: Most recent, Most recent file
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.