[00:43:04] (03CR) 10Bmansurov: [C: 031] Remove undeclared dependency on mw.user [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/198414 (https://phabricator.wikimedia.org/T93310) (owner: 10Gergő Tisza) [01:06:28] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1147373 (10bd808) [03:15:45] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Wrap jQuery AJAX callbacks in try..catch via mw.errorLogging - https://phabricator.wikimedia.org/T92247#1147500 (10Tgr) 5Open>3stalled Discussed with Ori and agreed to abandon this for now, concentrate on the... [03:18:25] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Test the impact of Javascript error logging on performance - https://phabricator.wikimedia.org/T92701#1147506 (10Tgr) Discussed with Ori, performance impact is no big deal but async JS wrapping is pushed back (may... [03:34:12] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Test the impact of Javascript error logging on performance - https://phabricator.wikimedia.org/T92701#1147518 (10Tgr) >>! In T92701#1146920, @Tgr wrote: > (side note: it should probably implement the [[ https://de... [03:34:54] 6Multimedia, 10Continuous-Integration, 10MediaWiki-extensions-ImageMetrics, 3Multimedia-Sprint-2015-03-18: Karma failing on an ImageMetrics test referencing a value set in a head script - https://phabricator.wikimedia.org/T93459#1147519 (10Tgr) [04:18:17] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1147632 (10bd808) [04:25:02] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1147645 (10bd808) [04:39:07] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1147688 (10bd808) [04:41:56] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1147700 (10bd808) [04:42:30] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1096558 (10bd808) [04:47:49] (03PS1) 10Gergő Tisza: Mock test dependency [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/199577 (https://phabricator.wikimedia.org/T93459) [04:52:45] (03CR) 10Gergő Tisza: [C: 032] "Self-merge, tests are breaking on independent patches." [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/199577 (https://phabricator.wikimedia.org/T93459) (owner: 10Gergő Tisza) [04:54:41] (03CR) 10Gergő Tisza: "recheck" [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/198414 (https://phabricator.wikimedia.org/T93310) (owner: 10Gergő Tisza) [04:55:45] (03Merged) 10jenkins-bot: Mock test dependency [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/199577 (https://phabricator.wikimedia.org/T93459) (owner: 10Gergő Tisza) [04:57:18] (03CR) 10Gergő Tisza: "recheck" [extensions/ImageMetrics] - 10https://gerrit.wikimedia.org/r/198414 (https://phabricator.wikimedia.org/T93310) (owner: 10Gergő Tisza) [06:09:26] 6Multimedia, 6Commons, 6Editing, 10Possible-Tech-Projects, and 4 others: Allow uploading of 3D files to Wikimedia Commons - https://phabricator.wikimedia.org/T3790#1147798 (10NiharikaKohli) [09:29:10] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1148036 (10Qgil) For the security review part, if we have candidate(s), mentors, and community excited about this project, we don't need to require t... [09:31:29] 6Multimedia, 6Release-Engineering, 6Scrum-of-Scrums, 7Blocked-on-RelEng, and 2 others: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1148047 (10Gilles) Repo is live at gerrit.wikimedia.org/r/operations/software/sentry and is a clone of yesterday's state of sentry on github [09:52:07] gilles: any clue how you will sort up sentry dependencies ? [09:52:24] would operations/software/sentry just contains everything you need and then git-deploy it ?:D [09:53:41] not sure yet, since trebuchet will be used to send the contents of that repo to a specific machine [09:53:46] whose purpose will be solely to run sentry [09:54:05] can't puppet run the regular setup.py in there which I assume will pull the dependencies? [09:56:27] I've never done that stuff before. if the dependencies needs to be defined in puppet it's not a big deal, the list isn't that long [10:22:08] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1148227 (10Tgr) If the script is going to be loaded in an off-domain iframe, we can be fairly lax about security. (The capability to do that in a gen... [10:23:33] (03PS3) 10Hashar: Use tox for running flake8 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/196744 (owner: 10Legoktm) [10:23:41] (03CR) 10Hashar: [C: 032] Use tox for running flake8 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/196744 (owner: 10Legoktm) [10:26:38] (03Merged) 10jenkins-bot: Use tox for running flake8 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/196744 (owner: 10Legoktm) [11:28:30] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1148316 (10Bawolff) >>! In T93679#1148227, @Tgr wrote: > If the script is going to be loaded in an off-domain iframe, we can be fairly lax about secu... [13:29:50] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10Quality-Assurance: Automated screenshots - https://phabricator.wikimedia.org/T77634#1148579 (10zeljkofilipin) We will investigate an existing tool at Lyon hachathon: T90884. [13:43:44] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Browser-Tests, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1148603 (10zeljkofilipin) p:5Triage>3Normal [13:44:25] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 7Browser-Tests: UploadWizard API tests failing on beta Commons due to login problem - https://phabricator.wikimedia.org/T89272#1148605 (10zeljkofilipin) p:5Triage>3Normal [13:53:50] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 7Browser-Tests: Automated screenshots - https://phabricator.wikimedia.org/T77634#1148660 (10zeljkofilipin) [13:55:23] 6Multimedia, 6MediaWiki-Core-Team, 6Parsoid-Team, 6Release-Engineering, and 3 others: Prepare Platform/Ops April 2015 quarterly review presentation - https://phabricator.wikimedia.org/T91803#1148666 (10Qgil) [14:06:27] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 7Browser-Tests: Links broken in UploadWizard-api-commons.wikimedia.beta.wmflabs.org job error report - https://phabricator.wikimedia.org/T66119#1148762 (10zeljkofilipin) [14:07:15] tgr|away: Any chance you can whip up a translatable PDF information page today? :D [14:09:58] 6Multimedia, 6Security, 7Design, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1148806 (10MarkTraceur) [14:47:03] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Move things around in preparation for moving newUpload to the Upload step controller - https://phabricator.wikimedia.org/T86679#1148932 (10MarkTraceur) [14:47:50] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move newUpload into the Upload step controller - https://phabricator.wikimedia.org/T93889#1148934 (10MarkTraceur) 3NEW a:3MarkTraceur [14:58:31] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1148971 (10MarkTraceur) 3NEW a:3MarkTraceur [14:58:49] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1148971 (10MarkTraceur) This method is in mw.UploadWizard.js and it accesses this.uploads... [14:58:59] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1148979 (10MarkTraceur) [15:00:42] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move showNext into the step controllers (and simplify it) - https://phabricator.wikimedia.org/T93892#1148989 (10MarkTraceur) 3NEW a:3MarkTraceur [15:01:12] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move startUploads into the upload step controller - https://phabricator.wikimedia.org/T93893#1148996 (10MarkTraceur) 3NEW a:3MarkTraceur [15:01:21] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move startUploads into the upload step controller - https://phabricator.wikimedia.org/T93893#1148996 (10MarkTraceur) [15:01:22] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move showNext into the step controllers (and simplify it) - https://phabricator.wikimedia.org/T93892#1148989 (10MarkTraceur) [15:04:02] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: "Error, running transcode job, for job that has already started" notices - https://phabricator.wikimedia.org/T89977#1149012 (10Gilles) p:5High>3Low It's not very frequent, I see one occurence in the... [15:05:13] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move resetFileStepUploads into the file step - https://phabricator.wikimedia.org/T93894#1149016 (10MarkTraceur) 3NEW a:3MarkTraceur [15:05:20] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move newUpload into the Upload step controller - https://phabricator.wikimedia.org/T93889#1149024 (10MarkTraceur) [15:05:21] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move resetFileStepUploads into the file step - https://phabricator.wikimedia.org/T93894#1149016 (10MarkTraceur) [15:06:11] Excuse me while I file all the bugs. [15:06:26] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Flickr option tends to reappear despite not having flickr import rights - https://phabricator.wikimedia.org/T89012#1149025 (10Gilles) 5Open>3Resolved Verified in production [15:07:12] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Remove this.uploads from the UploadWizard class - https://phabricator.wikimedia.org/T93895#1149028 (10MarkTraceur) 3NEW a:3MarkTraceur [15:07:34] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move resetFileStepUploads into the file step - https://phabricator.wikimedia.org/T93894#1149039 (10MarkTraceur) [15:07:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move newUpload into the Upload step controller - https://phabricator.wikimedia.org/T93889#1149038 (10MarkTraceur) [15:07:36] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move showNext into the step controllers (and simplify it) - https://phabricator.wikimedia.org/T93892#1149040 (10MarkTraceur) [15:07:38] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Remove this.uploads from the UploadWizard class - https://phabricator.wikimedia.org/T93895#1149037 (10MarkTraceur) [15:07:39] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move startUploads into the upload step controller - https://phabricator.wikimedia.org/T93893#1149041 (10MarkTraceur) [15:07:42] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1149042 (10MarkTraceur) [15:07:54] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: UploadWizardFlowEvent schema violation - https://phabricator.wikimedia.org/T89167#1149044 (10Gilles) 5Open>3Resolved [15:08:28] 6Multimedia, 10Continuous-Integration, 10MediaWiki-extensions-ImageMetrics, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Karma failing on an ImageMetrics test referencing a value set in a head script - https://phabricator.wikimedia.org/T93459#1149053 (10Gilles) 5Open>3Resolved [15:08:57] 6Multimedia, 10Continuous-Integration, 10MediaWiki-extensions-MultimediaViewer, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Chromium 41.0.2272 (Ubuntu) mmv.ui.ProgressBar jumpTo()/hide() FAILED / animateTo() FAILED - https://phabricator.wikimedia.org/T93540#1149055 (10Gilles) 5Open>3Resolved [15:09:23] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Use promises in handlers and transports - https://phabricator.wikimedia.org/T92640#1149057 (10Gilles) 5Open>3Resolved [15:09:29] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Fix iframe transport bugs introduced in refactor - https://phabricator.wikimedia.org/T93206#1149059 (10Gilles) 5Open>3Resolved [15:09:30] I think that, if everything goes according to plan, I can add "remove" buttons to every step. [15:09:34] Trivially. [15:09:38] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Move details-related methods on mw.UploadWizard.prototype to the details step - https://phabricator.wikimedia.org/T90771#1149061 (10Gilles) 5Open>3Resolved [15:11:20] 6Multimedia, 6Commons, 10MediaWiki-File-management, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: File cache is not updated correctly after move/delete/etc, causing files to disappear until purged - https://phabricator.wikimedia.org/T93009#1149068 (10Gilles) 5Open>3Resolved [15:12:23] 6Multimedia, 10Incident-20150312-whitespace, 10MediaWiki-File-management, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: thumb.php should not return HTTP 200 when streaming the file fails - https://phabricator.wikimedia.org/T92545#1149069 (10Gilles) 5Open>3Resolved I meant testing in production, b... [15:12:42] 6Multimedia, 3Multimedia-Sprint-2015-03-18, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1149071 (10Gilles) Can we revert that change soon? [15:13:29] marktraceur: is it realistic to have them all in the current sprint? do you think you're going to do all of them this coming sprint? [15:14:41] 6Multimedia, 6Release-Engineering, 6Scrum-of-Scrums, 7Blocked-on-RelEng, and 3 others: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1149083 (10Gilles) a:3Gilles [15:36:22] gilles: I think a good portion of them will be done this sprint. [15:36:27] Er, next sprint. [15:36:36] Within the next week or so. [15:37:14] alright [15:37:31] wanna leave most of them as 0 points? are they super easy? [15:39:46] gilles: I'll run through them and set it [15:39:53] I think they're mostly 0.5 [15:40:06] Like, that's how well this refactor is going [15:40:51] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move newUpload into the Upload step controller - https://phabricator.wikimedia.org/T93889#1149147 (10MarkTraceur) [15:40:57] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1149149 (10MarkTraceur) [15:41:02] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move showNext into the step controllers (and simplify it) - https://phabricator.wikimedia.org/T93892#1149150 (10MarkTraceur) [15:41:07] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move startUploads into the upload step controller - https://phabricator.wikimedia.org/T93893#1149152 (10MarkTraceur) [15:41:11] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Move resetFileStepUploads into the file step - https://phabricator.wikimedia.org/T93894#1149154 (10MarkTraceur) [15:41:18] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-18: Remove this.uploads from the UploadWizard class - https://phabricator.wikimedia.org/T93895#1149158 (10MarkTraceur) [15:41:29] gilles: I think that's a total of like 4-5 [15:41:33] Which should be doable. [15:41:49] from what I can see, marktraceur is going to complete 7 cards this week. [15:42:58] That marktraceur is a real go getter with upper middle management written all over him [15:43:20] Don't joke about that man [15:43:35] I'm a PM. I never joke [15:44:03] Aaaaaahhhhhhh [15:44:07] * marktraceur runs [15:44:51] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-25: Remove this.uploads from the UploadWizard class - https://phabricator.wikimedia.org/T93895#1149163 (10Gilles) [15:44:52] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-25: Move resetFileStepUploads into the file step - https://phabricator.wikimedia.org/T93894#1149164 (10Gilles) [15:44:55] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-25: Move newUpload into the Upload step controller - https://phabricator.wikimedia.org/T93889#1149168 (10Gilles) [15:44:56] 6Multimedia, 6Release-Engineering, 6Scrum-of-Scrums, 7Blocked-on-RelEng, and 3 others: Create basic puppet role for Sentry - https://phabricator.wikimedia.org/T84956#1149170 (10Gilles) [15:44:58] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: "Error, running transcode job, for job that has already started" notices - https://phabricator.wikimedia.org/T89977#1149171 (10Gilles) [15:44:59] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 10UploadWizard-Refactoring, 3Multimedia-Sprint-2015-03-25: Change isComplete to query step objects instead of upload objects - https://phabricator.wikimedia.org/T93891#1149167 (10Gilles) [15:45:00] 6Multimedia, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1149172 (10Gilles) [15:45:02] 6Multimedia, 10MediaWiki-extensions-ImageMetrics, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: mw.user.isAnon is not a function - https://phabricator.wikimedia.org/T93310#1149177 (10Gilles) [15:45:03] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Browser-Tests, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1149176 (10Gilles) [15:45:05] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Make uploads and details use promises for their transitions, and change transition methods to use them - https://phabricator.wikimedia.org/T91514#1149175 (10Gilles) [15:45:08] 6Multimedia, 6Security, 7Design, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Show warning about privacy/security issues on PDF file pages - https://phabricator.wikimedia.org/T89765#1149173 (10Gilles) [15:45:10] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Test the impact of Javascript error logging on performance - https://phabricator.wikimedia.org/T92701#1149178 (10Gilles) [15:45:11] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Wrap jQuery AJAX callbacks in try..catch via mw.errorLogging - https://phabricator.wikimedia.org/T92247#1149179 (10Gilles) [15:45:13] 6Multimedia, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Have step controllers drive the process and be linked together - https://phabricator.wikimedia.org/T93099#1149180 (10Gilles) [15:45:15] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Displaying the stashed file as a thumbnail in UploadWizard fails with an 503 - https://phabricator.wikimedia.org/T90599#1149181 (10Gilles) [15:45:17] 6Multimedia, 6MediaWiki-Core-Team, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: SHA-1 file name support - https://phabricator.wikimedia.org/T1210#1149182 (10Gilles) [15:45:18] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25: Update server-side JS error-logging RfC - https://phabricator.wikimedia.org/T91357#1149185 (10Gilles) [15:46:12] marktraceur: did you get the memo about TPS reports? [15:49:35] gilles: Yeah. I shredded it into confetti. It tasted awful. [15:53:24] * bd808 puts strychnine in the guacamole [15:56:59] gilles: Do you want to answer bawolff's question about the lack of a MM team or would you rather I spew some nonsense at him and the list? [15:57:16] I must have missed that email, which list? [15:57:23] wikitech-l [15:57:26] ~4 hours ago [15:58:32] well, my answer would be that we'll keep supporting the extensions that multimedia user to cover, in terms of addressing emergencies [15:58:53] agreed [15:59:08] active development will continue on a per-project/per-extension basis as part of other teams, or teams yet to be formed [15:59:55] and big projects that haven't been started yet will need to be scoped and proposed at the WMF planning level (eg structured commons) [16:00:08] the catch-all "multimedia" umbrella won't be a thing anymore, probably for good, because it was just as problematic as core's [16:01:44] bd808: should I sum this up into a reply, or are you already writing one? [16:02:55] If you've got the time I think it would be good for the info to come from you [16:03:20] I shall do that now, I have time because I thought our weekly meeting started 30 minutes earlier than it actually does :) [16:03:24] you are MM after all; I'm just some kid that they gave a weird interim title to [16:04:01] seems like I'm about to lose my own interim title [16:10:48] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1149305 (10Snhkicker) This link is to a heroku site i made to prove what i'm thinking and to show you the possible problems associated https://obscu... [16:19:45] replied... [16:20:27] now, what will do about this irc channel? [16:21:03] leave it be, for people who care about this topic? [16:21:38] I'll probably come here to rant about UploadWizard anyway. [16:21:50] We could leave the bots in here for Multimedia stuff. [16:26:48] 6Multimedia, 6operations: Add monitoring of upload rate on commons to icingia alerts - https://phabricator.wikimedia.org/T92322#1149372 (10fgiunchedi) btw the missing metrics should be related to T85641 [16:32:25] marktraceur: sorry, I will do it for reals today [16:32:52] tgr: No problem. I added a task for you even. [16:42:25] 6Multimedia, 6operations: Add monitoring of upload rate on commons to icingia alerts - https://phabricator.wikimedia.org/T92322#1149436 (10fgiunchedi) a:3fgiunchedi [16:42:40] 6Multimedia, 6operations: Add monitoring of upload rate on commons to icingia alerts - https://phabricator.wikimedia.org/T92322#1106360 (10fgiunchedi) p:5Triage>3Normal [17:47:24] gilles, tgr, was https://phabricator.wikimedia.org/T84956 the blocked task or did I misspeak? [17:54:18] Should we kill the weekly meeting? [17:56:17] absolutely [17:56:38] and yes that was the task which I'm working on completely cluelessly [17:56:46] Fun times. [17:57:00] marxarelli: ^^ confirmation that gilles needs help [17:57:03] And with the task too. [17:57:27] With that, it looks to be about lunch time. [17:57:53] got it. i'll find it a friend in releng [17:59:22] antoine's been advising me a bit during european hours, not sure if he wants to do more of that :) [18:00:11] i'm happy to help as well if you need general puppet advice. things are a bit different when it comes to ops puppet, however [18:00:29] e.g. i'm not sure they'll be down with a non-package installation [18:01:48] they're the ones who actually advised us to use trebuchet for this [18:02:12] good advice, I think, since sentry needs strange versions for all its dependencies that don't map well to any of the distros we use [18:02:23] ah, ok. no worries then [18:02:50] I actually tried packaging debs but when I compared versions I would have had to override so much stuff it was bound to cause an issue with another python thing we have in prod [18:03:22] * marxarelli nods [18:03:22] and we only need sentry on a machine dedicated to it, so it makes sense not to pollute apt with this [18:05:38] 6Multimedia, 6operations: Add monitoring of upload rate on commons to icingia alerts - https://phabricator.wikimedia.org/T92322#1149719 (10Tgr) [18:07:20] 6Multimedia, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1149753 (10Tgr) I'd like to do a sanity check first - do a non-CORS load at the same time as the CORS load, and see how often that fails. I find it hard to... [18:21:33] gilles: how does https://gerrit.wikimedia.org/r/#/c/199598/1/modules/sentry/manifests/init.pp work? [18:21:46] I don't get where it takes the pip packages from [18:54:50] tgr: I expect trebuchet to copy the entire sentry repo to /srv/deployment/sentry/sentry [18:55:18] so pip install is pointed to that folder, which finds setup.py and downloads the exact version it needs [18:55:40] for each python package sentry requires [18:55:53] but won't pip still install from non-WMF servers that way? [18:56:12] I thought avoiding that is the whole point [18:56:51] ah, I didn't know that. we do get our apt packages from non-wmf servers most of the time, is the pip repo consider more insecure than that? [18:56:58] considered [18:57:19] I think so [18:57:22] I can download all the dependencies and have puppet install them, not a big deal [18:57:28] I mean bake them into the sentry repo [18:57:56] IMO that would be ideal [18:58:11] we might be able to point pip to a folder to avoid redefining the dependencies in puppet, I'll see what can be done [18:58:12] basically run pip on a staging server and commit the results [18:59:12] https://wikitech.wikimedia.org/wiki/Puppet_coding#Packages_that_use_pip.2C_gem.2C_etc. [18:59:23] not sure if this page is still up-to-date [18:59:52] probably not [19:00:10] I'll figure something out tomrrow to avoid hitting external urls [20:34:57] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1150630 (10Snhkicker) @jayvdb can you give me a task related to this proposal? [21:10:01] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1150788 (10Bawolff) Some potential microtasks (these might not be perfectly on topic but hopefully kind of relavent. I havent looked at them in detai... [21:10:38] 6Multimedia, 10MediaWiki-File-management, 7Easy: PNG metadata extractor doesn't properly convert dates - https://phabricator.wikimedia.org/T58064#621686 (10Bawolff) [21:19:50] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1150837 (10Tgr) Maybe @Rillke or @Gilles can suggest a microtask (or just give advice in general)? The [[ https://phabricator.wikimedia.org/project/p... [21:30:30] (03PS1) 10MarkTraceur: Move showNext into the step controllers [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/199745 (https://phabricator.wikimedia.org/T93892) [21:44:02] (03PS1) 10MarkTraceur: Move setUploadFilled into the file step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/199749 (https://phabricator.wikimedia.org/T93888) [21:45:02] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Browser-Tests, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1151010 (10Tgr) [21:46:14] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Browser-Tests, 5Patch-For-Review: Automated tests for Sentry error logging - https://phabricator.wikimedia.org/T88078#1003637 (10Tgr) Backlogging. As long as we are not going to do any of the complicated callback wrapping, setting up an automated test is not rea... [21:47:13] 6Multimedia, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Wrap jQuery AJAX callbacks in try..catch via mw.errorLogging - https://phabricator.wikimedia.org/T92247#1151023 (10Tgr) [21:51:31] 6Multimedia, 10MediaWiki-extensions-Sentry, 10MediaWiki-extensions-UploadWizard, 7Epic, and 2 others: Deploy Sentry (JavaScript error logging) to production, configured to log only UploadWizard - https://phabricator.wikimedia.org/T91649#1151044 (10Tgr) [21:51:33] 6Multimedia, 10MediaWiki-extensions-Sentry, 6operations: Procure hardware for Sentry - placeholder (not a live request) - https://phabricator.wikimedia.org/T93138#1151045 (10Tgr) [21:52:13] 6Multimedia, 10MediaWiki-extensions-Sentry, 10MediaWiki-extensions-UploadWizard, 7Epic, and 3 others: Deploy Sentry (JavaScript error logging) to production, configured to log only UploadWizard - https://phabricator.wikimedia.org/T91649#1151054 (10gpaumier) [21:55:43] (03PS2) 10MarkTraceur: [WIP] Move setUploadFilled into the file step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/199749 (https://phabricator.wikimedia.org/T93888) [21:55:45] That'll need to wait... [21:55:48] See y'all tomorrow. [21:57:29] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Move setUploadFilled into the file step [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/199749 (https://phabricator.wikimedia.org/T93888) (owner: 10MarkTraceur) [21:57:43] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Tracking: Improve Sentry coverage - https://phabricator.wikimedia.org/T88399#1151099 (10Tgr) After some rounds of discussion, we decided to only implement global error handling for now. That catches less errors and retrieves less information about them than callba... [22:04:36] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Tracking: Improve Sentry coverage - https://phabricator.wikimedia.org/T88399#1151163 (10Tgr) [22:04:37] 6Multimedia, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Catch uncaught exceptions with Sentry - https://phabricator.wikimedia.org/T88874#1151161 (10Tgr) 5Resolved>3Open Reopening - removing the default handler is crappy behavior. Sentry should just unregister itself from mw.track instead. Also, th... [22:22:11] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Tracking: Improve Sentry coverage - https://phabricator.wikimedia.org/T88399#1151243 (10Tgr) [22:23:16] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Use source urls in mw.loader.store - https://phabricator.wikimedia.org/T90524#1061235 (10Tgr) [22:24:24] 6Multimedia, 10MediaWiki-extensions-Sentry, 6operations, 3Multimedia-Sprint-2015-03-25: Procure hardware for Sentry - placeholder (not a live request) - https://phabricator.wikimedia.org/T93138#1151260 (10Tgr) [22:25:42] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Catch uncaught exceptions with Sentry - https://phabricator.wikimedia.org/T88874#1151266 (10Tgr) [22:26:55] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25, 5Patch-For-Review: Catch uncaught exceptions with Sentry - https://phabricator.wikimedia.org/T88874#1151279 (10Tgr) p:5Triage>3High [22:55:28] no thumbnails in UpWiz from stash :-( http://imgur.com/jr0TLFX [22:56:02] neither from stash nor comupted [22:56:15] computed client side [23:16:38] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-Uploading: Upload errors (through all ways) at Wikimedia Commons - https://phabricator.wikimedia.org/T92790#1151446 (10Rillke) 5Open>3stalled [23:17:37] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7Tracking: Chunked upload (tracking) - https://phabricator.wikimedia.org/T61914#1151456 (10Rillke) [23:17:38] 6Multimedia, 10MediaWiki-File-management, 5Patch-For-Review: Chunked upload offset error should specify expected offset - https://phabricator.wikimedia.org/T87535#1151453 (10Rillke) 5Open>3Resolved a:3Rillke Verified in production. [23:25:46] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Distorted thumbnail for audio files due to wrong size - https://phabricator.wikimedia.org/T68852#1151524 (10Rillke) [23:35:52] 6Multimedia, 10MediaWiki-extensions-Sentry, 7JavaScript: Make sure async wrapping for Javascript error logging only happens when the call is truly async - https://phabricator.wikimedia.org/T93392#1151595 (10Tgr) [23:35:52] 6Multimedia, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Wrap jQuery AJAX callbacks in try..catch via mw.errorLogging - https://phabricator.wikimedia.org/T92247#1151594 (10Tgr) [23:39:09] 6Multimedia, 10MediaWiki-extensions-Sentry, 7Tracking: Improve Sentry coverage - https://phabricator.wikimedia.org/T88399#1151605 (10Tgr) [23:39:11] 6Multimedia, 10MediaWiki-extensions-Sentry, 5Patch-For-Review: Implement module wrapping for Sentry - https://phabricator.wikimedia.org/T78809#1151606 (10Tgr) [23:39:12] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-Editing, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#1151601 (10Tgr) 5Open>3stalled Abandoned for now, see T92247#1147500. [23:39:30] 6Multimedia, 6Commons, 6Editing, 10Security-Reviews, and 3 others: Uploading and Displaying 3D files with Fallback - https://phabricator.wikimedia.org/T93679#1151608 (10Rillke) - Creating thumbnails is essentially for getting media handling extensions installed at any Wikimedia Wiki. Making different smart... [23:39:40] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-Editing, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#1151616 (10Tgr) [23:39:41] 6Multimedia, 10MediaWiki-extensions-Sentry, 7JavaScript: Make sure async wrapping for Javascript error logging only happens when the call is truly async - https://phabricator.wikimedia.org/T93392#1151612 (10Tgr) 5Open>3stalled Abandoned for now, see T92247#1147500. [23:42:25] 6Multimedia, 6ArchCom, 10MediaWiki-RfCs, 7Epic: RfC: Server-side Javascript error logging - https://phabricator.wikimedia.org/T382#1151643 (10Tgr) [23:42:26] 6Multimedia, 5Patch-For-Review: Wrap scripts with exception handling for automatic JS error logging - https://phabricator.wikimedia.org/T513#1151641 (10Tgr) 5Open>3stalled We are not doing this for the first iteration - might be revisited when we have data on how much it would improve the coverage. See T92... [23:43:31] 6Multimedia, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-Sentry, 7Blocked-on-Editing, 5Patch-For-Review: Add startup script to automatically wrap asynchronous functions in try..catch - https://phabricator.wikimedia.org/T85262#1151654 (10Tgr) [23:43:32] 6Multimedia, 5Patch-For-Review: Wrap scripts with exception handling for automatic JS error logging - https://phabricator.wikimedia.org/T513#1151653 (10Tgr) [23:49:59] 6Multimedia, 6ArchCom, 10MediaWiki-RfCs, 7Epic: RfC: Server-side Javascript error logging - https://phabricator.wikimedia.org/T382#1151692 (10Spage) [23:50:00] 6Multimedia, 10MediaWiki-extensions-Sentry, 3Multimedia-Sprint-2015-03-25: Update server-side JS error-logging RfC - https://phabricator.wikimedia.org/T91357#1151693 (10Spage)