[01:06:19] legoktm: Might be an issue with extension repos at https://tools.wmflabs.org/coverage/charts/ - e.g. https://tools.wmflabs.org/coverage/charts/AdvancedSearch.png and others all dropping to zero and no further data points. [01:07:27] nvm, is just that one [01:07:35] https://tools.wmflabs.org/coverage/charts/WikibaseMediaInfo.png did drop as well, but to 10% not 0%. [01:07:38] so presumably genuine [17:36:27] James_F: Hm.. 'jquery.tipsy' is loaded by mmv? [17:36:49] (in the initial preloaded moduled) [17:37:11] Krinkle: Sadly. [17:37:52] Oh wow, and not a trivial amount of calls to it either. [17:37:55] aye, that's too bad. [18:05:38] Maybe I can lure Ed into replacing it at some point. [19:06:27] duesen: I wonder if you have an opinion on what to do about T231488? The multiple intricate patches being landed in the same week makes it very hard to revert… [19:06:28] T231488: UploadFromChunks.php: Call to undefined method MediaWiki\FileBackend\FSFile\TempFSFileFactory::getTempFSFile() - https://phabricator.wikimedia.org/T231488 [19:06:38] James_F: making a hot fix for the UploadFromChunks issue [19:06:47] duesen: Cool, thanks. [19:07:00] (This is why MW needs proper test coverage, *sighs*.) [19:07:30] James_F: yes - simetrical has been working hard to improve that. but making things testable means refactoring. which breaks things because there are no tests... [19:07:59] Also, testing all the different file backends is hard, because you need to have the actual backends set up... [19:08:04] anyway. I'll go and write code. [19:08:10] Oh, absolutely. But also MW has integration tests for uploading via API but not via actual UI, because the API was added later after we started to care about testing. [19:08:18] Systemic failure over years, sadly. [19:10:38] yay :P [19:10:52] James_F: one line patch is up, see https://gerrit.wikimedia.org/r/c/mediawiki/core/+/533079 [19:10:55] that *should* fix it [19:11:04] i'll work on a regression test now [19:11:08] Thanks. [19:11:20] note that i didn't test it :P [19:11:37] I have this great system to test it in called production. ;-) [19:13:00] James_F: ApiUploadTest has @group Broken [19:13:02] so it's skipped [19:13:11] it would have actually caught the issue [19:13:34] let's see how broken it is... [19:14:02] * James_F sighs. Even better. [19:14:29] passes for me locally [19:14:42] i'll make a patch that enables it. let's see what jenkins has to say about it.