Deploy new header to all wikis
Closed, ResolvedPublic2 Estimated Story Points
Actions

Assigned To

Authored By

	ovasileva
	Mar 14 2017, 9:36 PM

Description

Background

After the test on Catalan and Italian wikipedias, we have noted that internal page views (including internal search) have not been affected. Thus, we are ready to go ahead with the new branding deploy

Acceptance criteria

Ensure OK from new readers team (olga)
Ensure OK from discovery team (olga)
Publish announcement for deployment (chris)
Deploy new header on March 27 (actually deployed on 28)
Cleanup any artifacts relating to the feature flag and remove the feature flag - as reverting after this point would be a very visible change that should be avoided. rEMFRe12fdd73f714: Hygiene: Remove header-v2 and header-v1 code (correctly)

Testing Notes

Ensure that the header on the Beta Cluster mobile site and, say, https://en.m.wikipedia.org behave the same way across mobile, tablet, and desktop, focussing on layout and visible icons.

Details

Subject	Repo	Branch	Lines +/-
Hygiene: Remove header-v2 and header-v1 code (correctly)	mediawiki/extensions/MobileFrontend	master	+16 -58
Hygiene: Remove header-v2 and header-v1 code	mediawiki/extensions/MobileFrontend	master	+11 -55
HeaderV2 is the default and not configurable	mediawiki/extensions/MobileFrontend	master	+1 -27
Enable header version 2 on all wikis	operations/mediawiki-config	master	+2 -1

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		ABorbaWMF	T160471 Deploy new header to all wikis
		Resolved		ovasileva	T156796 Analyze results of new header

Event Timeline

ovasileva created this task.Mar 14 2017, 9:36 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 14 2017, 9:36 PM

ovasileva triaged this task as High priority.Mar 14 2017, 9:36 PM

ovasileva moved this task from Incoming to Upcoming on the Web-Team-Backlog board.

ovasileva updated the task description. (Show Details)

ovasileva added a subscriber: Ckoerner.

• Nirzar awarded a token.Mar 14 2017, 9:37 PM

ovasileva moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-94 board.Mar 14 2017, 9:41 PM

Jdlrobson added a project: Wikimedia-Site-requests.Mar 15 2017, 12:27 AM

Restricted Application added a subscriber: Dereckson. · View Herald TranscriptMar 15 2017, 12:27 AM

Needs estimation. Team hasn't said it's ready for "ToDo" yet.

Jdlrobson added a subtask: T156796: Analyze results of new header.Mar 15 2017, 5:16 PM

ovasileva moved this task from Needs Analysis to Blocked Externally on the Reading-Web-Sprint-94 board.Mar 15 2017, 5:45 PM

Jdlrobson updated the task description. (Show Details)Mar 15 2017, 5:47 PM

Jdlrobson set the point value for this task to 2.

CKoerner_WMF edited subscribers, added: CKoerner_WMF; removed: Ckoerner.Mar 15 2017, 7:30 PM

Change 342973 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend] HeaderV2 is the default and not configurable

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

gerritbot added a project: Patch-For-Review.Mar 16 2017, 12:06 AM

Change 342974 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend] Hygiene: Remove header-v2 and header-v1 code

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

Jdlrobson changed the task status from Open to Stalled.Mar 17 2017, 4:58 PM

MBinder_WMF moved this task from Upcoming to 2016-17 Q3 on the Web-Team-Backlog board.Mar 21 2017, 4:03 PM

Dereckson moved this task from Backlog to External on the Wikimedia-Site-requests board.Mar 21 2017, 10:38 PM

Jdlrobson changed the task status from Stalled to Open.Mar 28 2017, 4:58 PM

Jdlrobson moved this task from Blocked Externally to To Do on the Reading-Web-Sprint-94 board.

Jdlrobson updated the task description. (Show Details)

Change 345184 had a related patch set uploaded (by Jdlrobson):
[operations/mediawiki-config@master] Enable header version 2 on all wikis

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

Jdlrobson moved this task from To Do to Needs Code Review on the Reading-Web-Sprint-94 board.Mar 28 2017, 5:01 PM

Change 345184 merged by jenkins-bot:
[operations/mediawiki-config@master] Enable header version 2 on all wikis

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

Mentioned in SAL (#wikimedia-operations) [2017-03-28T23:16:31Z] <thcipriani@tin> Synchronized wmf-config/InitialiseSettings.php: SWAT: [[gerrit:345184|Enable header version 2 on all wikis]] T160471 (duration: 00m 45s)

https://gerrit.wikimedia.org/r/#/c/342973/ can now safely be merged.
https://gerrit.wikimedia.org/r/#/c/342973/2 should only be merged when the cache is clear. We can probably merge that now, but if we want to be more cautious, we may want to leave that till next week.

Jdlrobson updated the task description. (Show Details)Mar 29 2017, 3:53 PM

Jdlrobson updated the task description. (Show Details)

Change 342973 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] HeaderV2 is the default and not configurable

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

ReleaseTaggerBot added a project: MW-1.29-release (WMF-deploy-2017-04-04_(1.29.0-wmf.19)).Mar 29 2017, 8:00 PM

Change 342974 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Hygiene: Remove header-v2 and header-v1 code

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

In T160471#3140686, @Jdlrobson wrote:

https://gerrit.wikimedia.org/r/#/c/342973/ can now safely be merged.
https://gerrit.wikimedia.org/r/#/c/342973/2 should only be merged when the cache is clear. We can probably merge that now, but if we want to be more cautious, we may want to leave that till next week.

