Page MenuHomePhabricator

Deploy Reading Lists Service to production
Closed, ResolvedPublic

Description

Scheduled for Thursday Dec 7, 22h UTC.
FWIW the apps won't start using the API this year, so it won't get nontrivial load for at least a month or so after deployment.

Steps

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
Tgr renamed this task from Deploy ReadingLists to production to Deploy Reading Lists Service to production.Nov 21 2017, 11:51 PM
Tgr updated the task description. (Show Details)

Change 395687 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[operations/mediawiki-config@master] Deploy ReadingLists to production

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

Change 395688 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[operations/mediawiki-config@master] Enable ReadingLists on all SUL wikis

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

Change 395694 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[operations/puppet@production] Add cron job for purging ReadingLists data

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

@jcrespo does this seem OK? RL is set to use extension1 / wikishared (patch) and the plan for setting up the DB is running mwscript sql.php --cluster external1 --wikidb wikishared /srv/mediawiki-staging/php-1.31.0-wmf.11/extensions/ReadingLists/sql/readinglists.sql (readinglists.sql).

Tgr updated the task description. (Show Details)

WRT past performance discussions (such as T164990#3264314 and T168984#3659998), see T182053#3814254 for query performance and 8f11305c7b1e for limits.

Change 395687 merged by jenkins-bot:
[operations/mediawiki-config@master] Deploy ReadingLists to testwiki

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

Mentioned in SAL (#wikimedia-operations) [2017-12-07T23:22:43Z] <tgr@tin> Started scap: T181107 deploy ReadingLists to testwiki

Mentioned in SAL (#wikimedia-operations) [2017-12-07T23:47:32Z] <tgr@tin> Finished scap: T181107 deploy ReadingLists to testwiki (duration: 24m 44s)

Change 395688 merged by jenkins-bot:
[operations/mediawiki-config@master] Enable ReadingLists on all SUL wikis

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

Mentioned in SAL (#wikimedia-operations) [2017-12-07T23:59:49Z] <tgr@tin> Synchronized wmf-config/InitialiseSettings.php: T181107 enable ReadingLists on all wikis (duration: 00m 46s)

The MediaWiki part is now live everywhere. Follow-up item: fix T157651: sql.php must not run LoadExtensionSchemaUpdates which caused a bit of headache. (I guess I deserve it for not fixing when I first ran into it.)

@mobrovac @Pchelolo the MediaWiki side is now live. Any thoughts on scheduling the RESTBase deployment?

@Tgr given that the next week is the last one before deployment freeze an dI assume you want to be able to test against it during the freeze, we must get it out next week.

Monday or Tuesday we will be deploying quite huge Parsoid storage changes and will need at least a day of calm weather after that, but since your changes do not rely on storage at all and there will be no additional traffic, I think we can stack the reading lists together with parsoid deploy. So, if @mobrovac agrees I'd say we do it Mon/Tue.

In the meantime, could you create a PR from your reading_lists_beta brunch into master for the final glance over the changes?

Yup, +1. From what I can see, all concerns have been addressed so we should be good to go. I propose to deploy it Monday, 2017-12-11 during EU or SF hours, whichever works best for you guys (as you should be around when this happens).

PR #926. Thanks! SF hours would be better.

There is one thing missing that I would like you to address- I believe these tables will be private. While we do not replicate x1 to labsdb hosts, we do not discard that happening at some point. There is 2 places were we record private tables- if you could add the ones you added there, that would help prevent data leaks:

(yes, this is meaningless duplication, this is to be addressed at T103011)

We started adding x1 tables there, too for fear that they could end up in labs.

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:26:38Z] <mobrovac@tin> Started deploy [restbase/deploy@bce2885]: Expose the Reading Lists end points - T181107

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:28:05Z] <mobrovac@tin> Finished deploy [restbase/deploy@bce2885]: Expose the Reading Lists end points - T181107 (duration: 01m 26s)

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:28:35Z] <mobrovac@tin> Started deploy [restbase/deploy@bce2885]: Expose the Reading Lists end points, take #2 - T181107

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:34:07Z] <mobrovac@tin> Finished deploy [restbase/deploy@bce2885]: Expose the Reading Lists end points, take #2 - T181107 (duration: 05m 33s)

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:40:55Z] <mobrovac@tin> Started deploy [restbase/deploy@be7d72f]: Expose the Reading Lists end points, take #3 - T181107

Mentioned in SAL (#wikimedia-operations) [2017-12-11T19:47:14Z] <mobrovac@tin> Finished deploy [restbase/deploy@be7d72f]: Expose the Reading Lists end points, take #3 - T181107 (duration: 06m 19s)

mobrovac edited projects, added Services (done); removed Services (doing).

The RESTBase side of things has been deployed \o/

Tgr updated the task description. (Show Details)
Tgr updated the task description. (Show Details)

Change 395694 merged by Jcrespo:
[operations/puppet@production] Add cron job for purging ReadingLists data

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

Tgr claimed this task.
Tgr updated the task description. (Show Details)

Change 404049 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/extensions/ReadingLists@master] Fix purge.php parameter handling

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

Change 404049 merged by jenkins-bot:
[mediawiki/extensions/ReadingLists@master] Fix purge.php parameter handling

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

Change 409645 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[operations/puppet@production] Re-enable cron job for purging ReadingLists data

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

Change 409645 merged by Filippo Giunchedi:
[operations/puppet@production] Re-enable cron job for purging ReadingLists data

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