[16:10:16] Amir1: i guess you're the mediawiki media thumbnails expert now, sucks to be you. does the issue here ring any bells? https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Attack_at_Fromelles [16:17:13] 😭 [16:17:17] I can check after meeting [16:17:57] !bash <+MatmaRex> Amir1: i guess you're the mediawiki media thumbnails expert now, sucks to be you. does the issue here ring any bells? [16:17:57] taavi: Stored quip at https://bash.toolforge.org/quip/v6DyyJUBffdvpiTrTFXz [16:18:09] xD [16:18:34] thanks [16:18:37] thanks, both ;) [18:21:58] Krinkle: TimStarling: https://phabricator.wikimedia.org/T389734#10670365 is a potential excimer timeout issue, perhaps you might be able to guess what's happening [18:26:18] SUL3 issue? https://logstash.wikimedia.org/app/discover#/doc/0fade920-6712-11eb-8327-370b46f9e7a5/ecs-mediawiki-1-1.11.0-7-2025.13?id=GjBhyZUBRNrRHqkyVDOk [18:26:33] https://logstash.wikimedia.org/goto/f14e543d788732a9a7b2eb811117c8e9 [18:28:06] Amir1: unlikely. Reedy worked on that script recently [18:28:33] https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CentralAuth/+/1130643 [18:28:39] rofl [18:28:40] Reedy: you broke something :( [18:28:49] Well, I ran the script [18:28:55] I suspect it was already broken [18:29:18] 'lu_local_id' => $localNameToUid[$uname], [18:30:45] I wonder how much of that correlates with rows not being fixed [18:31:27] so that looks like it would set lu_local_id=null for those rows, because the local user really does not exist [18:32:23] i'm not entirely familiar with this code, but should it perhaps… delete those rows? is that a thing that we do? [18:33:07] (yeah, we do that when unattaching accounts) [18:33:55] huh. why does the schema even allow lu_local_id=null?? [18:37:24] are you making it a DBA problem? I see what you're doing 🤨 [18:38:02] even better, it defaults to null [18:38:14] I've put a patch up to prevent that specific logspam at least [18:38:26] As MatmaRex suggests... There's probably a delete or something we should be doing [18:43:09] Reedy: looking at your patch https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CentralAuth/+/1130695 , is it missing an `} else {` ? [18:43:33] lol [18:43:40] I started doing one thing and then... [19:05:52] I guess we need (to run) a script that either creates global accounts for those.. or deletes rows from localuser where name is not in globaluser [21:03:32] deleting is less hassle, otherwise you'd need to make sure they all belong to the same user [21:36:56] tgr_: thoughts re local require-dev? https://gerrit.wikimedia.org/r/c/mediawiki/extensions/OAuth/+/1130620 [21:41:30] That's usually how we do things... [23:24:51] replied there