Strip references from HTML on beta cluster beta mode, measure impact
Closed, ResolvedPublic3 Estimated Story Points
Actions

Assigned To

Authored By

	Jdlrobson
	Feb 9 2016, 10:47 PM

Description

Let's do a quick check before going down a rabbit hole to see what impact if any stripping references will have

Ensure this will not clash with T126558

JR: this will not be in the current sprint so we're good.
~~[] Disable navbox experiment (remove .navboxfrom wgMFRemovableClasses )~~
JR: decided not to do this as it would be an unfair test to test 7 days prior to stripping references with a completely different environment)

Update wgMFRemovableClasses to include .references on beta channel and stable channel of beta cluster

JR: this is now live as of Tue 16th February.

Update wgMFRemovableClasses to exclude .references on beta channel and stable channel of beta cluster

JR: this is now live as of Monday 22nd 5:20PM PST

Use script to measure impact on beta cluster post-change for webpagetest.enwiki-bc-mobile-beta-2gslow.anonymous.Barack_Obama.us-east-1.Google_Chrome-emulateMobile.firstView and webpagetest.enwiki-bc-mobile-2gslow.anonymous.Barack_Obama.us-east-1.Google_Chrome-emulateMobile.firstView

JR: Will measure Fri 27th Feb (5 days of data before and after)

Revert both changes after collecting 10 5-10 days worth of data (since we don't want to break sauce labs browser tests of the references feature).

JR: Wasn't necessary.

Details

	Subject	Repo	Branch	Lines +/-
	Revert "Strip references for experimentation"	operations/mediawiki-config	master	+2 -2
	Strip references for experimentation	operations/mediawiki-config	master	+2 -2

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Open	Release	None	T84936 Release VisualEditor-MediaWiki as "1.0"
Open		None	T50429 [Epic] Support editing parts of a page in VisualEditor-MediaWiki
Open		None	T54365 Explore performance gains from progressive (JIT?) de-alienation in VisualEditor
Open		None	T174303 Copy-pasting linked ISBN numbers from view mode HTML into VisualEditor inserts wikitext links to Special:BookSources (it should turn them into magic links?)
Open	Feature	None	T54091 The read HTML should have hinting to allow full DOM copying (as opposed to just rich copying) from read mode into VE surfaces
Open		None	T55784 [EPIC] Use Parsoid HTML for all page views
Resolved		dr0ptp4kt	T114542 Next Generation Content Loading and Routing, in Practice
Duplicate		• Jhernandez	T104432 [EPIC]: Improve mobile site performance
Duplicate		dr0ptp4kt	T120341 [GOAL] Make Wikipedia more accessible to all connections with new fast API-driven web experience in mobile web beta
Declined		None	T125920 [EPIC] Future exciting reading web performance endeavours
Resolved		Jdlrobson	T113066 [GOAL] Make Wikipedia more accessible to 2G connections
Declined		None	T123328 [GOAL] Lazy load references in mobile skin
Resolved		Jdlrobson	T126390 Strip references from HTML on beta cluster beta mode, measure impact

Event Timeline

Jdlrobson created this task.Feb 9 2016, 10:47 PM

Jdlrobson raised the priority of this task from to Medium.

Jdlrobson updated the task description. (Show Details)

Jdlrobson added projects: Reading-Web-Sprint-66-Harry is Tired, Reading-Web-Planning, Reading-Admin.

Jdlrobson subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 9 2016, 10:47 PM

Jdlrobson removed projects: Reading-Admin, Reading-Web-Planning.Feb 9 2016, 10:47 PM

Jdlrobson set Security to None.

Jdlrobson added a project: Web-Team-Backlog.

Jdlrobson updated the task description. (Show Details)Feb 10 2016, 11:46 PM

Jdlrobson updated the task description. (Show Details)Feb 10 2016, 11:51 PM

Jdlrobson mentioned this in T126558: Enable lazy loading of images on beta cluster and measure results.

Jdlrobson updated the task description. (Show Details)Feb 10 2016, 11:54 PM

Jdlrobson mentioned this in T125329: Save references in page_props and cache.Feb 11 2016, 6:30 PM

Jdlrobson updated the task description. (Show Details)Feb 16 2016, 5:37 PM

Jdlrobson updated the task description. (Show Details)

Jdlrobson edited a custom field.Feb 16 2016, 5:41 PM

Jdlrobson moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-66-Harry is Tired board.

Jdlrobson moved this task from To Do to Doing on the Reading-Web-Sprint-66-Harry is Tired board.Feb 16 2016, 9:32 PM

Change 271074 had a related patch set uploaded (by Jdlrobson):
Strip references for experimentation

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

gerritbot added a project: Patch-For-Review.Feb 16 2016, 9:40 PM

Jdlrobson updated the task description. (Show Details)Feb 16 2016, 9:49 PM

Jdlrobson claimed this task.Feb 16 2016, 10:08 PM

Change 271074 merged by jenkins-bot:
Strip references for experimentation

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

This has been SWATed to the beta cluster
Unfortunately due to the unexpected srcset removal that happened earlier today this is not a fair test.

We should look to reenable hdpi images on beta cluster and ignore the period of time both were active for. Alternatively I could revert this change wait 7 days and try again. Too tired to think of best thing to do right now.

Updated plan. Stalled until Monday.

Jdlrobson moved this task from Doing to To Do on the Reading-Web-Sprint-66-Harry is Tired board.Feb 17 2016, 7:16 PM

Jdlrobson updated the task description. (Show Details)Feb 17 2016, 7:19 PM

Change 271322 had a related patch set uploaded (by Jdlrobson):
Revert "Strip references for experimentation"

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

Peter subscribed.Feb 22 2016, 9:30 AM

Jdlrobson moved this task from To Do to Doing on the Reading-Web-Sprint-66-Harry is Tired board.Feb 22 2016, 5:03 PM

SWAT scheduled

Change 271322 merged by jenkins-bot:
Revert "Strip references for experimentation"

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

I can confirm references are now back in the HTML of beta cluster. Should soon see this in the graphs.

Jdlrobson updated the task description. (Show Details)Feb 23 2016, 1:27 AM

Jdlrobson moved this task from Code Review to Doing on the Reading-Web-Sprint-66-Harry is Tired board.Feb 23 2016, 6:04 PM

Jdlrobson updated the task description. (Show Details)Feb 23 2016, 8:43 PM

Jdlrobson updated the task description. (Show Details)

Report written:
https://www.mediawiki.org/wiki/Reading/Web/Projects/Performance/Lazy_loading_references

Feedback welcomed.

Jdlrobson updated the task description. (Show Details)Feb 27 2016, 1:19 AM

Hypothesis could be made clearer. For example,

Certain content doesn't necessarily need to be shipped to the user upfront and sometimes not at all.

doesn't state a reason.

And, these three lines don't have commands:

The commands used to measure the impact of the change were:
The commands used to measure the impact of the change were:
The commands used to measure the impact of the change were:

bd808 edited projects, added Reading-Web-Sprint-67-If, Then, Else...?; removed Reading-Web-Sprint-66-Harry is Tired.Mar 1 2016, 5:06 PM

bd808 moved this task from Needs Analysis to Ready for Signoff on the Reading-Web-Sprint-67-If, Then, Else...? board.