As we discussed earlier, if I'm reading rOPUP57780729352d: varnish: swap around backend ttl cap and keep values [1/2] correctly, then the maximum lifetime of an object in The Cache (cache_text) is 3 days.

phuedx assigned this task to ovasileva.Mar 29 2017, 10:23 PM

phuedx moved this task from Needs Code Review to Ready for Signoff on the Reading-Web-Sprint-94 board.

• MZMcBride subscribed.Mar 30 2017, 12:04 AM

Jdlrobson moved this task from Ready for Signoff to Needs Code Review on the Reading-Web-Sprint-94 board.Mar 30 2017, 12:38 AM

Change 345487 had a related patch set uploaded (by Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Hygiene: Remove header-v2 and header-v1 code (correctly)

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

I've reverted the original patch as I found an issue with it (the search box was showing in mobile one).
New one above.

Can you take another pass at CR?

There's a question from @Krinkle against rEMFRe7f41b3959c0: Hygiene: Remove header-v2 and header-v1 code (correctly). I posed it the other way around and C-1'd the change.

Jdlrobson moved this task from Needs More Work to Doing on the Reading-Web-Sprint-94 board.Apr 4 2017, 4:41 PM

Thanks @Krinkle and @phuedx. I moved this into one css rule to make this clearer. Hope it's good to go now.

@Jdlrobson - I just noticed this in the analyze results task: "Create task for full deployment. As part of this task, please capture in acceptance criteria the removal of the HeaderV2 config flag and the feature-header classes" - did we ever make a separate task/finish it as a part of the this task?

ovasileva closed subtask T156796: Analyze results of new header as Resolved.Apr 5 2017, 1:09 PM

This is that task and all that is captured in acceptance criteria. https://gerrit.wikimedia.org/r/345487 is the last piece.

ovasileva updated the task description. (Show Details)Apr 5 2017, 2:51 PM

Jdlrobson added a project: Reading-Web-Sprint-95.Apr 5 2017, 5:14 PM

Jdlrobson moved this task from Needs Analysis to Needs Code Review on the Reading-Web-Sprint-95 board.

Change 345487 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Hygiene: Remove header-v2 and header-v1 code (correctly)

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

ReleaseTaggerBot edited projects, added MW-1.29-release (WMF-deploy-2017-04-11_(1.29.0-wmf.20)); removed MW-1.29-release (WMF-deploy-2017-04-04_(1.29.0-wmf.19)).Apr 6 2017, 12:00 PM

phuedx reassigned this task from Jdlrobson to ovasileva.Apr 6 2017, 1:37 PM

phuedx updated the task description. (Show Details)

phuedx moved this task from Needs Code Review to Ready for Signoff on the Reading-Web-Sprint-95 board.

Actually, while this is mostly a process task, there were minor changes to the codebase (rEMFRe12fdd73f714: Hygiene: Remove header-v2 and header-v1 code (correctly)), this should be passed to @ABorbaWMF with instructions to check that the Beta Cluster and one/many of the Wikipedias headers behave the same way across devices.

phuedx moved this task from Ready for Signoff to Needs QA on the Reading-Web-Sprint-95 board.Apr 6 2017, 1:48 PM

phuedx updated the task description. (Show Details)Apr 6 2017, 1:50 PM

Jdlrobson removed a project: Patch-For-Review.Apr 6 2017, 5:40 PM

Iniquity added a project: Mobile.Apr 10 2017, 9:42 PM

Did some testing. Here are some screenshots. There is a small issue with the mobile version on IE browsers.

System	Screenshot
	Nexus 9 Android 6.0 - Chrome 55
	Nexus 6P Android 6.0 - Firefox 48
	Galaxy Tab S Android 4.4 - Chrome 43
	iPad Air 2 iOS 9.0 - Safari 9.0
	iPhone 5S iOS 7.0 - Safari 7.0
	iPhone 6S iOS 9.0 - Chrome 47
	Windows 7(64) - IE9
	Windows 8 - IE10
	Windows 8.1 - IE11
	Windows 10 - Edge
	Mac OSX 10.8 - Chrome 48(64)
	Mac OSX 10.12 - Firefox

Thanks @ABorbaWMF. Does T162647 capture the bug? If so I suggest we resolve this and prioritise and fix the newly introduced bug separately. I've moved it into current sprint so we can estimate/plan it during kick off.

Jdlrobson mentioned this in T162647: Mobile header shows both bell icon and red notification circle in Internet Explorer 9-11.Apr 10 2017, 11:37 PM

@Jdlrobson Yes that's the bug. Thanks

Krinkle unsubscribed.Apr 11 2017, 12:43 AM

ovasileva closed this task as Resolved.Apr 11 2017, 8:17 AM

MBinder_WMF moved this task from 2016-17 Q3 to 2016-17 Q4 on the Web-Team-Backlog board.Apr 13 2017, 7:10 PM

	F7472865: Header-Mac OSX 10.8 - Chrome 48(64).png
	Apr 10 2017, 11:25 PM

	F7472800: Header-iPhone 5S iOS 7.0 - Safari 7.0.png
	Apr 10 2017, 11:25 PM

	F7472787: Header-iPadAir2 iOS9.0 - Safari 9.0.png
	Apr 10 2017, 11:25 PM

	F7472766: Header-Galaxy Tab S A 4.4 - Chrome 43.png
	Apr 10 2017, 11:25 PM

	F7472813: Header-iPhone 6S iOS 9.0 - Chrome 47.png
	Apr 10 2017, 11:25 PM

Deploy new header to all wikisClosed, ResolvedPublic2 Estimated Story PointsActions