Page MenuHomePhabricator

MCS should request 2.0 Parsoid HTML
Closed, ResolvedPublic

Description

After deploying content negotiation and Parsoid 2.0, MCS should request 2.0 Parsoid HTML in it's Accept header. This should be done sooner rather than later, cause now all requests from MCS to RESTBase during updates require downgrading, so background update latency increased by 200 ms on average.

Event Timeline

Change 463866 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/services/mobileapps@master] Request Parsoid HTML 2.0.0, not 1.7.0

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

Just to be clear, should this be done now, or does something need to be deployed in Parsoid and/or RESTBase first? I see there's a new HTML 2.0.0 spec as of yesterday, but don't see any references in the commit logs.

@Mholloway Everything is deployed in both RB and Parsoid, but let's wait till tomorrow for MCS deployment in case we need to rollback?

Change 463866 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Request Parsoid HTML 2.0.0, not 1.7.0

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

Pchelolo claimed this task.

Beautiful. Look at the update latency impact!

Screen Shot 2018-10-03 at 7.50.58 AM.png (1×2 px, 378 KB)

Resolving. Thank you @Mholloway for testing and getting it out so quickly.