[01:37:49] 10Phabricator-Bot-Requests, 06Fundraising-Backlog: Phabricator Bot Request: Fundraising Tech Bug Reporter - https://phabricator.wikimedia.org/T426459#11947159 (10AKanji-WMF) a:03Aklapper Following up on this, I see {T425118} was assigned to @Aklapper - I'll do the same here, but please let me know if there'... [07:22:01] 10GitLab (CI & Job Runners), 06Release-Engineering-Team, 06cloud-services-team, 06collaboration-services, 13Patch-For-Review: gitlab workers ulimit nofiles 1073741816 slows down fakeroot - https://phabricator.wikimedia.org/T426827#11947353 (10fgiunchedi) [07:59:53] (03CR) 10Hashar: [C:03+2] Add BetaFeatures to phan dependencies for MMV [integration/config] - 10https://gerrit.wikimedia.org/r/1290944 (https://phabricator.wikimedia.org/T426225) (owner: 10Jdlrobson) [08:02:23] (03Merged) 10jenkins-bot: Add BetaFeatures to phan dependencies for MMV [integration/config] - 10https://gerrit.wikimedia.org/r/1290944 (https://phabricator.wikimedia.org/T426225) (owner: 10Jdlrobson) [08:23:40] (03open) 10kbach: Update OAD example [repos/ci-tools/wikimedia-spectral-ruleset] - 10https://gitlab.wikimedia.org/repos/ci-tools/wikimedia-spectral-ruleset/-/merge_requests/3 [09:17:33] 10Phabricator (Upstream), 07Upstream: Headers for some Phab notification emails reference a Herald rule that is currently disabled - https://phabricator.wikimedia.org/T420162#11947487 (10A_smart_kitten) Still seems to be occasionally happening, FWIW: e.g. the emails I got about the @gerritbot actions at T41915... [09:29:26] 10GitLab (Project Migration), 06collaboration-services: Migrate SRE repositories to GitLab - operations/software - https://phabricator.wikimedia.org/T341504#11947531 (10jcrespo) [11:02:24] (03open) 10trueg: wdqs-backend-health-monitor: Docker image to monitor the status of the WDQS deployment pod. [repos/releng/gitlab-trusted-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/merge_requests/173 (https://phabricator.wikimedia.org/T426748) [12:55:55] 10GitLab, 06Release-Engineering-Team (Radar), 06collaboration-services, 13Patch-For-Review: gitlab behind CDN - https://phabricator.wikimedia.org/T425441#11948229 (10ABran-WMF) [13:59:56] (03CR) 10Hashar: [C:03+2] "@fomafix@googlemail.com I apologize for the delay, I had a busy week. I will recheck the mentioned patch once this is merged and deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/1287380 (owner: 10Fomafix) [14:00:03] (03CR) 10Hashar: [C:03+2] "@fomafix@googlemail.com I apologize for the delay, I had a busy week. I will recheck the mentioned patch once this is merged and deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/1287838 (owner: 10Fomafix) [14:01:48] (03Merged) 10jenkins-bot: Zuul: [mediawiki/extensions/Survey] Add AdminLinks Phan dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1287380 (owner: 10Fomafix) [14:02:00] (03Merged) 10jenkins-bot: Zuul: [mediawiki/extensions/WSOAuth] Add PluggableAuth Phan dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1287838 (owner: 10Fomafix) [14:17:29] 10Continuous-Integration-Config, 06Release-Engineering-Team (Priority Backlog 📥), 10phan: Phan job should use composer instead of vendor for release branches ( undeclared class \Wikimedia\Equivset\Equivset ) - https://phabricator.wikimedia.org/T231966#11948484 (10hashar) From https://gerrit.wikimedia.org/r/c... [14:22:19] hashar: I still think we should just move the phan work into the regular `composer test` definition. That way it gets vendor/composer for free. [14:22:31] And we can get it working for release branches etc. [14:23:37] The downsides are (a) composer test is now a bit slower, but still the fastest job in CI, (b) we'll need to check the images all have php-ast installed (I think they do now), and (c) devs might be surprised. [14:25:56] (03PS1) 10Stoyofuku-wmf: [WikimediaCustomizations] Add a soft dependency on GlobalPreferences [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) [14:30:00] (03CR) 10Stoyofuku-wmf: "@kfilbert@wikimedia.org I believe this is necessary in order to get the tests passing (and hopefully fix the code coverage report 🤞), do y" [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [14:32:51] 10Phabricator-Bot-Requests, 06Fundraising-Backlog: Phabricator Bot Request: Fundraising Tech Bug Reporter - https://phabricator.wikimedia.org/T426459#11948520 (10Aklapper) Hi, sorry, this task fell between the cracks somehow. :( Please see https://www.mediawiki.org/wiki/Phabricator/Bots#Acquiring_a_bot for req... [15:00:30] (03merge) 10dancy: wdqs-backend-health-monitor: Docker image to monitor the status of the WDQS deployment pod. [repos/releng/gitlab-trusted-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/merge_requests/173 (https://phabricator.wikimedia.org/T426748) (owner: 10trueg) [15:00:40] James_F: sounds reasonable! [15:01:01] I think addshore introduced Phan as a standalone image because of the php-ast requirement [15:01:26] and possibly it took sometime [15:01:44] the devil is should we bother running phan under each of the php versions we support? [15:02:08] My understanding is we only need to run a single Phan with whatever php version it requires, and it has config to states which php versions are targetted [15:02:37] Aka run Phan with say php 8.3 and tell it code targets 8.2 to 8.5 [15:38:52] (03CR) 10Aude: "The change looks good, however (and sorry for the nitpick) there is a convention for this git repo to format commit message a certain way:" [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [15:52:38] (03CR) 10Stoyofuku-wmf: "No need to apologize! Thanks for letting me know (:" [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [15:54:48] (03PS2) 10Stoyofuku-wmf: Zuul: [mediawiki/extensions/WikimediaCustomizations] Add GlobalPreferences dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) [15:56:23] (03CR) 10CI reject: [V:04-1] Zuul: [mediawiki/extensions/WikimediaCustomizations] Add GlobalPreferences dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [15:58:41] (03PS3) 10Stoyofuku-wmf: Zuul: [mediawiki/extensions/WikimediaCustomizations] Add GP dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) [16:00:37] (03CR) 10Aude: "*sigh*" [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [16:01:15] (03CR) 10Aude: [C:03+1] Zuul: [mediawiki/extensions/WikimediaCustomizations] Add GP dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292006 (https://phabricator.wikimedia.org/T426322) (owner: 10Stoyofuku-wmf) [16:56:01] (03PS1) 10Harroyo-wmf: Include GlobalBlocking as a dependency for ConfirmEdit [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) [16:57:06] (03PS2) 10Harroyo-wmf: Include GlobalBlocking as a dependency for ConfirmEdit [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) [16:57:50] (03PS3) 10Harroyo-wmf: Include GlobalBlocking as a dependency for ConfirmEdit [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) [17:06:43] (03CR) 10Dreamy Jazz: [C:04-1] "This doesn't add the dependency to ConfirmEdit, but instead adds it to VisualEditor" [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [17:07:16] (03CR) 10Dreamy Jazz: [C:04-1] "I say this because the commit summary says "Include GlobalBlocking as a dependency for ConfirmEdit"" [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [17:07:58] (03CR) 10Dreamy Jazz: [C:04-1] Include GlobalBlocking as a dependency for ConfirmEdit (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [17:11:31] (03CR) 10Harroyo-wmf: "true, copypaste error, I'll fix" [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [17:12:00] (03PS4) 10Harroyo-wmf: Include GlobalBlocking as a dependency for VisualEditor [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) [17:14:19] (03PS5) 10Harroyo-wmf: zuul: [mediawiki/extensions/VisualEditor] Add GlobalBlocking dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) [17:14:34] (03CR) 10Harroyo-wmf: [C:03+1] zuul: [mediawiki/extensions/Wikibase] Add ConfirmEdit dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1289940 (https://phabricator.wikimedia.org/T426089) (owner: 10Kosta Harlan) [17:15:05] (03CR) 10Harroyo-wmf: "Done" [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [17:17:12] (03CR) 10Dreamy Jazz: zuul: [mediawiki/extensions/VisualEditor] Add GlobalBlocking dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1292060 (https://phabricator.wikimedia.org/T427086) (owner: 10Harroyo-wmf) [18:19:43] 10GitLab (Account Approval), 06Release-Engineering-Team: Requesting GitLab account activation for [YOUR DEVELOPER ACCOUNT USERNAME HERE] - https://phabricator.wikimedia.org/T427089 (10HAhmed-WMF) 03NEW [18:19:59] 10GitLab (Account Approval), 06Release-Engineering-Team: Requesting GitLab account activation for [@hahmed] - https://phabricator.wikimedia.org/T427089#11949358 (10HAhmed-WMF) [19:08:35] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 06collaboration-services, 13Patch-For-Review: setup 2 contint machines for jenkins - https://phabricator.wikimedia.org/T418521#11949450 (10Dzahn) 05Stalled→03Open The plan to switch over: - announce maintenance time slot (deployment... [19:09:14] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 06collaboration-services, 13Patch-For-Review: setup 2 contint machines for jenkins - https://phabricator.wikimedia.org/T418521#11949453 (10Dzahn) [20:00:49] 06Release-Engineering-Team, 06collaboration-services: Low priority: extra spaces in gitlab servers' known_hosts entries are confounding paramiko - https://phabricator.wikimedia.org/T427094 (10bking) 03NEW [20:01:33] 06Release-Engineering-Team, 06collaboration-services: Low priority: extra spaces in gitlab servers' known_hosts entries are confounding paramiko - https://phabricator.wikimedia.org/T427094#11949513 (10bking)