[00:01:48] so, looks like we've got gwtoolset on commons … finally! [00:01:57] thanks again for all your help guys! [00:02:05] bd [00:02:15] Happy to help when we can :) [00:02:21] It's a pretty sweet project [00:03:30] dan-nl: Congrats on getting it running [00:16:26] marktraceur bd808 : Do we know if the GW Toolset was released today? Did everything work out OK? [00:16:41] fabriceflorin: It was [00:16:52] There was discussion on IRC already, all positive [00:17:17] Wonderful, glad to hear it. I bet Dan-l was elated [00:17:26] dan-nl even, yeah [00:18:04] There were some configuration related bugs filed, but I think bawolff actually has fixes pending for those [00:18:11] Any action items on our end? I haven't heard back from David or Dan, will wait to see if they need anything. [00:18:18] Way to go, bawolff ! [00:18:39] bd808: Not all of them, I didn't touch the looking for group instead of right thing [00:19:05] I just finished my Video RfC draft at long last, sent it to the lawyers for review, so I can rejoin the real world now. [00:19:22] Getting started on the Media Viewer Blog post now. [00:21:33] bd808: Thanks again for sheparding the GW Toolset -- you're going to make some museum curators very happy ... as well as some important champions in the Wikimedia movement :) [00:22:30] fabriceflorin: Thanks. I just herded the cats but I guess that helped. [00:23:14] bd808: Every little bit counts. Herding cats can move mountains (if you don't mind my mixing a few metaphors :) [18:40:15] OK [18:40:17] I'm tired of this [18:40:33] https://gerrit.wikimedia.org/r/100706 <-- tgr, aarcos, someone should merge this so we don't have to look at it anymore [18:40:51] Like, if nothing else happens this week [18:40:58] I want this to go through [18:41:03] Because the black bar thing bugs me [18:41:04] So much. [18:42:06] (03CR) 10Aarcos: [C: 032] Extend the post-image div height for the drag affordance [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/100706 (owner: 10MarkTraceur) [18:42:12] My favourite [18:42:52] (03Merged) 10jenkins-bot: Extend the post-image div height for the drag affordance [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/100706 (owner: 10MarkTraceur) [19:05:59] tgr: Video call? [19:06:06] Agh lag. [19:28:25] marktraceur: Is the version of mediaviewer on testwiki a current version that I should report bugs against? [19:29:06] bawolff: Basically yes, but there are loads of patches open too [19:29:36] ok, The username is on a very dark background that I don't think is intentional [19:30:09] bawolff: That was the patch https://gerrit.wikimedia.org/r/100706 that just got merged [19:30:22] oh ok, I won't report that as a bug then :) [19:32:29] *nod* [19:32:36] Could test on beta and make sure it's working now [19:40:57] are there any images on beta commons ;) [19:41:55] Maybe we should enable instant commons on beta, just so all the images are there, but then I guess that would make it less "like" commons [19:43:14] bawolff: There are images on beta, and it has commons as a remote repo [19:43:51] I'm pretty sure it doesn't have commons as a remote repo. Unless I'm looking at a different beta wiki [19:44:25] Hrm [19:44:38] http://commons.wikimedia.beta.wmflabs.org/ right? [19:45:19] Yup [19:46:30] The alpha channel on the spinner could perhaps use with some improvement [19:48:01] * bawolff hasn't actually found an image that has the author listed in that spot, probably the types of images I'm looking at [20:41:53] bawolff: Here are some images on beta commons -- http://commons.wikimedia.beta.wmflabs.org/wiki/Category:GWToolset_Batch_Upload [20:42:09] ~14,000 of them [20:42:13] bd808: Thanks. I actually found some though [20:42:31] bd808: You say 14000, I see 3 :P [20:42:50] 3? [20:42:58] meh, that was just varnish cache [20:43:23] I think there's a bug that we never ever clear varnish cache of category pages on the assumption that logged out users don't care or something [20:43:43] Ah. That wouldn't surprise me [20:43:54] * bd808 is always logged in [20:44:09] bug 33238 [20:45:00] Nobody ever logs in...nobody ever logs out... [20:45:31] There seem to be a never ending supply of varnish cache purge issues [20:46:05] We cache them [20:46:23] *rim shot* [20:47:27] * marktraceur bows [21:05:12] (03PS1) 10MarkTraceur: Fix remote DB repos [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 [21:05:18] So this is a critical bug IMO [21:05:28] It would be super to deploy it today [22:05:23] (03CR) 10Aarcos: [C: 04-1] "Tests would be great, ;-)." (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 (owner: 10MarkTraceur) [22:05:42] Will add them after the rfc meeting, I have a few horses in the race [22:40:40] Hm, so I have a Mumble server set up if we want to try that on Friday [22:40:56] I will do what I can to be present for the standup thing [22:47:16] I guess we could test it [23:18:22] (03PS1) 10MarkTraceur: Fix perf logging [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102588 [23:18:37] That was annoying [23:19:16] aarcos: While I write tests for 102553, maybe we can merge that one-line fix (for which tests aren't terribly useful AFAICT) [23:24:57] marktraceur: Test should always be useful, otherwise they are not good tests, ;-). Tests are you specification. Anyways, approving for merge but waiting for the tests. [23:25:16] (03CR) 10Aarcos: [C: 032] Fix remote DB repos [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 (owner: 10MarkTraceur) [23:26:06] Hm [23:26:13] I was seriously writing tests for that [23:26:22] The one-liner at 102588 was what I meant [23:33:17] (03CR) 10Krinkle: [C: 032] "(debugging Zuul)" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 (owner: 10MarkTraceur) [23:36:04] (03CR) 10Krinkle: [C: 032] Fix remote DB repos [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 (owner: 10MarkTraceur) [23:36:35] (03Merged) 10jenkins-bot: Fix remote DB repos [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102553 (owner: 10MarkTraceur) [23:41:35] Hrm [23:41:45] I dunno what I'm supposed to do about user links to remote DB repos [23:44:28] I guess ask for hypermedia. [23:44:30] *sigh*. [23:52:56] (03PS1) 10Theopolisme: Include license in title attribute of link in FileUsageDialog [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102595 [23:54:26] [03:42:36 PM] <+marktraceur> I dunno what I'm supposed to do about user links to remote DB repos <-- context? [23:54:57] WikiMap::foreignUserLink [23:56:05] (03CR) 10Theopolisme: Include license in title attribute of link in FileUsageDialog (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102595 (owner: 10Theopolisme) [23:58:53] (03CR) 10Aarcos: [C: 032] Fix perf logging [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102588 (owner: 10MarkTraceur) [23:59:05] legoktm: On the frontend [23:59:25] (03Merged) 10jenkins-bot: Fix perf logging [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/102588 (owner: 10MarkTraceur) [23:59:48] I don't have any way of reliably fetching the article path of a foreign db filerepo [23:59:58] "reliably" is important there