Active
Project:
DrupalCI: Drupal.org Testing Infrastructure
Component:
Policy
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 Oct 2015 at 19:28 UTC
Updated:
2 Mar 2018 at 01:30 UTC
Jump to comment: Most recent
Comments
Comment #2
MixologicUnless Im mistaken, this is not how the current PIFT/PIFR bots behave, and therefore not a blocker to shutting them off.
This is something drupalCI should do, and we'll add it, but not before we shut off the old bots.
Comment #3
claudiu.cristeaIf you shut them off before fixing this, it will make possible to commit code with failed tests in core. Anyway many of reviewers/committers ar looking only to DrupalCI results because the other one gives random failures. So, I think, these issues are both critical and blockers.
Comment #4
MixologicWe definitely need to fix the fatal error that is related to this issue, but Strict Standards warnings are not indicative of a failed test, nor did they fail before on the old testbots.
Comment #5
claudiu.cristeaThis I don't know. IMO such patches should be reworked to fix the warning. But, who knows...
Comment #6
fabianx commentedYes, they failed on the old bots - hence the other core critical.
That is why this is critical and hence blocking turning off of the old bots.
Comment #7
MixologicI just went and looked deeper,
The old testbots did *not* run PHP Strict Standards at all, so this would not be a regression from current functionality.
Whether or not tests *should* fail on Strict Standards warnings is another matter entirely. It may be that we want to emit debug messages, or something else to indicate a warning, but not necessarily fail a whole test for it.
Comment #8
fabianx commentedSorry, I mixed things up with #2580293: Patch having test with "PHP Fatal error" is marked as PASSED, which is the critical part.
Comment #9
MixologicThis issue will be for discussing failures of strict standards. Added a related issue to at least provide that information, regardless of whether or not it fails the tests.
Comment #10
Mixologic