Gather can't merge code due to issue with karma (jenkins mwext-qunit job)
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Jdlrobson
	May 19 2015, 5:24 PM

Description

I'm not sure what changed recently but all patches to Gather are currently failing.
I can replicate this locally with karma but cannot identify the exact reason (seems to be related to initialisation code in modules that we are currently working to remove but it is not an easy fix)

I have no idea why this has suddenly become a problem since the code that is a problem in MobileFrontend (and MobileFrontend is not being impacted by this problem)

I would appreciate some insight asap from @Krinkle or anyone else who knows this stuff as this is blocking us from doing work on Gather.
Seems to have broken sometime between 14th and 18th of May.

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		Jdlrobson	T99686 Gather can't merge code due to issue with karma (jenkins mwext-qunit job)
		Resolved		hashar	T99877 MobileFrontend qunit tests should run Gather tests as well

Event Timeline

Jdlrobson created this task.May 19 2015, 5:24 PM

Jdlrobson raised the priority of this task from to High.

Jdlrobson updated the task description. (Show Details)

Jdlrobson added projects: Gather, Release-Engineering-Team, Jenkins.

Jdlrobson added subscribers: Jdlrobson, Krinkle.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 19 2015, 5:24 PM

Jdlrobson added a project: Gather Sprint Help!.May 19 2015, 5:53 PM

Jdlrobson set Security to None.

Jdlrobson moved this task from Needs triage to This Sprint on the Gather board.

Example: https://integration.wikimedia.org/ci/job/mwext-qunit/1011/console

phuedx subscribed.May 19 2015, 6:27 PM

The jobs fail with:

        Modules in error state
 	Expected: 
	Actual: mobile.notifications,mobile.notificat....

That is an assertion in MediaWiki core QUnit test runner which reports that bunch of RL modules have errors. The mw-debug-www.log debug file attached to the build does not seem to offer any hint though.

Luckily you can reproduce it on your local instance. Maybe you can try bisecting by using a commit of mw/core that was known to work and repeat with other extensions involved.

Seems to me Gather is no more compatible with whatever MobileFrontend change occurred. The qunit job running running for MF do not include Gather.

I think this should get fixed by
https://gerrit.wikimedia.org/r/212036
and https://gerrit.wikimedia.org/r/#/c/212032/ but these are big dependency changes.

I'm not sure what made this surface only in the last 4 days in Gather but not MobileFrontend (the issue is in MobileFrontend code) but it seems to be due to code that expects the MediaWiki HTML due to some legacy code issues.

@phuedx @kaldari I would appreciate review.
I notice MobileFrontend is not using karma @hashar - is that because of this issue?

Jdlrobson claimed this task.May 19 2015, 10:59 PM

Jdlrobson moved this task from In Analysis to In development on the Gather Sprint Help! board.

Jdlrobson moved this task from In development to Code review needed on the Gather Sprint Help! board.

it seems reverting https://gerrit.wikimedia.org/r/#/c/212050/ also fixes this @phuedx

https://gerrit.wikimedia.org/r/#/c/212211/1/resources/mobile.mainMenu/MainMenu.js may also fix this

@hashar sounds like MobileFrontend needs to run Gather's tests...?

@Jdlrobson: Did 212211 fix this?

This is now fixed. Summary: a seemingly harmless change in MobileFrontend broke Gather QUnit tests running in desktop. I've setup a sub task to stop this happening again.

Jdlrobson moved this task from Code review needed to Done on the Gather Sprint Help! board.May 21 2015, 8:50 AM

greg moved this task from INBOX to Done on the Release-Engineering-Team board.May 23 2015, 2:13 PM

Jdlrobson closed subtask T99877: MobileFrontend qunit tests should run Gather tests as well as Resolved.Jun 18 2015, 10:22 PM

Gather can't merge code due to issue with karma (jenkins mwext-qunit job)Closed, ResolvedPublicActions

Description

Related ObjectsSearch...

Event Timeline

Gather can't merge code due to issue with karma (jenkins mwext-qunit job)
Closed, ResolvedPublic
Actions

Related Objects
Search...