[00:00:25] I have a suspicion that Chrome just shows the standard HTTP error messages, not the message actually returned by the server [00:00:25] which link? [00:03:23] tgr: I can't open any of the thumbnail links in new tabs. Clicking the broken file icon just brings up the (blank) file preview modal. [00:03:45] (This during the Upload or Describe steps of UploadWizard.) [00:04:16] you can probably click the URL in the console [00:04:47] ah, yes [00:07:49] tgr: should I censor, say, my centralAuth token or cookie session? [00:09:04] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Thumbnails do not appear at Describe step during batch upload from Flickr - https://phabricator.wikimedia.org/T90599#1064442 (10Ragesoss) Here are details from an UploadStash request: Request URL: https://commons.wikimedia.org/wiki/Special:UploadStash/thumb/1... [00:09:51] tgr: is that what you wanted ^ [00:10:45] no [00:11:00] just go to https://commons.wikimedia.org/wiki/Special:UploadStash/thumb/12yxqfmyh91g.2i8jay.25308.jpg/100px-12yxqfmyh91g.2i8jay.25308.jpg [00:11:08] and tell what you see [00:11:54] I mean in the browser, not the console [00:12:03] ah [00:12:05] Request: GET http://commons.wikimedia.org/wiki/Special:UploadStash/thumb/12yxqi1bbxyg.vxczzv.25308.jpg/100px-12yxqi1bbxyg.vxczzv.25308.jpg, from 10.128.0.110 via cp1055 cp1055 ([10.64.32.107]:3128), Varnish XID 3287488792 [00:12:05] Forwarded for: 67.161.126.166, 10.128.0.110, 10.128.0.110, 10.128.0.110 [00:12:05] Error: 503, Service Unavailable at Wed, 25 Feb 2015 00:11:34 GMT [00:13:07] weird [00:13:16] that sounds like a varnish error [00:13:35] but this is a private URL, varnish should not cache it [00:13:56] anyway, at least it's clear this is not a duplicate, thanks [00:13:59] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Thumbnails do not appear at Describe step during batch upload from Flickr - https://phabricator.wikimedia.org/T90599#1064454 (10Ragesoss) Request: GET http://commons.wikimedia.org/wiki/Special:UploadStash/thumb/12yxqi1bbxyg.vxczzv.25308.jpg/100px-12yxqi1bbxyg.v... [00:14:49] tgr: when I open in a different browser (logged in with a different account) I get this instead: This file (12yxqfmyh91g.2i8jay.25308.jpg) doesn't belong to the current user. [00:15:00] which I guess is the normal expected thing. [00:15:08] yes, that's normal [00:56:12] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1064557 (10Ragesoss) [04:37:27] 6Multimedia, 10Analytics: Video view counts? - https://phabricator.wikimedia.org/T88775#1020132 (10Tbayer) As @Tgr points out (and @ezachte confirmed on the talk page there), this proposal is related: https://www.mediawiki.org/wiki/Requests_for_comment/Media_file_request_counts [07:19:04] (03CR) 10Gergő Tisza: "You seem to have a general idea of what's happening, can you update the bug description? This probably affects other products than UploadW" (032 comments) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [08:41:00] (03CR) 10Gilles: Set the preferences via the API when not testing the preferences page (031 comment) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192607 (https://phabricator.wikimedia.org/T87462) (owner: 10Gilles) [08:42:08] (03CR) 10Gilles: [C: 032] QA: disambiguate step name for Cucumber regex [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192617 (owner: 10Cmcmahon) [08:43:17] (03Merged) 10jenkins-bot: QA: disambiguate step name for Cucumber regex [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192617 (owner: 10Cmcmahon) [08:52:33] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1064970 (10Gilles) [08:58:23] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 10Multimedia-Sprint-2015-02-25: "Error, running transcode job, for job that has already started" notices - https://phabricator.wikimedia.org/T89977#1064981 (10Gilles) [09:01:20] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1064985 (10Gilles) [09:02:13] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-18: Mobile Web and Multimedia viewer should use same hash fragment - https://phabricator.wikimedia.org/T87769#1064986 (10Gilles) a:5Jdlrobson>3None [09:17:49] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25: Browser test failing on check_elements_in_viewer_for_image2 - https://phabricator.wikimedia.org/T90589#1064997 (10Gilles) [09:17:59] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 10Multimedia-Sprint-2015-02-25: "Error, running transcode job, for job that has already started" notices - https://phabricator.wikimedia.org/T89977#1065001 (10Gilles) [09:18:07] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1065002 (10Gilles) [09:18:33] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1065004 (10Gilles) [09:18:57] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Trying to upload causes wikimedia to return a - https://phabricator.wikimedia.org/T90233#1065005 (10Gilles) [09:21:03] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-18, 5Patch-For-Review: Don't use the preferences page to update preferences for each test. Only do that in the test dedicated to preferences - https://phabricator.wikimedia.org/T87462#1065015 (10Aklapper) [09:40:42] (03PS1) 10Gilles: Fix typo [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/192774 (https://phabricator.wikimedia.org/T89814) [09:40:59] (03CR) 10Gilles: [C: 032] Fix typo [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/192774 (https://phabricator.wikimedia.org/T89814) (owner: 10Gilles) [09:41:05] (03Merged) 10jenkins-bot: Fix typo [analytics/multimedia] - 10https://gerrit.wikimedia.org/r/192774 (https://phabricator.wikimedia.org/T89814) (owner: 10Gilles) [09:41:21] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-18, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Performance logging graphs suffer tiny sample size starting feb 12th - https://phabricator.wikimedia.org/T89814#1065066 (10Gilles) [13:47:32] (03PS6) 10Gilles: Browser test for chunked uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [13:47:39] (03CR) 10jenkins-bot: [V: 04-1] Browser test for chunked uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [13:47:44] (03CR) 10Gilles: "I've made the test work in Chrome (tested against local vm and commons beta), and I've reorganized a couple of things according to what I " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [13:47:49] (03PS7) 10Gilles: Browser test for chunked uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [13:49:39] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25: Mobile Web and Multimedia viewer should use same hash fragment - https://phabricator.wikimedia.org/T87769#1065606 (10Gilles) a:3Gilles [14:20:26] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25: Mobile Web and Multimedia viewer should use same hash fragment - https://phabricator.wikimedia.org/T87769#1065662 (10Gilles) [14:21:35] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 7Easy: Fixup whitespace - https://phabricator.wikimedia.org/T38654#1065664 (10TheDJ) I was working on the upstream repo for this a while ago: https://github.com/hartman/mwEmbed/compare/jscleanup I'm considering merging it into our tree instead. [14:55:09] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25: Mobile Web and Multimedia viewer should use same hash fragment - https://phabricator.wikimedia.org/T87769#1065766 (10Gilles) [14:55:49] (03PS1) 10Gilles: Make media viewer use the same prefix as mobile [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) [15:40:14] (03PS1) 10MarkTraceur: [Very WIP] Only use one file input, use Upload objects to represent upload process [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192809 (https://phabricator.wikimedia.org/T89356) [15:40:54] (03CR) 10jenkins-bot: [V: 04-1] [Very WIP] Only use one file input, use Upload objects to represent upload process [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192809 (https://phabricator.wikimedia.org/T89356) (owner: 10MarkTraceur) [15:45:32] (03CR) 10MarkTraceur: [C: 04-1] "Breaks the new copy metadata test, I'" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [15:45:40] (03CR) 10MarkTraceur: "Sigh." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [15:57:31] (03PS9) 10MarkTraceur: Fix browser tests in Chrome [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [15:57:53] (03CR) 10MarkTraceur: [C: 031] "Fixed the copy test. Looks good to me, someone else merge (Gilles can)" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [15:58:01] gi11es: ^^ merge that if you're OK with my PS9 changes [16:05:24] (03PS11) 10MarkTraceur: Respect preferences' default license when picking a license per file [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [16:14:33] (03CR) 10MarkTraceur: [C: 04-1] "https://phabricator.wikimedia.org/P332" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [16:15:27] (03CR) 10MarkTraceur: "Maybe because this is configurable. Try using the text instead? Or maybe I just don't have these licenses on my wiki? Hrm..." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [16:21:04] (03CR) 10MarkTraceur: "I have the NASA one, but maybe the list is thrown off because I have configured fewer licenses in my config?" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [16:30:09] (03CR) 10MarkTraceur: [C: 032] "Never mind, my config just threw things off. Can merge when parent is ready." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [16:30:37] (03CR) 10Jdlrobson: [C: 031] "Can +2 if needed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) (owner: 10Gilles) [16:34:21] (03CR) 10MarkTraceur: [C: 031] "@jdlrobson if you can test it, I'd appreciate the +2." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) (owner: 10Gilles) [16:53:55] (03PS5) 10MarkTraceur: Set the preferences via the API when not testing the preferences page [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192607 (https://phabricator.wikimedia.org/T87462) (owner: 10Gilles) [17:03:26] (03CR) 10MarkTraceur: [C: 032] "Awesome, set to go when parents are merged." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192607 (https://phabricator.wikimedia.org/T87462) (owner: 10Gilles) [17:40:22] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Review JS animations - https://phabricator.wikimedia.org/T90738#1066246 (10Gilles) 3NEW a:3Pginer-WMF [17:41:26] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 7Design: Review JS animations - https://phabricator.wikimedia.org/T90738#1066271 (10Gilles) [17:42:30] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Design: Move file metadata copy handling to describe step, change UX - https://phabricator.wikimedia.org/T90741#1066295 (10MarkTraceur) 3NEW [17:42:43] pginer: [17:42:45] https://phabricator.wikimedia.org/T90741 [17:43:31] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1066305 (10Gilles) [17:43:32] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Design: Move file metadata copy handling to describe step, change UX - https://phabricator.wikimedia.org/T90741#1066306 (10Pginer-WMF) a:3Pginer-WMF [17:43:42] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1066309 (10Gilles) a:3Pginer-WMF [17:43:52] thanks marktraceur [17:45:51] No problem [17:57:56] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: MediaViewer crops bottom of images - https://phabricator.wikimedia.org/T89631#1066402 (10Gilles) [18:00:40] pginer: Probably won't work on it this week, but maybe; no rush on comments [18:47:47] tgr_, gi11es, the PDF bugs, is someone commenting on those already? [18:49:42] what do you mean by commenting? [18:50:54] tgr: csteipp requested Multimedia to comment on them. [18:52:06] recently? [18:52:16] I left some comments a week ago [18:52:52] that culminated in https://phabricator.wikimedia.org/T89765 [18:53:22] He's just keeping on top of us, I guess, they're blocking the next release. [18:54:49] T89765 is scheduled for this sprint [18:57:32] Sweet. [19:04:57] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25: Error in dependency installation when provisioning Sentry - https://phabricator.wikimedia.org/T90701#1066747 (10Tgr) Per Google feeling lucky, this is a missing dependency on `libffi-dev`. (Unfortunately `pip`... [19:05:00] FYI teammates, I'll be taking a little over a week off in April to drive to SF via circuitous route. [19:05:10] Or hopefully anyway. [19:05:18] Gotta pick some things up from storage. [19:29:49] (03CR) 10Gergő Tisza: Make media viewer use the same prefix as mobile (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) (owner: 10Gilles) [19:30:27] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1066944 (10Krenair) [19:31:26] Keegan: https://phabricator.wikimedia.org/T87769 is a small user-facing change that's probably worth announcing [19:33:02] tgr: All righty [19:39:13] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1066980 (10kevinator) [19:40:56] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1066936 (10kevinator) I'm probably going to revisit this for Q4 planning. not right now :-) [19:41:27] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1067000 (10kevinator) a:3kevinator [19:41:44] Keegan: btw, is your team going to use the #notice system? Seems more convenient than bothering you on irc [19:42:20] It's not something we've talked about yet [19:42:29] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1066936 (10kevinator) [19:42:36] I don't mind IRC, but quiddity and I are the only ones that use it regularly [19:43:11] * quiddity denies everything. [19:52:08] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1067050 (10Tgr) > Schema errors are hard to notice - they are logged to the console but tend to get los... [19:55:16] What's the #notice system? [19:56:53] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1067071 (10Nuria) @tgr: But you know that schema validation is real easy to test in vagrant with the d... [19:57:42] 6Multimedia, 10Analytics-EventLogging, 6Analytics-Kanban, 10MediaWiki-extensions-MultimediaViewer: Parse mediaviewer team's requirements for EventLogging {oryx} - https://phabricator.wikimedia.org/T90766#1067081 (10Nuria) @tgr: But you know that schema validation is real easy to test in vagrant with the d... [19:59:02] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: MediaViewer crops bottom of images - https://phabricator.wikimedia.org/T89631#1067087 (10Tgr) I don't think it's a timing issue, it's very deterministic on multiple sites and browsers. Also it's the size of the image that is wrong, so it's not a local CSS d... [20:00:40] (03CR) 10Gilles: Make media viewer use the same prefix as mobile (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) (owner: 10Gilles) [20:01:45] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Move details-related methods on mw.UploadWizard.prototype to the details step - https://phabricator.wikimedia.org/T90771#1067106 (10MarkTraceur) 3NEW [20:02:01] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Move details-related methods on mw.UploadWizard.prototype to the details step - https://phabricator.wikimedia.org/T90771#1067116 (10MarkTraceur) [20:02:07] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Move details-related methods on mw.UploadWizard.prototype to the details step - https://phabricator.wikimedia.org/T90771#1067106 (10MarkTraceur) [20:04:18] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25: Fix log directory permissions in Sentry vagrant role - https://phabricator.wikimedia.org/T90773#1067212 (10Tgr) 3NEW a:3Tgr [20:04:45] 6Multimedia, 6WMF-Legal, 5Patch-For-Review: Set up and test Sentry on Labs for JS error logging - https://phabricator.wikimedia.org/T1345#1067226 (10Tgr) [20:04:46] 6Multimedia, 10MediaWiki-Vagrant, 7Epic: All multimedia extensions/systems should have a MW-Vagrant role - https://phabricator.wikimedia.org/T88072#1067227 (10Tgr) [20:05:02] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Vagrantize Sentry - https://phabricator.wikimedia.org/T84957#1067228 (10Tgr) [20:05:13] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25: Fix log directory permissions in Sentry vagrant role - https://phabricator.wikimedia.org/T90773#1067212 (10Tgr) [20:14:15] 6Multimedia, 6ArchCom, 10MediaWiki-RfCs, 7Epic: RfC: Server-side Javascript error logging - https://phabricator.wikimedia.org/T382#1067275 (10daniel) a:3brion [20:21:30] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1067304 (10Tgr) [20:22:14] (03PS2) 10Gilles: Make media viewer use the same prefix as mobile [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) [20:22:31] (03CR) 10Gilles: Make media viewer use the same prefix as mobile (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/192806 (https://phabricator.wikimedia.org/T87769) (owner: 10Gilles) [20:24:44] (03CR) 10Gilles: [C: 032] Fix browser tests in Chrome [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [20:25:05] (03PS8) 10Gilles: Browser test for chunked uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [20:25:33] (03Merged) 10jenkins-bot: Fix browser tests in Chrome [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191645 (https://phabricator.wikimedia.org/T86770) (owner: 10Gilles) [20:25:38] Yay! [20:25:50] (03Merged) 10jenkins-bot: Respect preferences' default license when picking a license per file [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191191 (https://phabricator.wikimedia.org/T89717) (owner: 10Gilles) [20:25:53] (03Merged) 10jenkins-bot: Set the preferences via the API when not testing the preferences page [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192607 (https://phabricator.wikimedia.org/T87462) (owner: 10Gilles) [20:25:56] All the tests. [20:27:07] I still have a couple of easy-to-review test changesets in the queue [20:27:23] https://gerrit.wikimedia.org/r/#/c/190811/ (media viewer + safari) [20:27:46] https://gerrit.wikimedia.org/r/#/c/192777/ (renaming stuff) [20:28:15] I'll hop on those when I'm done with the detailsSubmit patch I'm writing. [20:28:22] and https://gerrit.wikimedia.org/r/#/c/191246/ which needs tgr to approve my small changes to his code [20:28:42] (03CR) 10Gilles: [C: 031] Browser test for chunked uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/191246 (https://phabricator.wikimedia.org/T89289) (owner: 10Gergő Tisza) [20:30:14] (03PS1) 10MarkTraceur: Move parts of detailsSubmit to details step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192873 (https://phabricator.wikimedia.org/T90771) [20:30:48] (03CR) 10jenkins-bot: [V: 04-1] Move parts of detailsSubmit to details step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192873 (https://phabricator.wikimedia.org/T90771) (owner: 10MarkTraceur) [20:31:04] I'm sure, jenkins. I'm sure. [20:31:26] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 7Design: Review JS animations - https://phabricator.wikimedia.org/T90738#1067353 (10Gilles) [20:34:21] gi11es: I looked at the MMV one yesterday, but had no idea how to test a safari-specific hack [20:34:36] Good point. [20:34:41] I don't think something like virtualIE exists for Safari [20:34:55] use saucelabs [20:35:46] is there a tutorial for that? [20:35:48] export SAUCE_ONDEMAND_USERNAME= [20:35:54] export SAUCE_ONDEMAND_ACCESS_KEY= [20:36:09] ooh, that sounds nice [20:36:20] and then BROWSER, VERSION, PLATFORM values can be found somewhere on the saucelabs site [20:36:27] Wow, that's super simple. [20:36:36] it's great, you can see the screenshots on a timeline, watch the recoding, etc. [20:36:37] I think there's an officewiki page? [20:36:44] chrismcmahon! [20:36:44] a bit slow, that's all [20:36:52] yes there's a page somewhere [20:37:00] listing those 2 shell variable names I just gave you :) [20:37:15] And presumably an account name and access key for WMF. [20:37:17] Or something. [20:37:21] http://www.mediawiki.org/wiki/Quality_Assurance/Browser_testing/Running_tests [20:37:29] you can register your own [20:37:38] it's free for open source work [20:37:46] the caveat is that everything you record is public [20:37:50] Hrm. [20:39:22] https://www.mediawiki.org/wiki/Quality_Assurance/Browser_testing/Running_tests#Running_browser_tests_at_SauceLabs to be more precise [20:39:41] it is totally worth the 10 minutes it will take you to set the whole thing up [20:43:14] Noted. tgr are you testing this one? [20:43:22] so for the latest safari i think the correct values are "Safari" for BROWSER, "OS X 10.10" for PLATFORM and "8" for VERSION [20:43:38] I'll do it later today [20:48:04] OK [20:48:40] (03CR) 10MarkTraceur: [C: 032] Change internal name of existing features [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192777 (https://phabricator.wikimedia.org/T90696) (owner: 10Gilles) [20:49:22] (03Merged) 10jenkins-bot: Change internal name of existing features [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/192777 (https://phabricator.wikimedia.org/T90696) (owner: 10Gilles) [20:54:35] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Default license in preferences not respected for multi-file uploads going through the per-file license picking step - https://phabricator.wikimedia.org/T89717#1067412 (10Gilles) [20:54:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Browser tests no longer work on Chrome - https://phabricator.wikimedia.org/T86770#1067413 (10Gilles) [20:54:37] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Don't use the preferences page to update preferences for each test. Only do that in the test dedicated to preferences - https://phabricator.wikimedia.org/T87462#1067411 (10Gilles) [20:54:39] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: "Clicking the image closes the download menu" browser test failing on IE - https://phabricator.wikimedia.org/T89087#1067421 (10Gilles) [20:54:40] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Mobile Web and Multimedia viewer should use same hash fragment - https://phabricator.wikimedia.org/T87769#1067416 (10Gilles) [20:54:41] 6Multimedia, 10Analytics, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Set up varnish 204 beacon endpoint for virtual media views and use it in Media Viewer - https://phabricator.wikimedia.org/T89088#1067420 (10Gilles) [20:54:43] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25, 7JavaScript, 5Patch-For-Review: Add error id generation to mw.errorLogging - https://phabricator.wikimedia.org/T89384#1067422 (10Gilles) [20:54:45] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 10Quality-Assurance, 5Patch-For-Review: UploadWizard browser test for chunked upload - https://phabricator.wikimedia.org/T89289#1067417 (10Gilles) [20:54:46] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Move the interface for adding new files to a new UI class and refactor things around it - https://phabricator.wikimedia.org/T89855#1067419 (10Gilles) [20:54:48] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Browser test for flickr import - https://phabricator.wikimedia.org/T86120#1067418 (10Gilles) [20:54:49] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Performance logging graphs suffer tiny sample size starting feb 12th - https://phabricator.wikimedia.org/T89814#1067424 (10Gilles) [20:54:51] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Catch uncaught exceptions with Sentry - https://phabricator.wikimedia.org/T88874#1067423 (10Gilles) [20:58:09] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Improve reliability of cucumber tests - https://phabricator.wikimedia.org/T87831#1067435 (10Gilles) 5Open>3Resolved I've made new tickets following that theme, with more specific issues to fix. [21:15:10] gi11es: marktraceur yeah, SauceLabs is pretty great, and we're all set up to use it. If you need anything you don't have there, let me or Zeljko know, or email the QA mail list. [21:44:27] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 7Regression: Video thumbnails not fitting into packed-hover gallery mode - https://phabricator.wikimedia.org/T77988#1067716 (10Bawolff) See also related 88d8dd2bf78af39bcd8c1436bc84426cfc1c2dd4 [21:59:35] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25: Fix log directory permissions in Sentry vagrant role - https://phabricator.wikimedia.org/T90773#1067789 (10dduvall) I think this may only be a problem when using vboxsf instead of nfs. @tgr, do you have `nfs_sh... [22:56:50] 6Multimedia, 10MediaWiki-Vagrant, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-02-25: Fix log directory permissions in Sentry vagrant role - https://phabricator.wikimedia.org/T90773#1068014 (10Tgr) >>! In T90773#1067789, @dduvall wrote: > @tgr, do you have `nfs_shares` disabled by chance? On so... [23:32:37] 6Multimedia, 10MediaWiki-Export-or-Import, 10MediaWiki-File-management: Add image data option to Special:Export - https://phabricator.wikimedia.org/T15827#1068173 (10Tgr)