[16:27:38] <dr0ptp4kt>	 raynor: around?
[16:29:23] <raynor>	 yes
[16:29:57] <raynor>	 dr0ptp4kt: whats up ?
[16:30:11] <dr0ptp4kt>	 raynor: wanted to check in on a few things. able to do a google hangout?
[16:31:05] <raynor>	 yes, sure
[16:33:40] <niedzielski>	 dbrant mdholloway bearND : o/ i think gerrit is really messed up. i seem to get emails about new patches but no reviewers are being added. for example: https://gerrit.wikimedia.org/r/#/c/326457/
[16:52:01] <Reedy>	 jdlrobson: About?
[16:52:22] <Reedy>	 niedzielski: Reviewers are done by a bot... Which may be broken atm
[16:52:52] <niedzielski>	 Reedy: ok thanks
[16:53:11] <Reedy>	 https://www.mediawiki.org/wiki/Git/Reviewers
[17:35:34] <dbrant>	 mdholloway: hey, when's the next service deploy?
[17:35:58] <mdholloway>	 dbrant: next one scheduled is 4pm today
[17:37:28] <dbrant>	 mdholloway: ok great. (i'll have a tiny patch)
[18:40:58] <jdlrobson>	 Reedy: hey! yes!
[18:45:20] <Reedy>	 jdlrobson: Can you review that GCI task? :D
[18:45:21] <Reedy>	 <3
[18:45:33] <jdlrobson>	 Reedy: which one - ive got a few on my plate
[18:45:55] <jdlrobson>	 link me up
[18:47:34] <jdlrobson>	 If RelatedArticles one Reedy im on it
[18:51:08] <Reedy>	 Perfect, thanks
[19:46:47] <ebernhardson>	 how can i trigger the different between <lang>.<project>.org and <lang>.m.<project>.org on a local vagrant install? useskin=minerva gets half way there, but it doesn't seem to be the same because the resources included are different. I'm trying to test if changing a resource definition to have 'targets' => ['desktop','mobile'] will have the desired effect
[19:48:12] <niedzielski>	 ebernhardson: can you click mobile view at the bottom (&mobileaction=toggle_view_mobile)?
[19:48:56] <ebernhardson>	 checking
[19:54:24] <ebernhardson>	 niedzielski: that seems to have done the trick, thanks!
[19:54:46] <niedzielski>	 ebernhardson: sweet!
[20:03:31] <mdholloway>	 ^ i actually often find this behavior flaky in a local vagrant instance
[20:03:47] <mdholloway>	 as in, clicking the mobile view link does nothing
[20:03:54] <mdholloway>	 i've managed to work around it as needed, though
[20:31:53] <Pchelolo>	 bearND: mdholloway: just to double-check: /feed/announcements is only used on en.wikipedia domain and nothing will happen if we remove it from the other domains, right?
[20:32:27] <bearND>	 Pchelolo: well, it's supposed to return an empty array on other domains
[20:32:45] <Pchelolo>	 but what if it starts returning 404?
[20:33:15] <bearND>	 I can check the Android code. Not sure how the iOS code would behave
[20:33:48] <Pchelolo>	 we're going to split out a special project for enwiki in RESTBase and so we can remove the announcements from all the other domain's docs
[20:34:23] <mdholloway>	 bearND: Pchelolo: the Android app *should* just omit the announcement card from the feed if that request 404's, but we should test that first
[20:34:49] <Pchelolo>	 but if we're not entirely sure this won't break the apps I'd better be safe then sorry and not remove it..
[20:34:58] <Pchelolo>	 At least make a separate PR for that.
[20:39:13] <bearND>	 When I wrote https://phabricator.wikimedia.org/T151085 I added that clients should restrict this to enwiki for now and stop sending requests if a 404 is encountered for the announcement endpoint (for the case this endpoint goes away) but it looks like those have not been implemented
[20:59:05] <mdholloway>	 bearND: if you agree with putting back the manual blacklist for most-read, want to try to get that into today's deployment? https://gerrit.wikimedia.org/r/#/c/326544
[21:00:06] <mdholloway>	 bearND: actually, that might need a sanity check, don't merge just yet (if you were)
[21:00:43] <bearND>	 mdholloway: ok, was about to. But it can probably wait until tomorrow, too
[21:05:53] <mdholloway>	 bearND: yeah, let's wait on it.  it won't take long to fix but it's not not critical.  needs updating to work with the $merge nodes
[21:05:58] <mdholloway>	 i'll start the deploy prep