Page MenuHomePhabricator

Regression: Page Previews not using normalised title in requests
Closed, ResolvedPublic1 Estimated Story Points

Description

Symptom

See below.

Example(s)

Dwelling on the "Yorkshire Terrier" link on https://en.wikipedia.beta.wmflabs.org/wiki/Dog causes the following HTTP requests:

Screen Shot 2017-06-12 at 08.45.26.png (66×558 px, 19 KB)

Cause

This regression was introduced in rEPOP35bf61396451: eventLogging: Add missing *Hover properties.

Event Timeline

Change 358338 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/extensions/Popups@master] actions/rest: Use DB-key version of title

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

Jhernandez reassigned this task from Jhernandez to phuedx.
Jhernandez added a subscriber: Jhernandez.

Change 358338 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] actions/rest: Use DB-key version of title

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

Better! Dwelling on the link now causes the following HTTP request:

Screen Shot 2017-06-12 at 12.07.27.png (45×314 px, 5 KB)

Today I noticed a significant latency regression for page previews, and @Pchelolo realized that this task might be the explanation. A large percentage of requests now get redirect responses, which increased overall latency significantly:

image.png (1×1 px, 300 KB)

I can backport the fix and deploy it either this evening or tomorrow morning (BST) /cc @Jdlrobson

I'll sign off/swat today.

The patch can't be cleanly picked to wmf/1.30.0-wmf.4 (even if I do that manually and run npm run compile)
The change looks good, but @phuedx would you mind doing the SWATing?
I don't think I'm going to make it till 4pm today :(

The patch can't be cleanly picked to wmf/1.30.0-wmf.4 (even if I do that manually and run npm run [build])
The change looks good, but @phuedx would you mind doing the SWATing?

Sure! Thanks for picking it up in the first place!

Change 358528 had a related patch set uploaded (by Phuedx; owner: Phuedx):
[mediawiki/extensions/Popups@wmf/1.30.0-wmf.4] actions/rest: Use DB-key version of title

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

Change 358528 merged by jenkins-bot:
[mediawiki/extensions/Popups@wmf/1.30.0-wmf.4] actions/rest: Use DB-key version of title

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

Mentioned in SAL (#wikimedia-operations) [2017-06-13T13:13:40Z] <hashar@tin> Synchronized php-1.30.0-wmf.4/extensions/Popups: actions/rest: Use DB-key version of title - T167633 (duration: 00m 41s)

phuedx set the point value for this task to 1.