[01:48:59] <Romaine>	 James_F: thanks for the comment here https://bugzilla.wikimedia.org/show_bug.cgi?id=67700
[01:49:25] <Romaine>	 earlier we got the impression this was done within a few weeks, but it is good to know it takes longer
[01:54:04] <James_F>	 Romaine: It was. It completely and catastrophically broke pages when you edited them, however, which is why it was reverted.
[01:54:17] <Romaine>	 ok, good to know
[01:54:35] <James_F>	 We think we sort-of know why it broke, but not how to fix it a different way yet, sorry. :-(
[01:55:08] <Romaine>	 I hope you will find a way, as it is really something we wait for :-)
[01:55:15] * James_F too! :-)
[01:55:21] <Romaine>	 :)
[01:56:29] * Romaine remembers now that I wanted to meet you at Wikimania
[01:57:14] <Romaine>	 a pity
[11:11:59] <wikibugs>	 3MediaWiki / 3File management: Could not delete file "mwstore://local-swift-eqiad/local-public/..." on hu.wp - 10https://bugzilla.wikimedia.org/69717#c1 (10j.l) see bug 69760
[11:19:38] <thedj>	 hmmm
[11:19:39] <thedj>	 t::ERR_CONTENT_LENGTH_MISMATCH https://upload.wikimedia.org/wikipedia/commons/transcoded/7/76/Multimedia_vision_2016.webm/Multimedia_vision_2016.webm.480p.webm
[11:25:47] <wikibugs>	 3MediaWiki / 3File management: Could not delete file "mwstore://local-swift-eqiad/local-public/..." on hu.wp - 10https://bugzilla.wikimedia.org/69717#c2 (10Jesús Martínez Novo (Ciencia Al Poder)) 5NEW>3RESO/DUP Marking as dupe since bug 69760 has more examples  *** This bug has been marked as a duplicate...
[13:56:18] <wikibugs>	 3MediaWiki / 3File management: Could not delete file "mwstore://local-swift-eqiad/local-public/..." on hu.wp - 10https://bugzilla.wikimedia.org/69717#c3 (10Tisza Gergő) Why do you think it is the same issue? The error message is clearly different, and bug 69760 comment 1 claims that issue is intermittent, wh...
[14:06:22] <grrrit-wm1>	 (03PS3) 10Inchikutty: Adds OSM parameter to coordinate template [extensions/UploadWizard] (osm2) - 10https://gerrit.wikimedia.org/r/144324 (https://bugzilla.wikimedia.org/56612) 
[14:06:40] <grrrit-wm1>	 (03CR) 10jenkins-bot: [V: 04-1] Adds OSM parameter to coordinate template [extensions/UploadWizard] (osm2) - 10https://gerrit.wikimedia.org/r/144324 (https://bugzilla.wikimedia.org/56612) (owner: 10Inchikutty)
[14:08:42] <grrrit-wm1>	 (03PS4) 10Inchikutty: Adds OSM parameter to coordinate template [extensions/UploadWizard] (osm2) - 10https://gerrit.wikimedia.org/r/144324 (https://bugzilla.wikimedia.org/56612) 
[15:02:00] <grrrit-wm1>	 (03PS1) 10Gergő Tisza: Fix global optout counts [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/155272 
[15:02:52] <marktraceur>	 Well, good news/bad news
[15:03:02] <marktraceur>	 'm still sick, but at least I can work today.
[15:56:11] <marktraceur>	 Arghhhh
[15:57:13] <marktraceur>	 tgr, gi11es, Keegan|Away, robla - are you ready to start the meeting at 09:00? I see no reason not to start if one person is late
[15:57:38] <marktraceur>	 We can do the community/product parts when Fabrice arrives
[16:00:54] <tgr>	 I'm ready, not sure which parts we can do without Fabrice though
[16:01:25] <marktraceur>	 Evaluate old tasks, carry things forward into the new sprint
[16:01:40] <marktraceur>	 We can hear anything robla has to say (he said he could only make the 09:00-09:30 slot anyway)
[16:02:05] <tgr>	 ok
[16:22:26] <tgr>	 https://bugzilla.wikimedia.org/show_bug.cgi?id=69717
[17:33:13] <marktraceur>	 tgr: Wait, which card was the one that milimetric should look at?
[18:06:01] <fabriceflorin>	 8
[18:07:17] <marktraceur>	 9
[18:50:40] <marktraceur>	 Ugh, I cannot remember why the onbeforeunload caching business was necessary
[19:15:28] <grrrit-wm1>	 (03CR) 10MarkTraceur: Move confirmCloseWindow to core (031 comment) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/151127 (owner: 10MarkTraceur)
[19:15:41] <marktraceur>	 All the patches
[19:41:02] <tgr>	 marktraceur: UW funnel limn config
[19:41:12] <tgr>	 I'll ask him
[19:41:36] <tgr>	 (I clearly need a better way if tracking IRC highlights)
[19:54:50] <marktraceur>	 Ah right
[19:59:41] <grrrit-wm1>	 (03CR) 10MarkTraceur: "I've said in IRC that I'll do those things in a separate patchset. I'll submit that patch now, but this should go in first." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/153818 (https://bugzilla.wikimedia.org/64685) (owner: 10MarkTraceur)
[20:08:22] <grrrit-wm1>	 (03CR) 10Milimetric: Query UploadWizard funnel data (032 comments) [analytics/multimedia/config] - 10https://gerrit.wikimedia.org/r/150750 (owner: 10Gergő Tisza)
[20:14:05] <grrrit-wm1>	 (03PS1) 10MarkTraceur: Add beforeActive and afterActive to campaigns [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/155351 (https://bugzilla.wikimedia.org/64685) 
[20:14:06] <marktraceur>	 THERE.
[20:14:12] <marktraceur>	 Are you people happy now
[20:16:39] <fabriceflorin>	 Thanks, marktraceur — glad we could address that campaign issue in time for WLM. Hopefully, it will make Romaine very happy :)
[20:17:56] <Romaine>	 yesterday I saw something strange had happened, the campaign tables had gone
[20:18:26] <marktraceur>	 Romaine: Uhhhh what
[20:18:41] <marktraceur>	 Resolved now? :/
[20:19:18] <marktraceur>	 Oh, wait, campaigns table? That's not used anymore anyway
[20:20:53] <Romaine>	 it isn't solved
[20:21:24] <marktraceur>	 Romaine: What effects do you see in production?
[20:21:26] <Romaine>	 as I stated before in 2013, the campaign pages are only used for creating the technical parts of campaigns
[20:22:15] <Romaine>	 in 2013, just before the start of WLM, it was considered a good idea to remove the grey tables and instead show the pictures
[20:22:39] <Romaine>	 one developer was so nice to write a piece of .js to get that back, but now that is gone
[20:23:32] <marktraceur>	 Huh.
[20:23:38] <marktraceur>	 Romaine: Do you remember where the gadget is?
[20:25:46] <Romaine>	 I think this is it: https://commons.wikimedia.org/wiki/User:Bawolff/campaignAsJson.js
[20:27:58] <marktraceur>	 I'll take a look
[20:29:11] <Romaine>	 :)
[20:32:53] <marktraceur>	 Hm, seems to do *something*
[20:32:56] <marktraceur>	 Just not much useful
[20:33:12] <tgr>	 forgot to mention bug 69311 at the meeting, duh
[20:33:17] <legoktm>	 oh, I think that might have been me
[20:33:45] <marktraceur>	 legoktm: GOSH.
[20:33:48] <marktraceur>	 Stop breaking things
[20:33:50] <legoktm>	 you probably need to load the 'mediawiki.content.json' module as well, since I moved stuff out of EventLogging and into core
[20:34:33] <legoktm>	 Romaine: change mw.loader.load( 'ext.eventLogging.jsonSchema'); to mw.loader.load( ['ext.eventLogging.jsonSchema', 'mediawiki.content.json']);
[20:34:35] <legoktm>	 probably
[20:34:38] * legoktm tests
[20:35:19] <legoktm>	 yeah, fixes it for me
[20:35:35] <Romaine>	 it showed since yesterday only black text on a white background, borders gone, etc
[20:35:46] <legoktm>	 yeah, it needs the additional CSS module
[20:37:51] <marktraceur>	 Does bawolff need to make that changeL
[20:38:00] <marktraceur>	 ?
[20:38:18] <marktraceur>	 "yes"
[20:38:43] <marktraceur>	 Where's bawolff when you need him
[20:38:55] <legoktm>	 or an admin
[20:39:48] * Romaine is admin at Commons
[20:41:06] <marktraceur>	 Then do it!
[20:44:47] <Romaine>	 yes, that works
[20:45:13] <Romaine>	 tkanks!
[20:45:16] <marktraceur>	 Neato burrito
[20:46:21] <marktraceur>	 Our config variable names are expanding
[20:48:26] <marktraceur>	 Romaine: While you're performing admin actions on commons, do you think you could help me disable the HideMediaViewer gadget if our config change works? :D
[20:49:28] <marktraceur>	 Else I can bother -commons for help later
[20:49:52] <legoktm>	 I'm still amazed at how good of a hack https://commons.wikimedia.org/wiki/User:Bawolff/campaignAsJson.js is.
[20:49:56] <Romaine>	 I promissed myself to stay out of MediaViewer things
[20:50:06] <marktraceur>	 Romaine: Fair enough
[20:50:12] <marktraceur>	 It should be nil effect, but OK :)
[20:50:47] <legoktm>	 marktraceur: are you going to switch people's preferences if they opted out of the gadget --> opt in for MV?
[20:51:42] <marktraceur>	 legoktm: Wasn't planning to...
[20:51:47] <marktraceur>	 Probably should/could
[20:51:56] <legoktm>	 should just be some SQL
[20:52:03] <marktraceur>	 True
[20:52:39] <marktraceur>	 I don't think I've ever run SQL on the cluster before
[20:52:43] <marktraceur>	 Not super looking forward to it now
[20:52:56] <legoktm>	 $ sql commonswiki_p
[20:53:04] <legoktm>	 or you can use sql.php
[20:53:59] <marktraceur>	 legoktm: The first command does nothing for me, where is the script?
[20:54:55] <marktraceur>	 I guess maintenance/
[20:55:15] <legoktm>	 mwscript sql.php --wiki=commonswiki
[20:55:32] <legoktm>	 you can use `sql` from terbium at least
[20:55:39] <marktraceur>	 I'm on tin, though
[21:00:18] <marktraceur>	 legoktm: Do you know the prefix for gadget preferences? gadget-HideMediaViewer seems not to work
[21:00:31] <marktraceur>	 Nor gadgets-
[21:00:45] <marktraceur>	 Oh, sorry, gadget- was it.
[21:01:05] <legoktm>	 mysql:wikiadmin@db1040 [commonswiki]> select count(*) from user_properties where up_property ="gadget-HideMediaViewer";
[21:01:07] <legoktm>	 51!
[21:01:33] <marktraceur>	 Yeah
[21:01:57] <legoktm>	 https://meta.wikimedia.org/wiki/Letter_to_Wikimedia_Foundation:_Superprotect_and_Media_Viewer compared to 22!!!
[21:02:46] <marktraceur>	 So update user_properties set up_value = 1 where up_property = "enable-multimediaviewer" and up_user in (select up_user from user_properties where up_property = "gadget-HideMediaViewer" and up_value = 0); ?
[21:02:59] * marktraceur not a DBA
[21:03:40] <legoktm>	 uhhhhh
[21:03:56] <legoktm>	 I think you might need to use upsert?
[21:04:05] <marktraceur>	 Huh, guess so
[21:12:21] <marktraceur>	 Rrg, need a multiple insert
[21:19:21] <marktraceur>	 OK, running the query
[21:19:24] * marktraceur knocks on wood
[21:25:02] <grrrit-wm1>	 (03CR) 10MarkTraceur: [C: 032] Add an option to disable by default for logged-in users [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/154395 (https://bugzilla.wikimedia.org/69363) (owner: 10Gergő Tisza)
[21:25:15] <grrrit-wm1>	 (03PS1) 10MarkTraceur: Add an option to disable by default for logged-in users [extensions/MultimediaViewer] (wmf/1.24wmf17) - 10https://gerrit.wikimedia.org/r/155443 (https://bugzilla.wikimedia.org/69363) 
[21:26:22] <grrrit-wm1>	 (03Merged) 10jenkins-bot: Add an option to disable by default for logged-in users [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/154395 (https://bugzilla.wikimedia.org/69363) (owner: 10Gergő Tisza)
[21:26:30] <grrrit-wm1>	 (03CR) 10jenkins-bot: [V: 04-1] Add an option to disable by default for logged-in users [extensions/MultimediaViewer] (wmf/1.24wmf17) - 10https://gerrit.wikimedia.org/r/155443 (https://bugzilla.wikimedia.org/69363) (owner: 10MarkTraceur)
[21:28:36] <grrrit-wm1>	 (03CR) 10MarkTraceur: [C: 032] Add an option to disable by default for logged-in users [extensions/MultimediaViewer] (wmf/1.24wmf17) - 10https://gerrit.wikimedia.org/r/155443 (https://bugzilla.wikimedia.org/69363) (owner: 10MarkTraceur)
[21:29:14] <grrrit-wm1>	 (03CR) 10jenkins-bot: [V: 04-1] Add an option to disable by default for logged-in users [extensions/MultimediaViewer] (wmf/1.24wmf17) - 10https://gerrit.wikimedia.org/r/155443 (https://bugzilla.wikimedia.org/69363) (owner: 10MarkTraceur)
[21:30:20] <tgr>	 ooh, I like the new look of jenkinsbot
[21:30:48] <marktraceur>	 Yeah it's neat
[21:30:59] <tgr>	 those errors are safe to ignore
[21:31:06] <marktraceur>	 Just force it to merge?
[21:31:07] <marktraceur>	 :/
[21:31:22] <tgr>	 yeah, just fixed them on master a few days ago
[21:31:35] <marktraceur>	 K
[21:31:53] <grrrit-wm1>	 (03CR) 10MarkTraceur: [V: 032] "Sorry, Jenkins, you're wrong." [extensions/MultimediaViewer] (wmf/1.24wmf17) - 10https://gerrit.wikimedia.org/r/155443 (https://bugzilla.wikimedia.org/69363) (owner: 10MarkTraceur)
[21:32:21] <marktraceur>	 Need to do a submodule update still
[21:32:27] <marktraceur>	 *sigh* when deploys aren't planned properly
[22:01:14] <marktraceur>	 fabriceflorin: FYI we're off on Commons now, for logged-in by default. I had Steinsplitter turn off the gadget.
[22:01:18] <marktraceur>	 Should be set now
[22:03:15] <marktraceur>	 Uhhh...
[22:03:25] <marktraceur>	 fabriceflorin_: FYI we're off on Commons now, for logged-in by default. I had Steinsplitter turn off the gadget.
[22:04:21] <fabriceflorin_>	 marktraceur: Splendid, thank you. I am halfway done with my multimedia update for the mailing list, will post it between 4 and 5, and mention that config change. Much appreciated.
[22:06:31] <marktraceur>	 Coolio
[22:06:36] <marktraceur>	 My pleasure
[22:06:46] <marktraceur>	 I'm going to call it a little early because 1. stress 2. things to do
[22:07:05] <marktraceur>	 Need to bring a big pile of crap to a friend's place today, and hopefully move things to a smaller storage space
[23:04:40] <tgr>	 somehow changing the default on Commons confused the one-click opt-out
[23:06:19] <tgr>	 it works most of the time, but sometimes it's strange
[23:06:28] <tgr>	 maybe I am just having cache issues
[23:42:00] <neilk_>	 did something change about how git-review works? It's complaining it can't find https://gerrit.wikimedia.org/tools/hooks/commit-msg
[23:44:51] <tgr>	 neilk_: could be an ssh/scp issue
[23:45:25] <neilk_>	 tgr: no, that file doesn't load in the browser either
[23:45:45] <neilk_>	 I'm confused that it's even checking. Perhaps it looks every time to see if it was modified.