[05:02:32] 10GitLab (CI & Job Runners), 06Release-Engineering-Team, 06collaboration-services, 10function-evaluator, and 4 others: Wikifunctions function orchestrator and evaluator test suites failing on GitLab CI with OOM errors - https://phabricator.wikimedia.org/T399348#11442846 (10ecarg) I believe this is ready to... [08:26:51] 06Release-Engineering-Team (Doing 😎), 06Growth-Team, 10MediaWiki-extensions-CommunityConfiguration, 07Essential-Work: CommunityConfiguration unconditionally requires CommunityConfigurationExample - https://phabricator.wikimedia.org/T404939#11442962 (10hashar) [09:05:49] 06Release-Engineering-Team (Doing 😎), 06Growth-Team, 10MediaWiki-extensions-CommunityConfiguration, 07Essential-Work: CommunityConfiguration unconditionally requires CommunityConfigurationExample - https://phabricator.wikimedia.org/T404939#11443003 (10hashar) I have tried to have the tests skipped from the... [09:50:01] (03PS1) 10Hashar: Zuul: [WP25EasterEggs] add CommunityConfigurationExample [integration/config] - 10https://gerrit.wikimedia.org/r/1216759 (https://phabricator.wikimedia.org/T404939) [09:50:03] (03PS1) 10Hashar: Revert "Zuul: [WP25EasterEggs] add CommunityConfigurationExample" [integration/config] - 10https://gerrit.wikimedia.org/r/1216760 (https://phabricator.wikimedia.org/T404939) [09:51:01] (03CR) 10Hashar: [C:03+2] Zuul: [WP25EasterEggs] add CommunityConfigurationExample [integration/config] - 10https://gerrit.wikimedia.org/r/1216759 (https://phabricator.wikimedia.org/T404939) (owner: 10Hashar) [09:52:36] (03Merged) 10jenkins-bot: Zuul: [WP25EasterEggs] add CommunityConfigurationExample [integration/config] - 10https://gerrit.wikimedia.org/r/1216759 (https://phabricator.wikimedia.org/T404939) (owner: 10Hashar) [11:41:10] 10MediaWiki-Releasing, 05MW-1.45-release: Write and send release announcement for 1.45.0 - https://phabricator.wikimedia.org/T408481#11443610 (10MSantos) 05Open→03Resolved The dropping of PHP 8.1 support is still going, so it's not part of the release statement. I just wanted to let you know that what... [14:05:03] maintenance-disconnect-full-disks build 761756 integration-agent-docker-1049 (/: 36%, /srv: 98%, /var/lib/docker: 28%): OFFLINE due to disk space [14:10:03] maintenance-disconnect-full-disks build 761757 integration-agent-docker-1049 (/: 36%, /srv: 16%, /var/lib/docker: 27%): RECOVERY disk space OK [16:17:53] Hello. Could you tell me, please, if there is a place where I can officially ask to comment "recheck" on my Gerrit patch? It's a first one, I can't do it by myself yet, and I'm just waiting until somebody accidentally pays attention. I could work much much faster, but the Patch Demo needs Jenkins bot to run on every patch set, and I'm waiting for [16:17:53] hours, or even days this time. Thank you in advance. [16:29:52] Guest91: whats your gerrit link? [16:30:34] https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GlobalWatchlist/+/1211732 [16:32:39] Guest91: I see that is in "WIP" status and not yet in "ready for review" status. this might influence things [16:32:59] I could click "mark as active", ok? [16:33:40] ok, I sent the "recheck" first without changing anything else. [16:34:51] about the official place.. this channel is probably close to it [16:36:03] Exactly because of the WIP mark I need the bot. If "active" means "opened for review", than please don't, because I need to debug it first. Thank you for "recheck". If this is the place I hope people here will not hate me after many requests. Sure you want me to do it? [16:36:50] ok, got you. ok, only recheck and nothing else changed. [16:37:12] nah, keep doing it and eventually you can be added to a list of trusted users who can do it themselves [16:37:42] I am not actually in release engineering team but dont worry about people hating you [16:37:54] OK. Thanks again for your help. [16:38:23] no problem [18:03:21] (03update) 10audreypenven: Adds support for shellbox score with httpd service [repos/releng/cli] - 10https://gitlab.wikimedia.org/repos/releng/cli/-/merge_requests/640 [18:11:39] If it's not too much trouble, can anyone please comment 'recheck' on https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GlobalWatchlist/+/1211732 ? Thank you in advance. [18:13:19] done [18:13:30] Thanks a lot. [18:23:12] Can somebody do it again, please? Sorry. I just really want to finish the work and to open the patch for reviewers in the next minutes. [18:23:39] done [18:23:51] Thank you very much. [18:41:40] And again. Sorry! I'm sure 99% that this is a penultimate one, and the last one in a couple of minutes will be rebase. Thanks in advance. [18:46:26] done [18:46:58] You are great! [18:59:57] And one last time. I performed the rebase, and it's empty, so I believe it's just routine. Thank you again in advance. [19:02:41] IKhitron: done [19:03:35] IKhitron: if you plan to contribute often, you can get added to the CI list so you don't have to ask [19:04:44] Thank you. Yes, I know, but Pppery told me it can be done only after more a few patches. [19:05:32] IKhitron: I'd just make the patch to add you ready if I was you [19:06:53] wasnt sure if there is an actual rule :) ty RhinosF1 [19:06:54] Maybe. But I tend to listen to everything that (any) "big boss" said, so I'll wait for my second patch. [19:08:01] mutante: hashar normally reviews them patches but I'm fairly sure it's basically won't DDOS CI or upload malware [19:08:05] I don't think there's an active rule.... [19:08:12] *actual [19:08:17] but IKhitron isn't exactly new to the community etc either [19:08:19] https://www.mediawiki.org/wiki/Continuous_integration/Allow_list says "just that they're not malicious" [19:08:31] so before we type recheck on someone's patch.. we should scan for malware ?:) [19:08:31] so I'm happy to push out a patch adding IKhitron to the list [19:08:50] ack, thanks [19:09:03] mutante: yes, if you are re-checking someone's patch you're saying you've checked that it is safe to run on our infrastructure [19:09:16] s/run/test/ [19:10:19] ack, and this one seemed safe to me. but I won't necessarily feel that about everything [19:10:54] Thank you. About "but IKhitron isn't exactly new to the community etc either" - yes, I'm an interface admin on two wikis for years, hope it says something. [19:11:51] seems to me it does. +1 [19:12:02] so is someone making a patch? [19:24:17] (03PS2) 10RhinosF1: zuul: add IKhitron to allowlist [integration/config] - 10https://gerrit.wikimedia.org/r/1216841 [19:24:30] There you go [19:27:22] Thank you so much. [19:28:33] It will be very rude for me to ask somebody to remove the WIP mark from my patch? [19:29:51] IKhitron: do you not see a mark as active button? [19:31:05] No, I don't. And I didn't see the WIP button in the More menu earlier either. [19:31:54] marked as active [19:32:17] Thank you! [19:34:41] paladox: what are gerrit perms like for marking WIP / Active if not the patch owner (as IKhitron originally used the patch uploader)? I couldn't see mark as active either. I feel uploader/author should have that. [19:35:22] trusted contributor? [19:35:49] I am trusted contributor and I didn't see the mark as active flag [19:36:00] I'm guessing mutante did as they have admin [19:36:45] And thank you all again for your help and the welcoming. I'l add some reviewers, write testing explanations for them, and start thinking about the next patch, hopefully I will be able to crack phab:T278440, I want it to be solved for a long time, and now I can finally try. [19:36:45] T278440: Local time compatibility - https://phabricator.wikimedia.org/T278440 [19:38:34] (03CR) 10Dzahn: [C:03+1] "https://meta.wikimedia.org/wiki/Special:CentralAuth?target=IKhitron" [integration/config] - 10https://gerrit.wikimedia.org/r/1216841 (owner: 10RhinosF1) [19:39:00] (03CR) 10Majavah: [C:03+2] zuul: add IKhitron to allowlist [integration/config] - 10https://gerrit.wikimedia.org/r/1216841 (owner: 10RhinosF1) [19:40:03] I think patch owners can, I don't think we set the toggle work in progress. [19:40:32] (03Merged) 10jenkins-bot: zuul: add IKhitron to allowlist [integration/config] - 10https://gerrit.wikimedia.org/r/1216841 (owner: 10RhinosF1) [19:40:39] !log reload zuul for https://gerrit.wikimedia.org/r/c/integration/config/+/1216841 [19:40:40] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [19:41:25] paladox: is there any reason patch author / uploader / trusted contributors can't ? Is it configurable? [19:41:47] I think that Gerrit doesn't understand that I'm the author, it thinks that IKhitron in bold and IKhitron without bold are different people. Not sure, but it looks like that. [19:41:59] No, and yes. You can change https://gerrit.wikimedia.org/r/admin/repos/All-Projects,access to add that acl for the trusted group for refs/* [19:43:13] paladox: I'm guessing the no bit is for author / uploader. Where's gerrits upstream? [19:43:19] To file a feature request [19:46:36] the no is there's no reason (I don't believe) and you can change it for the wmf via the above by allowing the trusted group [19:46:46] https://www.gerritcodereview.com/issues.html [19:47:58] 10Gerrit: Allow Trusted-Contributors to change WIP/Active state of a patch - https://phabricator.wikimedia.org/T412150 (10RhinosF1) 03NEW [19:48:05] I suppose if you can't do it before you published a change to someone else's change then I can see the logic in why it doesn't allow you to touch wip/active [19:48:46] 10Gerrit: Allow Trusted-Contributors to change WIP/Active state of a patch - https://phabricator.wikimedia.org/T412150#11445203 (10RhinosF1) Allowing patch author / owner to update WIP/Active is probably worth upstream task too. Note to future me: https://www.gerritcodereview.com/issues.html [19:49:26] paladox: IKhitron had made multiple changes to https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GlobalWatchlist/+/1211732 and still couldn't see it [19:49:29] you need to join https://groups.google.com/g/repo-discuss to report issues on their bug tracker now (anti spam measure) [19:49:43] oh that's why [19:49:54] cuz the owner is Gerrit Patch Uploader [19:50:18] Ye but my point is author/uploader should be able to do it too paladox [19:50:25] In case someone else takes over a task [19:50:30] Patch* [19:51:08] yeh [19:53:24] paladox: so that bit future me can upstream and we can allow trusted contributors to do it too [19:53:37] yeh [19:53:58] Good [20:12:29] 10Continuous-Integration-Config, 06Release-Engineering-Team (Priority Backlog 📥), 10MediaWiki-Core-Tests, 10QS-Test-Automation, and 9 others: Reduce runtime of MW shared gate Jenkins jobs to 5 min - https://phabricator.wikimedia.org/T225730#11445235 (10vaughnwalters) [20:14:19] (03open) 10cjming: make-release: Start branching TestKitchen [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/226 (https://phabricator.wikimedia.org/T407806) [20:14:37] (03update) 10cjming: make-release: Start branching TestKitchen [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/226 (https://phabricator.wikimedia.org/T407806) [20:16:36] (03update) 10cjming: make-release: Start branching TestKitchen [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/226 (https://phabricator.wikimedia.org/T407806) [20:19:52] (03open) 10taavi: runner: Add context object to hold information about current run [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/97 [20:19:52] (03update) 10taavi: runner: Handle core's weird Jest config file location [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/96 [20:19:53] (03update) 10taavi: runner: Add context object to hold information about current run [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/97 [20:19:53] (03update) 10taavi: runner: Demo for handler not relying on current working directory [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/98 [20:19:54] (03open) 10taavi: runner: Demo for handler not relying on current working directory [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/98 [20:20:03] (03update) 10taavi: runner: Add context object to hold information about current run [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/97 [20:20:04] (03update) 10taavi: runner: Demo for handler not relying on current working directory [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/98 [21:45:03] maintenance-disconnect-full-disks build 761848 integration-agent-docker-1056 (/: 36%, /srv: 100%, /var/lib/docker: 32%): OFFLINE due to disk space [21:50:03] maintenance-disconnect-full-disks build 761849 integration-agent-docker-1056 (/: 36%, /srv: 33%, /var/lib/docker: 31%): RECOVERY disk space OK [21:51:49] 10Continuous-Integration-Infrastructure, 10Browser Test Platform, 10QS-Test-Automation, 07ci-test-error (WMF-deployed Build Failure), 13Patch-For-Review: Selenium timeouts can cause the job to remain stuck until the build times out - https://phabricator.wikimedia.org/T389536#11445701 (10vaughnwalters) [21:52:28] 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 10Browser Test Platform, and 4 others: CI jobs failing with various timeouts (March 2025) - https://phabricator.wikimedia.org/T388416#11445702 (10vaughnwalters)