Last modified: 2013-11-24 17:46:58 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T59488, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57488 - ERROR! It seems this job introduced a regression although it might be caused by a previous commit.
ERROR! It seems this job introduced a regression although it might be caused ...
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Continuous integration (Other open bugs)
unspecified
All All
: Unprioritized major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-23 20:23 UTC by Bartosz Dziewoński
Modified: 2013-11-24 17:46 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Bartosz Dziewoński 2013-11-23 20:23:40 UTC
https://integration.wikimedia.org/ci/job/mediawiki-core-regression-master/3488/console : ERROR! It seems this job introduced a regression although it might be caused by a previous commit. in 3m 32s

This is happening on almost every commit, even ones which don't even touch PHP files. Please unfuck this up, I'm fed up with these stupid false errors and ignoring possibly real ones.
Comment 1 Aude 2013-11-23 22:25:23 UTC
the issue seems to be introduced with https://gerrit.wikimedia.org/r/#/c/74170/ although I am not quite sure why.

Possibly the regression tests are being run with $wgDevelopmentWarnings = false or something is different with how they are run (e.g. without --group) that is causing the issue.  The issue does not appear with the gate-submit and other jobs.

wfWarn is supposed to throw a warning with $wgDevelopmentWarnings and we have phpunit converting warnings, notices and errors to exceptions.


convertErrorsToExceptions="true"                                                                   
convertNoticesToExceptions="true"                                                                  
convertWarningsToExceptions="true"
Comment 2 Gerrit Notification Bot 2013-11-24 00:28:35 UTC
Change 97352 had a related patch set uploaded by Aude:
Call parent::setUp() in DiffHistoryBlobTest before marking skipped tests

https://gerrit.wikimedia.org/r/97352
Comment 3 Gerrit Notification Bot 2013-11-24 09:40:03 UTC
Change 97352 merged by jenkins-bot:
Call parent::setUp() in DiffHistoryBlobTest before marking skipped tests

https://gerrit.wikimedia.org/r/97352
Comment 4 Marius Hoch 2013-11-24 10:34:10 UTC
Approved the change, looks good now.
Comment 5 Bartosz Dziewoński 2013-11-24 17:46:58 UTC
Truly, this seems to have stopped the madness. Aude, you are my favorite person right now.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links