Page MenuHomePhabricator
Create Task
Maniphest T91967

Update to Pimple 2.x in Flow broke merge pipelines for Thanks, VisualEditor
Closed, ResolvedPublic
Actions

Description

See e.g. https://gerrit.wikimedia.org/r/#/c/194437/ , which now fails with

13:06:34 1) ApiFlowThankTest::testRequestWithoutToken
13:06:34 RuntimeException: Cannot override frozen service "storage".

I suspect this was caused by https://gerrit.wikimedia.org/r/#/c/192608/ . For some reason this only affects the HHVM tests, not the Zend tests, which is why Jenkins didn't prevent this from being merged: Flow does not run its unit tests in HHVM, but Thanks and VE do.

Details

ProjectBranchLines +/-Subject
mediawiki/extensions/Thankswmf/1.25wmf20+4 -4Flow container needs to be reset before each test
mediawiki/extensions/Thanksmaster+4 -4Flow container needs to be reset before each test
Customize query in gerrit

Related Objects

Event Timeline

Catrope created this task.Mar 9 2015, 1:23 PM
Catrope assigned this task to EBernhardson.
Catrope raised the priority of this task from to Unbreak Now!.
Catrope updated the task description. (Show Details)
Catrope added projects: StructuredDiscussions, Thanks, VisualEditor.
Catrope added a subscriber: Catrope.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 9 2015, 1:23 PM
Catrope mentioned this in T91968: mediawiki-extensions-hhvm should run for all covered extensions (eg. Flow).Mar 9 2015, 1:26 PM
Jdforrester-WMF mentioned this in T91974: Run Flow tests in HHVM as well as Zend.Mar 9 2015, 2:51 PM
Jdforrester-WMF moved this task from To Triage to Freezer on the VisualEditor board.
gerritbot added a subscriber: gerritbot.Mar 9 2015, 4:41 PM

Change 195296 had a related patch set uploaded (by Matthias Mullie):
Don't override existing Pimple container thingy, unset it first

https://gerrit.wikimedia.org/r/195296

gerritbot added a project: Patch-For-Review.Mar 9 2015, 4:41 PM
Krenair added a subscriber: Krenair.Mar 9 2015, 4:44 PM
DannyH added a project: Collaboration-Team-Sprint-R-2015-03-11.Mar 9 2015, 5:51 PM
DannyH set Security to None.
DannyH moved this task from Sprint R - ends March 11 to Code Review Started on the Collaboration-Team-Sprint-R-2015-03-11 board.
gerritbot added a comment.Mar 9 2015, 5:56 PM

Change 195296 merged by jenkins-bot:
Flow container needs to be reset before each test

https://gerrit.wikimedia.org/r/195296

EBernhardson mentioned this in rETHAddf1310eea9c: Flow container needs to be reset before each test.Mar 9 2015, 5:57 PM
• Mattflaschen-WMF closed this task as Resolved.Mar 9 2015, 6:02 PM
• Mattflaschen-WMF reassigned this task from EBernhardson to matthiasmullie.
• Mattflaschen-WMF reassigned this task from matthiasmullie to EBernhardson.Mar 9 2015, 6:08 PM
DannyH moved this task from Code Review Started to Product Review on the Collaboration-Team-Sprint-R-2015-03-11 board.Mar 9 2015, 6:38 PM
gerritbot added a comment.Mar 9 2015, 10:47 PM

Change 195470 had a related patch set uploaded (by Jforrester):
Flow container needs to be reset before each test

https://gerrit.wikimedia.org/r/195470

gerritbot added a comment.Mar 9 2015, 10:51 PM

Change 195470 merged by Catrope:
Flow container needs to be reset before each test

https://gerrit.wikimedia.org/r/195470

Jdforrester-WMF mentioned this in rETHA12babdbcb8b9: Flow container needs to be reset before each test.Mar 9 2015, 10:51 PM
matthiasmullie mentioned this in rMEXT9fb155969ee1: Updated mediawiki/extensions Project: mediawiki/extensions/Thanks….Mar 11 2015, 6:28 AM
hashar merged a task: T91951: Jenkins reports test failures in current master: Cannot override frozen service "storage".Mar 12 2015, 8:56 AM
hashar added subscribers: Jdforrester-WMF, Krinkle, • Mattflaschen-WMF and 2 others.
Jdforrester-WMF removed a project: Patch-For-Review.Mar 20 2015, 11:32 PM
Jdforrester-WMF added a project: VisualEditor 2014/15 Q3 blockers.Mar 29 2015, 11:27 PM
Jdforrester-WMF removed a project: VisualEditor 2014/15 Q3 blockers.Mar 29 2015, 11:29 PM
Jdforrester-WMF moved this task from Freezer to Q4 on the VisualEditor board.Apr 16 2015, 9:13 PM
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptApr 16 2015, 9:13 PM
DannyH moved this task from Untriaged to Resolved on the Collaboration-Team-Triage board.Apr 17 2015, 5:34 PM
Jdforrester-WMF added a project: VisualEditor 2014/15 Q3 blockers.Aug 5 2015, 1:16 AM
Restricted Application added projects: Growth-Team, User-Ryasmeen. · View Herald TranscriptSep 2 2018, 10:07 AM
Restricted Application added a subscriber: TerraCodes. · View Herald Transcript
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL