[06:10:42] 10Multimedia, 10UploadWizard, 10Wiki-Loves-Monuments (2017): UploadCampaign sometimes use old autoAdd configuration - https://phabricator.wikimedia.org/T174914#4314134 (10Effeietsanders) @JeanFred is this still an issue? [10:37:52] (03CR) 10Matthias Mullie: "recheck" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/415330 (https://phabricator.wikimedia.org/T182307) (owner: 10Matthias Mullie) [10:44:04] (03CR) 10jerkins-bot: [V: 04-1] Rename 'Next' to 'Publish' [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/415330 (https://phabricator.wikimedia.org/T182307) (owner: 10Matthias Mullie) [11:29:06] 10Multimedia, 10Commons, 10MediaWiki-File-management, 10Performance-Team, 10Patch-For-Review: Support optimized WebP thumbnails as alternative to JPEG, PNG - https://phabricator.wikimedia.org/T27611#4314953 (10Gilles) A preview of what's to come, once the Varnish change gets merged. Thumbnail from the cu... [13:08:57] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315351 (10jcrespo) Sorry, I didn't understand your original report, @AlexisJazz - you were mentioning 2 errors that were not the main... [13:18:58] (03PS1) 10Esanders: Use stdClass to force JSON object encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442093 [13:35:02] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315466 (10AlexisJazz) @jcrespo it would be an incredible coincidence if the MWException error had nothing to do with this. Unfortunat... [13:35:53] (03PS1) 10Esanders: Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 [13:35:56] (03PS1) 10Esanders: Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 [13:38:13] (03PS2) 10Esanders: Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 [13:38:38] (03CR) 10jerkins-bot: [V: 04-1] Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 (owner: 10Esanders) [13:38:56] (03CR) 10jerkins-bot: [V: 04-1] Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [13:51:19] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315540 (10jcrespo) It certainly makes no sense to me, according to medatada, we have: ``` root@db1068.eqiad.wmnet[commonswiki]> sel... [13:57:13] 10Multimedia, 10Commons, 10MediaWiki-extensions-GWToolset, 10Move-Files-To-Commons, and 3 others: Re-implement GWToolset ontop of FileImporter - https://phabricator.wikimedia.org/T193526#4172026 (10Addshore) You could also look at the FileImporter as an improved version of the upload a file from a URL alre... [14:00:35] (03PS1) 10Esanders: Indentation fixes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442105 [14:07:22] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315590 (10jcrespo) This is a summary of the transaction, and the weird thing is the insert at the beginning- that insert shouldn't ha... [14:11:45] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315597 (10AlexisJazz) @jcrespo I don't know how the systems work, so I'm just guessing here. When I uploaded the first time, I think... [14:14:23] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315600 (10jcrespo) Oh, I just saw the `INSERT IGNORE`, so that should have just silently failed- it happens every time. I am going to... [14:28:26] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315655 (10jcrespo) This is the first relevant write to oldimage- it writes the wrong filename, `20180626085559!WikipediaTomeRaider3.p... [14:42:28] 10Multimedia, 10Commons, 10media-storage, 10Wikimedia-log-errors: Overwritten a file on Commons, literally, the original is gone - https://phabricator.wikimedia.org/T198177#4315705 (10AlexisJazz) @jcrespo Although something went wrong, the outcome is actually desirable. I may have requested a revdel otherw... [15:30:35] 10Multimedia, 10UploadWizard: Three suggestions for improvements - https://phabricator.wikimedia.org/T198223#4315843 (10ClemRutter) [15:34:46] (03CR) 10Cparle: [C: 032] Redesign arrow steps [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/434652 (https://phabricator.wikimedia.org/T197924) (owner: 10Matthias Mullie) [17:58:26] 10Multimedia, 10UploadWizard: Three suggestions for improvements - https://phabricator.wikimedia.org/T198223#4316636 (10Aklapper) 05Open>03Invalid Hi @ClemRutter, thanks for the feedback, however as written in T164263 and T171102, tasks have to deal with only one single issue per task (so it is possible to... [20:39:17] (03CR) 10Jforrester: "recheck" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [20:41:22] (03CR) 10jerkins-bot: [V: 04-1] Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [20:51:28] 10Multimedia, 10Structured-Data-Commons, 10Wikidata, 10Multimedia-Team-Working-Board: Investigate and test "statement traversal" for depicts search - https://phabricator.wikimedia.org/T198261#4317414 (10Ramsey-WMF) [20:53:34] (03PS2) 10Jforrester: Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [20:53:37] (03PS3) 10Jforrester: Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 (owner: 10Esanders) [20:53:40] (03PS2) 10Jforrester: Indentation fixes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442105 (owner: 10Esanders) [20:55:01] 10Multimedia, 10Structured-Data-Commons, 10Multimedia-Team-Working-Board: Investigate and test "statement traversal" for depicts search - https://phabricator.wikimedia.org/T198261#4317446 (10Ramsey-WMF) [20:55:50] 10Multimedia, 10Structured-Data-Commons, 10Multimedia-Team-Working-Board: Investigate and test "statement traversal" for depicts search - https://phabricator.wikimedia.org/T198261#4317414 (10Ramsey-WMF) p:05Triage>03High [20:56:35] (03CR) 10Jforrester: [C: 032] Use stdClass to force JSON object encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442093 (owner: 10Esanders) [21:12:46] (03Merged) 10jenkins-bot: Use stdClass to force JSON object encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442093 (owner: 10Esanders) [21:13:49] (03CR) 10Jforrester: [C: 032] Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [21:13:52] (03CR) 10Jforrester: [C: 032] Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 (owner: 10Esanders) [21:13:55] (03CR) 10Jforrester: [C: 032] Indentation fixes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442105 (owner: 10Esanders) [21:26:54] (03Merged) 10jenkins-bot: Use much smaller BlackPixel encoding [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442099 (owner: 10Esanders) [21:29:07] (03Abandoned) 10Jforrester: Use new path for register [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/350616 (owner: 10Esanders) [21:32:19] (03Merged) 10jenkins-bot: Use short array syntax and a consistent quoting style [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442100 (owner: 10Esanders) [21:32:22] (03Merged) 10jenkins-bot: Indentation fixes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442105 (owner: 10Esanders) [21:44:42] brion: So, good news; mediawiki.org is running wmf.10, Special:Version doesn't mention MwEmbedHandler, and yet the videos on https://www.mediawiki.org/wiki/Extension:TimedMediaHandler play as expected. Success! [21:53:49] Victory is ours! [21:54:29] Now we march on til videojs :) [22:36:15] brion: I'm working on extension.json-ifying. From scratch. Again. :-( [22:36:37] "whee" [22:36:46] It's… a mess. [22:36:50] ;_; [22:36:53] I think we're going to have to give up on the constants. [22:36:57] a lot of weird options in there yeah [22:37:04] i say we drop the constants in favor of just using the string keys [22:37:08] WebVideoTranscode::ENC_OGG_OPUS etc. [22:37:10] Yeah. [22:37:16] they're less verbose for one ;) [22:37:17] OK, if we do that it becomes lots easier. [22:37:22] and they work in the json much better [22:39:27] at runtime constants can be nice for guaranteeing they're defined but they're almost exclusively used in config here [22:39:32] and that's string territory big time [22:49:49] hrm how do i enable that new wikitext editor mode on my local setup... [22:50:52] found it [22:57:10] (03PS1) 10Jforrester: [WIP] Convert to extension.json [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442220 (https://phabricator.wikimedia.org/T87981) [22:57:33] brion: ^^ I feel unclean. [22:57:37] :-) [22:57:46] :D [22:59:42] (03CR) 10jerkins-bot: [V: 04-1] [WIP] Convert to extension.json [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/442220 (https://phabricator.wikimedia.org/T87981) (owner: 10Jforrester)