[08:00:59] 6Multimedia, 10MediaWiki-extensions-PdfHandler: Show coordinates in PDF file - https://phabricator.wikimedia.org/T91482#1085535 (10Thgoiter) 3NEW [08:14:54] 6Multimedia, 10MediaWiki-File-management: Thumbs stretch small images (with no width specified) - https://phabricator.wikimedia.org/T3220#1085613 (10epriestley) [08:15:44] 6Multimedia, 10MediaWiki-Uploading: Files with special characters to upload : Chineses (some bugs) - https://phabricator.wikimedia.org/T4165#1085660 (10epriestley) 5Invalid>3Resolved [08:20:00] 6Multimedia, 10MediaWiki-Uploading: [PATCH] Existing images are overwritten without notice - https://phabricator.wikimedia.org/T2548#1086063 (10epriestley) [08:20:10] 6Multimedia, 10MediaWiki-Uploading: File-Uploads from MacOS X has Problems with UTF-8 - https://phabricator.wikimedia.org/T2215#1086074 (10epriestley) [08:20:36] 6Multimedia, 10MediaWiki-File-management: Commons images cannot be thumbnail - https://phabricator.wikimedia.org/T2797#1086137 (10epriestley) 5Invalid>3Resolved [08:20:38] 6Multimedia, 10MediaWiki-File-management: Image upload replacements should create watchlist entry - https://phabricator.wikimedia.org/T2778#1086146 (10epriestley) [08:23:27] 6Multimedia, 10MediaWiki-Uploading, 7I18n: Can't upload file with non-ASCII name (eg cyrillic) on Windows host - https://phabricator.wikimedia.org/T3780#1086785 (10epriestley) [08:45:44] 6Multimedia, 10MediaWiki-Uploading: Pull copyright metadata from files on upload - https://phabricator.wikimedia.org/T2657#1087320 (10Qgil) 5Resolved>3Open Accidental clash. Known issue. Reverting status. [09:17:19] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1087525 (10Gilles) Yeah, we'll schedule it for our next sprint, which starts later today. [09:18:01] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1087526 (10Gilles) a:5Pginer-WMF>3None [09:24:19] 6Multimedia, 10MediaWiki-File-management: Thumbs stretch small images (with no width specified) - https://phabricator.wikimedia.org/T3220#1087578 (10Gilles) @epriestley is something misconfigured on our phabricator instance, that made your commit partially show up here? [09:55:35] Why are the Multimedia sprints at https://phabricator.wikimedia.org/project/sprint/profile/1107/ and https://phabricator.wikimedia.org/project/sprint/profile/1073/ only joinable by members of Multimedia? [09:56:10] That means that noone has a change to subscribe and follow the work. Plus you do not use the correct color and icon which violates the project guidelines. [09:56:12] Please fix. [10:03:43] the boards are publicly visible, isn't that enough for people to follow what we do? [10:04:07] as for the color and icon, why should I have to memorise that stuff instead of the defaults being correct? [10:07:40] after all, I'm ticking that this is a sprint, so the software should be able to apply appropriate icon and colour automatically [10:21:40] 6Multimedia, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1087741 (10Gilles) Do you know which DB the beta EL events are recorded to? [10:45:35] (03CR) 10Gilles: [C: 04-1] Move makeTransitioner to the step objects (031 comment) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [10:49:03] 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#1087794 (10Gilles) a:3Gilles [11:22:35] 6Multimedia, 10MediaWiki-Uploading: Files with special characters to upload : Chineses (some bugs) - https://phabricator.wikimedia.org/T4165#1087845 (10Aklapper) 5Resolved>3Invalid a:3Aklapper [11:23:58] 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#1087854 (10Gilles) More recent examples: ``` 2015-03-02 10:23:55 tmh1001 commonswiki: webVideoTranscode F... [11:30:00] 6Multimedia, 10MediaWiki-File-management: convert images to interlaced / progressive format - https://phabricator.wikimedia.org/T2269#1087903 (10Aklapper) 5Resolved>3declined a:3Aklapper [11:36:25] 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#1087934 (10Gilles) Looking up the value of transcode_time_startwork for those in the DB: СВ-ДНР-360._Обст... [11:39:16] 6Multimedia, 10MediaWiki-File-management: Option to load uploads from another domain - https://phabricator.wikimedia.org/T2638#1087955 (10Aklapper) 5Resolved>3Invalid a:3Aklapper [11:39:41] 6Multimedia, 10MediaWiki-File-management: Commons images cannot be thumbnail - https://phabricator.wikimedia.org/T2797#1087962 (10Aklapper) 5Resolved>3Invalid a:3Aklapper [11:41:34] 6Multimedia, 10MediaWiki-File-management: Scaled image is larger than original for palletted PNG (in terms of size) - https://phabricator.wikimedia.org/T3218#1087986 (10Aklapper) 5Resolved>3declined a:3Aklapper [11:51:43] 6Multimedia, 10MediaWiki-extensions-GWToolset: GWT should url-encode URLs if necessary - https://phabricator.wikimedia.org/T91499#1088077 (10Kelson) 3NEW [11:58:04] 6Multimedia, 10MediaWiki-extensions-GWToolset: GWT should url-encode URLs if necessary - https://phabricator.wikimedia.org/T91499#1088099 (10Kelson) [13:54:28] 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#1088253 (10Gilles) Looking at this one specifically: ``` 2015-03-02 10:23:55 tmh1001 commonswiki: webVid... [14:09:27] 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#1088285 (10Gilles) I've read the history of the file, it seems like it's the remaining race conditions tha... [14:12:18] 6Multimedia, 10MediaWiki-API, 7Technical-Debt: ApiQueryImageInfo is crufty, needs rewrite - https://phabricator.wikimedia.org/T89971#1088291 (10Anomie) [14:40:10] (03PS1) 10Gilles: Improve transcode job insertion deduping [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) [14:40:14] (03CR) 10jenkins-bot: [V: 04-1] Improve transcode job insertion deduping [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [14:41:21] (03PS2) 10Gilles: Improve transcode job insertion deduping [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) [14:57:01] Aww. [14:57:06] It worked so well [14:57:41] gi11es: Oh man, you're talking about a whole other can o' beans [14:57:50] That's...geez, OK, commenting inline [15:02:29] (03CR) 10MarkTraceur: Move makeTransitioner to the step objects (031 comment) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [15:04:29] (03CR) 10Gilles: Move makeTransitioner to the step objects (031 comment) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194168 (https://phabricator.wikimedia.org/T91258) (owner: 10MarkTraceur) [15:04:59] (03CR) 10Gilles: "recheck" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194096 (https://phabricator.wikimedia.org/T86120) (owner: 10Gilles) [15:05:42] \o/ [15:05:48] Back to zero [15:05:50] Like a boss [15:06:11] I'll +2 it once the remaining browser tests have made it through and I can run them against this [15:06:14] (but yeah, that can be the next stop, promisifying the details submitter) [15:06:18] Oh, sure. [15:06:33] Let's see if I can merge some o' them. [15:06:56] jenkins might be a pain with the unrelated qunit failure caused by something in core [15:07:06] Oh dear. [15:07:08] I'm not sure if that's been fixed yet, the task for it is still open [15:07:10] OK [15:08:03] https://phabricator.wikimedia.org/T91399 [15:08:12] marked as resolved [15:08:56] yep, it's gone [15:11:42] Sweet. [15:12:08] I'm running the NASA patch on my laptop, I'm going to let it finish then run it on beta, then merge. [15:12:24] gi11es: You said tgr|away needed to comment on the Flickr tests, right? [15:13:03] nope, why? [15:13:06] ...oh [15:13:17] I kept closing the tab after reading the first line, my bad [15:13:21] I'll try to review those next [15:43:36] 6Multimedia, 10MediaWiki-Watchlist, 10MediaWiki-extensions-GlobalUsage, 10Possible-Tech-Projects, 7Epic: Show edits made on WikimediaCommons on the Watchlist - https://phabricator.wikimedia.org/T91192#1088669 (10Bawolff) [15:44:07] * bawolff does not understand phab formatting commands [15:49:12] T100000 support wikitext in phabricator [15:51:17] Lets hope it gets fixed faster than bug 1 of old [15:51:49] (03CR) 10MarkTraceur: [C: 032] Fix selectors for radio buttons [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/193144 (https://phabricator.wikimedia.org/T90890) (owner: 10Gilles) [15:54:12] (03Merged) 10jenkins-bot: Fix selectors for radio buttons [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/193144 (https://phabricator.wikimedia.org/T90890) (owner: 10Gilles) [15:55:19] I dislike all the media plans on https://phabricator.wikimedia.org/T89084 . thumbnailer seems like a poor choice for an independent "service", and get dominent colour is not something that should really be in the thumbnailing pipeline, but instead part of search or maybe category sort key [15:57:00] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Failure for NASA option of browser test on integration server - https://phabricator.wikimedia.org/T90890#1088775 (10MarkTraceur) 5Open>3Resolved [16:10:43] 6Multimedia, 10MediaWiki-Watchlist, 10MediaWiki-extensions-GlobalUsage, 10Possible-Tech-Projects, 7Epic: Show edits made on WikimediaCommons on the Watchlist - https://phabricator.wikimedia.org/T91192#1088828 (10Steinsplitter) >>! In T91192#1088567, @NiharikaKohli wrote: > @Steinsplitter, @Rillke is the... [16:12:58] (03PS4) 10MarkTraceur: Browser test for flickr uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194096 (https://phabricator.wikimedia.org/T86120) (owner: 10Gilles) [16:15:00] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Make uploads and details use promises for their transitions, and change transition methods to use them - https://phabricator.wikimedia.org/T91514#1088851 (10MarkTraceur) 3NEW a:3MarkTraceur [16:15:08] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Make uploads and details use promises for their transitions, and change transition methods to use them - https://phabricator.wikimedia.org/T91514#1088860 (10MarkTraceur) [16:21:24] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088897 (10Gilles) p:5Triage>3Low I've found these exceptions: ``` 2015-02-24 23:49:49 mw1182 commonswiki: [55234211] /wiki/Special:U... [16:24:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088923 (10Gilles) a:3Gilles [16:25:18] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088932 (10Gilles) a:3Gilles [16:25:58] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088933 (10Ragesoss) @Gilles I did try pasting those urls into another browser (logged in with a different account, or... [16:26:29] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25: Trying to upload causes wikimedia to return a - https://phabricator.wikimedia.org/T90233#1088934 (10Gilles) a:3Gilles I can't find anything in exception.log, api.log nor upload.log @Lklundin does it... [16:29:59] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088946 (10Gilles) Ah ok, so those exceptions were when you were troubleshooting this, but presumably not the issue it... [16:42:00] bd808: So if I understand correctly, the plan with thumbnailer as a service is to maintain two separate code paths for thumbnailing images (service version vs in core mw version)? [16:43:18] I think that's how it would start. In an ideal world that split would narrow or collapse by developing library interfaces that were compatible between the in wiki version and the stand alone verision [16:44:13] That would ideally allow both implementations to share the same underlying libraries for actually performing the image scaling tasks [16:44:13] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1088989 (10Ragesoss) BTW, I just tested this again with a different set of images in the range of 7 to 20MB, and I sti... [16:44:58] while allowing the service version to have a much lighter and more focused front-end implementation to increase performance and security [16:45:18] Ops really wants better process isolation for image scaling. [16:45:53] So that strange bugs in the image manipulation libraries can be well isolated from other processes [16:47:07] 6Multimedia, 10MediaWiki-Watchlist, 10MediaWiki-extensions-GlobalUsage, 10Possible-Tech-Projects, 7Epic: Show edits made on WikimediaCommons on the Watchlist - https://phabricator.wikimedia.org/T91192#1089003 (10Bawolff) I haven't decided if I will participate as a mentor in gsoc this year. If I was to,... [16:48:30] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1089011 (10Gilles) Given the lack of exception for the actual requests, it's unlikely to be an issue with the request... [16:48:38] bd808: I'm really not thrilled about the idea of having to maintain two separate code paths to do the same thing, but if it can be made librarizationized so that both can share the important bits, then I'm less concerned about that [16:49:09] "maintain two separate code paths to do the same thing" -- MW, Parsoid [16:49:14] it's the future :/ [16:49:29] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-02-25, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1089012 (10Gilles) It makes sense that there would be a correlation to file size, because thumbnails for those take lo... [16:49:31] yeah, and I didn't like the MW vs parsoid thing either :P [16:50:17] if written in php, there wouldn't need to be two versions [16:50:39] ideally you'd want it to be able to run inline, the oldschool way, or invoked through a service [16:50:46] *nod* [16:51:14] I think the MVP would have to forego that but that's where it should end up [16:52:37] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04: Make uploads and details use promises for their transitions, and change transition methods to use them - https://phabricator.wikimedia.org/T91514#1089021 (10Gilles) [16:52:38] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: "Error, running transcode job, for job that has already started" notices - https://phabricator.wikimedia.org/T89977#1089022 (10Gilles) [16:52:39] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1089019 (10Gilles) [16:52:40] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1089024 (10Gilles) [16:52:41] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04: Trying to upload causes wikimedia to return a - https://phabricator.wikimedia.org/T90233#1089020 (10Gilles) [16:52:43] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1089025 (10Gilles) [16:52:46] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 7Design: Review JS animations - https://phabricator.wikimedia.org/T90738#1089032 (10Gilles) [16:52:47] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1089035 (10Gilles) [16:52:48] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04: Move the interface for adding new files to a new UI class and refactor things around it - https://phabricator.wikimedia.org/T89855#1089034 (10Gilles) [16:52:50] 6Multimedia, 10Analytics, 10MediaWiki-extensions-MultimediaViewer, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Set up varnish 204 beacon endpoint for virtual media views and use it in Media Viewer - https://phabricator.wikimedia.org/T89088#1089033 (10Gilles) [16:53:42] (03CR) 10MarkTraceur: [C: 032] "Works for me, code looks OK, good to go!" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194096 (https://phabricator.wikimedia.org/T86120) (owner: 10Gilles) [16:54:16] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Browser test for flickr import - https://phabricator.wikimedia.org/T86120#1089048 (10MarkTraceur) 5Open>3Resolved [16:54:25] (03Merged) 10jenkins-bot: Browser test for flickr uploads [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/194096 (https://phabricator.wikimedia.org/T86120) (owner: 10Gilles) [17:17:37] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Move newUpload to the Upload step controller - https://phabricator.wikimedia.org/T86679#1089086 (10MarkTraceur) [17:18:41] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Make the transports mostly independent, and add tests for them - https://phabricator.wikimedia.org/T85629#1089088 (10MarkTraceur) [17:19:03] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Make the step UI and controller classes more self-contained - https://phabricator.wikimedia.org/T78057#1089090 (10MarkTraceur) [17:42:27] 6Multimedia, 10MediaWiki-File-management: Allow image sizing by height - https://phabricator.wikimedia.org/T3944#1089259 (10Aklapper) 5Resolved>3declined a:3Aklapper [17:43:36] 6Multimedia, 10MediaWiki-File-management: Inline images not displaying in Firefox 1.0 - https://phabricator.wikimedia.org/T3252#1089272 (10Aklapper) 5Resolved>3declined a:3Aklapper [17:45:32] 6Multimedia, 10MediaWiki-File-management: Patch for wfgettype() - https://phabricator.wikimedia.org/T2956#1089308 (10Aklapper) 5Resolved>3declined a:3Aklapper [17:58:06] marktraceur: the RelEng blocker for Sentry is https://phabricator.wikimedia.org/T84956 [17:58:48] Why are the Multimedia sprints at https://phabricator.wikimedia.org/project/sprint/profile/1107/ and https://phabricator.wikimedia.org/project/sprint/profile/1073/ only joinable by members of Multimedia? [17:58:55] That means that noone has a change to subscribe and follow the work. Plus you do not use the correct color and icon which violates the project guidelines... [17:59:01] Please fix. Thank you. [17:59:01] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Tracking: Improve Sentry coverage - https://phabricator.wikimedia.org/T88399#1089372 (10Tgr) [17:59:02] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Track module initialization errors in ResourceLoader - https://phabricator.wikimedia.org/T85263#1089371 (10Tgr) 5Open>3Resolved [17:59:03] 6Multimedia, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Implement module wrapping for Sentry - https://phabricator.wikimedia.org/T78809#1089373 (10Tgr) [18:01:34] andre__: Multimedia is joinable by anyone, OTOH [18:03:01] I talked about the Sprints. [18:03:26] Please fix them. [18:03:36] your attitude is unhelpful [18:03:40] yes, but since anyone can become a member of Multimedia, it's not really a limitation [18:03:52] exactly, it's just an extra step [18:04:11] I think it was an attempt to limit mailspam although I don't remember the details anymore [18:04:12] gi11es, what's my attitude? [18:04:16] which is a good thing to avoid people randomly dragging things in our board, which is hard to undo [18:04:26] telling us do X without discussing the why [18:04:30] My attitude is that restriction should not be applied if restriction does not make sense. [18:04:32] and I actually replied to you here, earlier [18:04:43] you just came back and re-pasted the same text [18:04:54] yeah. because noone answered. That's how IRC works. [18:05:09] I answered right after you wrote [18:05:42] hah. true, Thank you. Now that I scroll up I see it, 8min after my question. Pinging by name is welcome so I don't miss that. :) [18:06:02] So what's the reason for restricting membership? [18:06:14] there's no good undo for board actions [18:06:33] nobody besides people who are actively interacting with us has any business shuffling our sprint board around [18:06:57] gi11es: You do understand that restricting membership has NOTHING to do with blocking people from moving stuff on workboards? [18:07:09] I cannot join your project. I can move stuff on your workboard. [18:07:14] no, this is all very obscure [18:07:23] what's the point of joining a sprint? [18:07:33] What's the point of not allowing to join a project? [18:07:40] Following progress is the point. [18:07:47] Being able to subscribe and receive notifications. [18:08:05] Plus I'm afraid that access restrictions do not do what you think they do. :) [18:08:10] I doubt anyone will subscribe to a specific sprint [18:08:31] Is your doubt that interest exists a valid reason to restrict access? [18:08:42] https://www.mediawiki.org/wiki/Phabricator/Creating_and_renaming_projects#Policy [18:08:59] There is no reason to protect Phabricator project pages by default. [18:09:06] well if the restriction doesn't limit workboard write access, there is no point in having it [18:09:12] YES. Thanks. ;) [18:09:45] although it's a bit scary that workboards are world-editable with no audit trail at all [18:09:56] so we *could* technically restrict access (by membership in a group) to moving tasks on workboards and assigning. But that is a global setting and currently not enabled. [18:10:05] We can enable it once we see vandalism. [18:10:20] and if you have faced vandalism I am very interested to hear about it so we can act. [18:10:41] the concern here is exactly that we would not see it [18:11:04] Yeah, but we also restrict wiki pages once we have seen vandalism, not beforehand. :) [18:11:05] there is no notification whatsoever for a workboard change [18:11:17] wikis have an audit trail [18:11:29] it's pretty much the defining feature of a wiki [18:11:51] liberal access control with easy revert [18:11:56] tgr: There IS notification if do not set "A task is moved between columns on a workboard." on https://phabricator.wikimedia.org/settings/panel/emailpreferences/ to "None" [18:12:05] *you [18:12:57] Anyway, I ask you to please follow the project guidelines and not restrict access to projects (and fix the color and icon). Plus restricting to join does not do what you think it does anyway, it seems. :) [18:13:09] Thank you for following the project guidelines. [18:13:17] ah, I missed that [18:13:50] would be nice to have notifs on the ticket itself, but that works too I guess [18:13:55] please make the software set the icon and color for us [18:14:08] I have no interest doing these extra manual steps every week [18:14:20] sprint are already clunky enough to create as it is [18:14:23] gi11es, makes sense. want to file a task against the Phabricator Sprint extension? [18:14:30] True that :-/ [18:14:43] Regarding anti-vandalism: https://phabricator.wikimedia.org/T84 [18:14:47] sure, I'll file that [18:14:47] Restricting modification of tasks when they enter sprints: https://phabricator.wikimedia.org/T819 [18:14:57] I don't even know which color and icon are the right ones for sprints anyway [18:15:02] does Sprint have a proper API? [18:15:07] Alright. Thanks for your patience with me. And I am sorry that I missed your earlier answer. :-/ [18:15:09] not that I know of [18:15:13] Uh... don't really think so [18:15:21] writing a bot to create sprints might be useful for all teams [18:18:33] (03CR) 10Aaron Schulz: [C: 04-1] Improve transcode job insertion deduping (032 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [18:22:19] (03CR) 10Gilles: Improve transcode job insertion deduping (032 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [18:25:55] (03CR) 10Aaron Schulz: Improve transcode job insertion deduping (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/194333 (https://phabricator.wikimedia.org/T89977) (owner: 10Gilles) [18:26:25] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1089427 (10Tgr) >>! In T507#1087741, @Gilles wrote: > Do you know which DB the beta EL events are recorded to? See https://wikitech.wikimedia.org/wiki/Eve... [18:26:52] (03PS3) 10Gergő Tisza: Fix typo in CorsLogger [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/194008 (https://phabricator.wikimedia.org/T507) [18:27:22] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1089429 (10Tgr) Which I forgot to link to the bug, duh. [18:35:18] 6Multimedia, 7Blocked-on-RelEng, 7Puppet: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1089461 (10MarkTraceur) [18:38:22] marktraceur: oops, got disctracted by the Phab stuff [18:38:51] the JS core blocker for Sentry is https://phabricator.wikimedia.org/T85262 although the task name/description is really outdated [18:39:27] I'll update it, the associated commit is https://gerrit.wikimedia.org/r/#/c/188328/ [18:44:10] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#1089479 (10Tgr) [18:44:21] gi11es: Can you link me to the burndown chart you referenced earlier? [18:44:41] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#942722 (10Tgr) Rewrote what this task is about based on the discussion in the comments. [18:44:52] this? https://phabricator.wikimedia.org/project/sprint/burn/1073/ [18:46:36] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-MediaWiki-Core, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#942722 (10MarkTraceur) [18:57:47] 6Multimedia, 6Scrum-of-Scrums, 7Blocked-on-RelEng, 7Puppet: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1089526 (10dduvall) [19:16:00] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-03-04: Update server-side JS error-logging RfC - https://phabricator.wikimedia.org/T91357#1089564 (10Tgr) [19:56:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1089740 (10Tgr) [19:56:37] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-UploadWizard, 10Wikimedia-Media-storage, 7Tracking: Uploadstash errors (tracking) - https://phabricator.wikimedia.org/T85568#1089739 (10Tgr) [20:16:04] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04: Trying to upload causes wikimedia to return a - https://phabricator.wikimedia.org/T90233#1089884 (10Lklundin) I reproduced it reliably, multiple times, both from scratch and by clicking the 'Retry the f... [20:17:13] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 6Security, 7Design: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1089892 (10MarkTraceur) p:5Triage>3High a:3MarkTraceur [20:19:21] gi11es, tgr, if one of you has thoughts about what form the PDF warning should take - like a gadget, extension, or just a core patch - speak now or forever hold your piece [20:23:11] I guess it's "peace", that never sounded right to me. [20:33:28] fwiw my vote is to make it a configurable core function. [20:59:53] marktraceur: definitely a core patch [21:00:45] I was thinking of checking for a message called 'fileformat-warning-' and show the value of the message in the popup if that exists [21:01:08] that way sysops can extend/cancel if they wish to [21:07:53] tgr: I don't see any reason for that to be necessary...it's a security issue that site admins should decide on [21:09:52] true [21:10:29] at any rate, it should be configurable what message is shown for what mime types [21:10:42] site config is indeed a saner place for it [21:11:01] especially since extensions handling new formats might need to add to it [21:12:14] tgr: True. [21:18:33] 6Multimedia, 6Scrum-of-Scrums, 7Blocked-on-RelEng, 7Puppet: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1090183 (10Tgr) See also the //Guidance on creating Debian packages for puppet// ops thread from January. The consensus there seemed to be that apt packages are an over... [21:25:11] 6Multimedia, 10MediaWiki-extensions-UploadWizard: UploadWizard preview file is not resized - https://phabricator.wikimedia.org/T89908#1090223 (10Tgr) [21:30:57] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 10Wikimedia-Media-storage: Global usage shows file usage on deleted pages - https://phabricator.wikimedia.org/T65594#1090245 (10Tgr) [21:31:39] 6Multimedia, 10Multimedia-Sprint-2015-03-04, 10Wikimedia-Media-storage: Global usage shows file usage on deleted pages - https://phabricator.wikimedia.org/T65594#1090250 (10Tgr) a:3Tgr Will run this in the next few days. [21:44:43] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-UploadWizard, 10Wikimedia-Media-storage, 7Tracking: Uploadstash errors (tracking) - https://phabricator.wikimedia.org/T85568#1090268 (10Tgr) [21:44:44] 6Multimedia, 10MediaWiki-extensions-UploadWizard: UploadWizard preview file is not resized - https://phabricator.wikimedia.org/T89908#1090269 (10Tgr) [21:45:25] 6Multimedia, 10MediaWiki-extensions-UploadWizard: UploadWizard preview file is not resized - https://phabricator.wikimedia.org/T89908#1048454 (10Tgr) [21:48:17] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1090288 (10Tgr) >>! In T90599#1089011, @Gilles wrote: > The duration of your original requests suggests some sort of t... [21:48:57] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Thumbnails do not appear during batch upload - https://phabricator.wikimedia.org/T90599#1090297 (10Tgr) [21:48:58] 6Multimedia, 10MediaWiki-extensions-UploadWizard: Investigate thumbnailing failures in Chrome, via Flickr import - https://phabricator.wikimedia.org/T78713#1090296 (10Tgr) [21:50:13] 6Multimedia, 10MediaWiki-File-management, 7Testme, 7Upstream: SVG thumbnail is cropped when displayed at natural size - https://phabricator.wikimedia.org/T11832#1090302 (10TheDJ) Tested with: ImageMagick 6.9.0-0 Q16 x86_64 2015-01-12 and still a problem for me. Full Imagemagick config: P356 rsvg-convert -... [21:50:21] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10Multimedia-Sprint-2015-03-04, 5Patch-For-Review: Displaying the stashed file as a thumbnail in UploadWizard fails with an 503 - https://phabricator.wikimedia.org/T90599#1090303 (10Tgr) [22:18:34] tgr: I heard that you need some kind of mw-core help with T85262? [22:19:53] bd808: it's a core patch, beyond that I have no clue what team it should belong to [22:20:47] Timo is doing reviews on it from time to time, I just wanted the dependency tracked somewhere since this is now the main blocker for Sentry [22:21:15] Yeah, sorry for blocking on that. [22:21:59] It's a major change that I want to make sure I understand properly first. At this point the overal intent makes sense, but I've nog been able to get it to work yet. And I don't want it get stale later or be in the way when doing other refactoring in RL, lots of which is in the pipeline. [22:22:10] * bd808 will leave it Krinkle's capable hands [22:22:33] Krinkle: no worries, it's my fault for not coordinating before setting our quarterly goals [22:22:35] tgr has been great at explaining the Sentry stack earlier. I need to get to that. [22:22:46] I just wamt tp figure out if they are still realistic [22:22:51] And yeah, as such I'm not really resourced to do this at the moment [22:23:03] Got other obligations. [22:23:20] But you've got the bandwidth to make the changes. I'll try and be as lightweight as I can. [22:23:23] And let you do the work :) [22:24:12] Krinkle: I sent you a mail recently, not sure if you have seen that [22:24:35] tgr: The immediate thing I'm looking for now is 1) what will subscribe to the event emitted from mw.errorLogging, 2) somehow see raven using those underscore properties. [22:25:00] if that's still being worked on, no worries, but then there's no need to land this one yet. [22:25:22] that part is merged already [22:25:34] Cool [22:25:36] tgr: Where? [22:25:47] https://github.com/wikimedia/mediawiki-extensions-Sentry/blob/master/resources/init.js [22:26:12] not sure about the underscores, I just copied that from Raven's own wrap module [22:26:22] in case it uses them somehow internally [22:26:33] s/module/function [22:26:58] although I think it's only used to make Raven.wrap idempotent [22:30:54] Krinkle: also, sorry about not having proper documentation; I plan to fix that in the current sprint, you can subscribe to https://phabricator.wikimedia.org/T91357 if you want a notification [22:32:12] tgr: So in case of extensions to EventLogging, I don't usually set up a local el server (though bin/eventlogging-devserver is quite nice), instead I just add a trackSubscribe in common.js and see that it receives things properly in the expected format and not break code. That's for regular events and stuff. [22:32:48] I'm basically trying to get in a state where I have the Sentry extension report to either a basic cli, or js event, or even a local sentry install or in labs, receive events and see it work. [22:33:37] if you use vagrant, all that should come out of the box [22:34:23] the Sentry upsteam moves rather fast and the version installed on vagrant is not fixed, so it breaks sometimes [22:34:40] last week I fixed all breakages so it should be good for a while now [22:35:44] if you don't use vagrant and don't want to bother with the local Sentry install, you can just create a new group on sentry-beta on labs and set that as the endpoint for the Sentry mediawiki extension [22:36:21] although I haven't tested that one yet [22:36:23] OK. I'll go with the vagrant one. [22:36:58] I'm only avoiding that slightly to keep things managable and to verify I'm testing the thing and not vagrant or an interdependent part it provides to make it work. [22:37:02] 6Multimedia, 10MediaWiki-extensions-Sentry, 10Multimedia-Sprint-2015-03-04: Test Sentry extension reporting to the Sentry install on Labs - https://phabricator.wikimedia.org/T91579#1090460 (10Tgr) 3NEW [22:38:16] as for the Sentry extension reporting to something other than Sentry, the extension itself is not much more than reporting code over mw.trackSubscribe so another level of JS indirection does not make sense [22:38:24] and Raven is not very flexible [22:39:09] you can use the network console to see it report without installing Sentry, but it's not very user-friendly [22:39:48] tgr: yeah, totally. I'm listening to mw.trackSubscribe already [22:40:05] mw.trackSubscribe('', function (topic, data) { [22:40:06] console.log('[mw.track] ' + topic, data); [22:40:06] }); [22:40:15] part of my browser pratically :) [22:40:47] looking forward to see what Raven does in this stack. Been catching up with that part. [22:40:50] OK. gtg [22:41:06] cool, thanks again for helping with this [23:37:32] bd808: do you know how I can increase the line length for logging? [23:37:52] I want to verify some long DB query [23:38:20] line lenght...? [23:38:29] things are getting truncated? [23:38:51] yes [23:38:58] oh I think the db queries may do that internally somewhere. [23:39:47] I can work around it if it's not easy to change, I was hoping there is some config setting for this [23:42:07] tgr: $wgDebugDumpSqlLength [23:42:34] thanks! [23:42:39] line 1149 of Databsase.php [23:42:41] I'll add that to the mw.org page [23:43:10] this all needs to be redone... [23:43:23] so much logging work in the world [23:43:48] oh, pre-structured stuff? [23:56:56] gi11es: I just started an upload with that testwiki debug extension, I think. [23:58:31] tgr, rather. ^ [23:59:09] gotta go for now, but happy to do it again tomorrow if it's helpful and there's anything I missed. [23:59:14] ragesoss: thanks! [23:59:32] not sure how well it works for uploads but we'll see