[00:05:59] 10Release-Engineering-Team (Priority Backlog 📥), 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10621660 (10Jdlrobson-WMF) [01:51:49] I've manually stopped the jobs for https://gerrit.wikimedia.org/r/c/mediawiki/vendor/+/1126193 so that we don't wait another 30min before other stuff can start after the already 45min delay [01:52:39] Perhaps Zuul v3+ listens for CR-2 and/or CR+2 revocation and thus cancels the same way it does for e.g. a rebase on the patchset pipeline. [02:55:33] Hey - something crazy just happened on gerrit - a non-approved patch merged [02:56:17] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1126204 - it looks like I 'force merged it' - but that's above my paygrade to be able to force-push.... [03:05:55] OK - I can see that I did issue `git push` - but in the past that has always failed because I should have typed `git reivew` [03:54:15] !log civicrm upgraded from f2222fcd to ec20a105 [03:54:15] * config revision changed from 44fc7885 to de33fff3 [03:54:24] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [03:54:43] opps wrong channel [06:54:32] eileen: hi, you did send the patch for review and then 1H10 later went to do a git push of that same commit [06:55:00] since the commit was already known to Gerrit and you asked to push that commit on master branch, Gerrit accepted your push [06:55:46] and since it knew this commit is attached to that change, it wrote a message stating it got pushed [06:56:02] and closed the change marking it as "merged", since the push made the commit part of the branch [06:56:15] which is the normal behavior, but IS surprising [06:56:44] the verified-1 does not block the push, that is only blocking when asking to submit the change [06:57:01] but pushes are not subjects to a check of the scores/votes [06:58:14] what we can do is block pushes (in Gerrit access list terms: `DENY` the `Push` permission on `refs/heads/*` ) [06:59:11] the repo inherit access from wikimedia/fundraising which grants push to member of the fundraising group https://gerrit.wikimedia.org/r/admin/repos/wikimedia/fundraising,access [06:59:11] :) [06:59:33] that is it. Hopefully that clarify and the above few lines can serve as documentation for the future generations [07:36:21] 10Phabricator: Decrease scope of personal Herald rule H440 - https://phabricator.wikimedia.org/T388315#10622321 (10ABran-WMF) 05Open→03Resolved thanks for the hint, I changed the rule. Let me know if it's not enough! [07:47:52] 10Phabricator, 10Bitu, 06Infrastructure-Foundations, 10Striker: Inconsistent Wikimedia SUL Account Connections Across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498 (10Arendpieter) 03NEW [07:48:13] 10Phabricator, 10Bitu, 06Infrastructure-Foundations, 10Striker: Inconsistent Wikimedia SUL Account Connections Across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10622357 (10Arendpieter) [08:06:25] 10Phabricator, 10Bitu, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10622397 (10Arendpieter) [08:37:28] 10Gerrit, 10CAS-SSO, 06Infrastructure-Foundations: gerrit.wikimedia.org should use IDP for login - https://phabricator.wikimedia.org/T147864#10622444 (10hashar) [08:37:29] 10Gerrit, 10Release-Engineering-Team (Seen), 07Security, 07Upstream: Integrate MFA into Gerrit - https://phabricator.wikimedia.org/T198813#10622447 (10hashar) →14Duplicate dup:03T147864 [08:41:13] 10GitLab (Infrastructure), 06collaboration-services: SystemdUnitFailed (backup-restore.service on gitlab replicas) - https://phabricator.wikimedia.org/T388308#10622453 (10Jelto) [08:42:31] 10GitLab (Infrastructure), 06collaboration-services: SystemdUnitFailed (backup-restore.service on gitlab replicas) - https://phabricator.wikimedia.org/T388308#10622454 (10Jelto) 05Open→03Resolved Restore looks good again, I'll close the task. But I'll monitor the next `partial-backup` restore cycle and... [09:17:50] 10Phabricator, 10Bitu, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10622549 (10Peachey88) > Phabricator: Accessible at https://phabricator.wikimedia.org/. Users can aut... [09:42:48] 10Gerrit, 06collaboration-services: Investigate missing commit from old changes - https://phabricator.wikimedia.org/T388507 (10hashar) 03NEW [10:01:10] 10Gerrit, 06collaboration-services: Investigate missing commit from old changes - https://phabricator.wikimedia.org/T388507#10622705 (10hashar) [10:27:57] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10622828 (10Arendpieter) [10:44:58] (03open) 10ollieshotton: Revert to `buster-php81-fpm:1.0.1-s2` by default for mediawiki services [repos/releng/cli] - 10https://gitlab.wikimedia.org/repos/releng/cli/-/merge_requests/608 (https://phabricator.wikimedia.org/T388411) [10:50:25] (03merge) 10ollieshotton: Revert to `buster-php81-fpm:1.0.1-s2` by default for mediawiki services [repos/releng/cli] - 10https://gitlab.wikimedia.org/repos/releng/cli/-/merge_requests/608 (https://phabricator.wikimedia.org/T388411) [11:06:46] 10Gerrit, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations: gerrit.wikimedia.org should use IDP for login - https://phabricator.wikimedia.org/T147864#10622954 (10Arendpieter) [11:11:51] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 13Patch-For-Review: Phabricator should use IDP for developer account logins - https://phabricator.wikimedia.org/T377061#10622978 (10Arendpieter) [11:26:10] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623051 (10Aklapper) > Phabricator: Accessible at https://phabricator.wikimedia.org/. U... [11:58:30] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623156 (10Arendpieter) [11:58:52] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623158 (10Arendpieter) >>! In T388498#10623051, @Aklapper wrote: >> Phabricator: Acces... [12:55:28] 10Gerrit, 06collaboration-services: Investigate missing commit from old changes - https://phabricator.wikimedia.org/T388507#10623314 (10ABran-WMF) [13:08:30] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 2 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as parameter... - https://phabricator.wikimedia.org/T388526 [13:09:48] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 2 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as ... - https://phabricator.wikimedia.org/T388526#10623396 [13:12:07] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 2 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as ... - https://phabricator.wikimedia.org/T388526#10623409 [13:16:47] (03merge) 10jforrester: Stop branching ActiveAbstract for production, no longer used [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/152 (https://phabricator.wikimedia.org/T382069) [13:17:15] (03CR) 10Jforrester: [C:03+2] Zuul: [mediawiki/extensions/ActiveAbstract] Mark as archived [integration/config] - 10https://gerrit.wikimedia.org/r/1126085 (https://phabricator.wikimedia.org/T382069) (owner: 10Jforrester) [13:18:35] (03Merged) 10jenkins-bot: Zuul: [mediawiki/extensions/ActiveAbstract] Mark as archived [integration/config] - 10https://gerrit.wikimedia.org/r/1126085 (https://phabricator.wikimedia.org/T382069) (owner: 10Jforrester) [13:19:22] !log Zuul: [mediawiki/extensions/ActiveAbstract] Mark as archived, for T382069 [13:19:24] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [13:19:25] T382069: Undeploy and archive ActiveAbstract - https://phabricator.wikimedia.org/T382069 [13:23:28] 10Continuous-Integration-Config, 07Browser-Tests, 10Test-Platform (The Next One): Remove wdio-video-reporter from all repositories - https://phabricator.wikimedia.org/T294341#10623477 (10zeljkofilipin) [13:34:06] 10Continuous-Integration-Infrastructure, 10Castor, 10Testing Support: Running `cypress` in Wikimedia CI requires unusual env variables - https://phabricator.wikimedia.org/T361624#10623525 (10zeljkofilipin) [13:39:50] 10Continuous-Integration-Config, 10Testing Support, 07Browser-Tests, 10Test-Platform (The Next One): Remove wdio-video-reporter from all repositories - https://phabricator.wikimedia.org/T294341#10623641 (10zeljkofilipin) [13:40:00] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623643 (10Aklapper) I don't see how this //blocks// T189531. [13:41:15] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 2 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as ... - https://phabricator.wikimedia.org/T388526#10623664 [13:44:07] 10Beta-Cluster-Infrastructure, 10MediaWiki-Core-Tests, 10Testing Support, 07Browser-Tests: Run Selenium tests targeting Beta cluster - https://phabricator.wikimedia.org/T373680#10623683 (10zeljkofilipin) [13:48:20] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 2 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as ... - https://phabricator.wikimedia.org/T388526#10623707 [13:50:26] 10Release-Engineering-Team (Priority Backlog 📥), 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10623714 (10Lucas_Werkmeister_WMDE) [13:58:38] 10GitLab (Infrastructure), 10Ceph, 06collaboration-services, 10Data-Persistence-Backup, and 2 others: Migrate gitlab storage to apus (also: backups from S3?) - https://phabricator.wikimedia.org/T378922#10623770 (10Jelto) > So that sounds promising that Bacula should be able to backup straight from an S3 st... [14:07:41] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623802 (10fnegri) @Arendpieter thanks for creating this task, it's something that has... [14:31:23] (03merge) 10aklapper: Link to Wikimedia docs how to silence a bulk/batch edit [repos/phabricator/deployment] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/deployment/-/merge_requests/62 (https://phabricator.wikimedia.org/T386147) [14:31:56] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Doing 😎): Make it clearer/easier how to silence task batch edits - https://phabricator.wikimedia.org/T386147#10623908 (10Aklapper) [14:34:05] (03merge) 10aklapper: Avoid linking to wrong previous train task in train blocker tasks [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/48 (https://phabricator.wikimedia.org/T336686) [14:34:43] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions: Train blocker task has wrong navigation to previous train - https://phabricator.wikimedia.org/T336686#10623929 (10Aklapper) [14:35:05] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 3 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as ... - https://phabricator.wikimedia.org/T388526#10623934 [14:35:55] (03update) 10aklapper: Remove unused code in WMFSecurityPolicy.php [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/47 (https://phabricator.wikimedia.org/T385850) [14:36:37] (03merge) 10aklapper: Remove unused code in WMFSecurityPolicy.php [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/47 (https://phabricator.wikimedia.org/T385850) [14:37:26] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions, 07Technical-Debt: Dead code and non-existing classes mentioned in WMFSecurityPolicy.php - https://phabricator.wikimedia.org/T385850#10623946 (10Aklapper) [14:40:48] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623966 (10Aklapper) [14:40:57] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10623967 (10Aklapper) [14:51:09] 10Phabricator (phabricator-next), 06Release-Engineering-Team, 06collaboration-services: Deploy Phabricator/Phorge 2025-03-11 - https://phabricator.wikimedia.org/T388551 (10brennen) 03NEW [14:51:51] (03open) 10brennen: update submodules for 2025-03-11 deploy [repos/phabricator/deployment] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/deployment/-/merge_requests/63 (https://phabricator.wikimedia.org/T388551) [14:52:13] (03update) 10brennen: update submodules for 2025-03-11 deploy [repos/phabricator/deployment] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/deployment/-/merge_requests/63 (https://phabricator.wikimedia.org/T388551) [14:56:03] (03open) 10dancy: helm-check.py: Operate on all namespaces [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/438 [14:56:08] (03update) 10dancy: helm-check.py: Operate on all namespaces [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/438 [14:56:27] 06Release-Engineering-Team, 06collaboration-services: ProbeDown (gerrit1003) - https://phabricator.wikimedia.org/T388235#10624109 (10Jelto) 05Open→03Resolved I'll close the task, blocking of the IP ranges solved the issue. I also added some more documentation at https://wikitech.wikimedia.org/wiki/Gerr... [14:58:47] (03update) 10dancy: helm-check.py: Operate on all namespaces [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/438 [15:02:58] (03update) 10dancy: helm-check.py: Operate on all namespaces [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/438 [15:03:34] (03merge) 10dancy: helm-check.py: Operate on all namespaces [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/438 [15:10:24] (03open) 10dancy: helm-check.py: Operate on all namespaces (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/439 [15:10:27] (03update) 10dancy: helm-check.py: Operate on all namespaces (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/439 [15:10:59] (03merge) 10dancy: helm-check.py: Operate on all namespaces (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/439 [15:15:24] (03open) 10dancy: helm-check.py: Operate on all namespaces (followup2) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/440 [15:15:26] (03update) 10dancy: helm-check.py: Operate on all namespaces (followup2) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/440 [15:16:22] (03merge) 10dancy: helm-check.py: Operate on all namespaces (followup2) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/440 [15:26:52] 10Release-Engineering-Team (Priority Backlog 📥), 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10624297 (10Tgr) >>! In T386215#10616703, @Hokwelum wrote: > ##### Risky Patch! 🚂🔥 > > * **Change**: https://gerrit.wikimedia.org/r/c/mediawiki/cor... [15:30:11] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 3 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as... - https://phabricator.wikimedia.org/T388526#10624314 [15:33:43] 10Continuous-Integration-Config, 10Release-Engineering-Team (Seen), 10MW-on-K8s, 07Epic, 13Patch-For-Review: Have all Wikimedia production extensions and skins in the CI gate - https://phabricator.wikimedia.org/T249674#10624345 (10A_smart_kitten) [15:37:23] 10GitLab (Infrastructure), 10Ceph, 06collaboration-services, 10Data-Persistence-Backup, and 2 others: Migrate gitlab storage to apus (also: backups from S3?) - https://phabricator.wikimedia.org/T378922#10624358 (10jcrespo) >>! In T378922#10623769, @Jelto wrote: >> So that sounds promising that Bacula shoul... [15:42:58] Hi, I'm wondering if anyone knows how to fix: [15:43:01] https://www.irccloud.com/pastebin/CTCVgePc/ [15:45:45] paladox: Was https://www.mediawiki.org/wiki/Performance_budgeting unclear? [15:47:34] oh, I think I read it before but I'll give it ago again. [15:48:01] https://gerrit.wikimedia.org/r/c/mediawiki/extensions/SocialProfile/+/1119814 was what I tried before [15:48:40] Which repo are you dealing with that has this issue? [15:48:46] SocialProfile [15:49:08] Oh, but that patch didn't work? [15:49:44] yeh [15:55:02] > 15:53:09 tests/phpunit/BundleSizeTest.php:5 PhanUndeclaredExtendedClass Class extends undeclared class \MediaWiki\Tests\Structure\BundleSizeTestBase [15:55:03] hmm [15:55:37] https://gerrit.wikimedia.org/g/mediawiki/core/+/HEAD/tests/phpunit/structure/BundleSizeTest.php definitely exists. [15:56:00] And used e.g. in https://gerrit.wikimedia.org/g/mediawiki/extensions/ContentTranslation/+/HEAD/tests/phpunit/structure/BundleSizeTest.php [15:57:04] oh wait don't I have to add this file to the test autoloader? [15:57:35] You might, if your extension does that rather than auto-discovery. [16:02:40] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 3 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as... - https://phabricator.wikimedia.org/T388526#10624558 [16:04:31] it uses a mix of php and extension.json [16:04:42] 10Continuous-Integration-Config, 10MediaWiki-Core-Tests, 10Testing Support, 07Browser-Tests, and 2 others: Make MediaWiki Wdio tests less slow (Sept 2019) - https://phabricator.wikimedia.org/T234002#10624572 (10zeljkofilipin) [16:11:34] hmm [16:12:44] paladox: I am slightly enhancing that test message with https://gerrit.wikimedia.org/r/c/mediawiki/core/+/1126596 [16:12:49] moare newline and some emoji :) [16:12:59] :D [16:13:16] the structure tests are in mediawiki core [16:13:59] I see https://github.com/wikimedia/mediawiki-extensions-Acrolinx/tree/master does similar to https://gerrit.wikimedia.org/r/c/mediawiki/extensions/SocialProfile/+/1126590/ [16:14:12] but doesn't work for me [16:18:14] 10Release-Engineering-Team (Doing 😎), 07OKR-Work, 10Test-Platform (Radar): [WE6.2.6] Create design document for Pretrain (née Group -1) deployment - https://phabricator.wikimedia.org/T379683#10624724 (10SDunlap) [16:30:50] dunno how to fix it :( [16:37:44] (03open) 10dancy: monitoring: Bump kube-prometheus-stack to 69.8.2 [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/441 (https://phabricator.wikimedia.org/T363919) [16:37:46] (03update) 10dancy: monitoring: Bump kube-prometheus-stack to 69.8.2 [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/441 (https://phabricator.wikimedia.org/T363919) [16:38:54] (03merge) 10dancy: monitoring: Bump kube-prometheus-stack to 69.8.2 [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/441 (https://phabricator.wikimedia.org/T363919) [16:45:48] (03CR) 10Hashar: [C:03+1] Docker: [quibble-bullseye-php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125557 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [16:46:32] (03PS2) 10Scott French: Docker: [quibble-bullseye-php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125557 (https://phabricator.wikimedia.org/T386006) [16:46:48] (03PS3) 10Scott French: Docker: [php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125558 (https://phabricator.wikimedia.org/T386006) [16:47:20] (03CR) 10Hashar: "I have built it locally and that is good to go indeed. I will let James CR+1 :)" [integration/config] - 10https://gerrit.wikimedia.org/r/1125557 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [16:48:33] (03CR) 10Hashar: [C:03+1] Docker: [php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125558 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [16:49:54] 10Continuous-Integration-Infrastructure, 10Quibble: EarlyWarningBot comments are unreadable when the output includes ANSI colors - https://phabricator.wikimedia.org/T386773#10624903 (10matmarex) Thanks, seems to be working well now. I noticed some problems with the warnings I got today (on https://gerrit.wikim... [16:50:50] 10Release-Engineering-Team (Priority Backlog 📥), 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10624913 (10Jdlrobson-WMF) [16:52:56] James_F: appears it doesn't work no matter what I try. [16:53:19] tried adding the test to wgAutoloadClasses, doesn't work. [16:59:38] (03open) 10dancy: Add Grafana dashboard definition [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/92 (https://phabricator.wikimedia.org/T363919) [16:59:41] (03update) 10dancy: Add Grafana dashboard definition [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/92 (https://phabricator.wikimedia.org/T363919) [17:00:11] (03update) 10dancy: Add Grafana dashboard definition [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/92 (https://phabricator.wikimedia.org/T363919) [17:02:05] (03merge) 10dancy: Add Grafana dashboard definition [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/92 (https://phabricator.wikimedia.org/T363919) [17:03:14] (03open) 10dancy: Chart.yaml: bump version to 1.5.0 [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/93 [17:03:16] (03update) 10dancy: Chart.yaml: bump version to 1.5.0 [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/93 [17:04:47] (03merge) 10dancy: Chart.yaml: bump version to 1.5.0 [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/93 [17:18:21] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10625058 (10bd808) > Striker: Deployed at https://toolsadmin.wikimedia.org. Users can lo... [17:19:30] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10625093 (10bd808) [17:22:47] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625124 (10sbassett) [17:23:41] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625126 (10sbassett) [17:31:14] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625199 (10sbassett) [17:32:10] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625217 (10sbassett) 05In progress→03Resolved [17:35:09] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10625257 (10bd808) > Consistent and synchronized account connections across Phabricator,... [17:38:12] 10Phabricator (2025-03-11), 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions: Train blocker task has wrong navigation to previous train - https://phabricator.wikimedia.org/T336686#10625296 (10Aklapper) 05Open→03Resolved I believe that this shouldn't happen anymore thanks to rP... [17:38:43] 10Phabricator (2025-03-11), 10Release-Engineering-Team (Doing 😎): Make it clearer how to silence task batch edits - https://phabricator.wikimedia.org/T386147#10625301 (10Aklapper) 05Open→03Resolved [17:43:35] 10Phabricator, 10Release-Engineering-Team (Priority Backlog 📥): Unable to preview MP4 video in Phabricator task comments and descriptions - https://phabricator.wikimedia.org/T309222#10625342 (10Aklapper) Change deployed to add `video/mp4` to the "Global Default" but https://phabricator.wikimedia.org/config/edi... [18:01:30] 10Phabricator (2025-03-11), 06Release-Engineering-Team, 06collaboration-services: Deploy Phabricator/Phorge 2025-03-11 - https://phabricator.wikimedia.org/T388551#10625491 (10Aklapper) 05Open→03Resolved a:03brennen [18:01:57] (03PS1) 10Daimona Eaytoy: zuul: Add CommunityConfiguration as phan dependency of CampaignEvents [integration/config] - 10https://gerrit.wikimedia.org/r/1126616 (https://phabricator.wikimedia.org/T387966) [18:08:44] 06Project-Admins, 10Test-Platform (The First One): Create new "Testing Support" project for organizing test platform browser test work - https://phabricator.wikimedia.org/T383565#10625512 (10Aklapper) [18:11:53] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625524 (10Aklapper) All these project tags now violate https://www.mediawiki.org/wiki/Phabricator/Project_management#Typ... [18:13:19] 10Continuous-Integration-Infrastructure, 07Jenkins, 10Release-Engineering-Team (Priority Backlog 📥), 07ci-test-error, and 2 others: Various CI jobs failing after "mkdir: cannot create directory ‘log’: Permission denied" - https://phabricator.wikimedia.org/T282893#10625538 (10matmarex) I just got the error... [18:20:16] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 3 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as... - https://phabricator.wikimedia.org/T388526#10625636 [18:26:13] 10Phabricator (2025-03-11), 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions, 07Technical-Debt: Dead code and non-existing classes mentioned in WMFSecurityPolicy.php - https://phabricator.wikimedia.org/T385850#10625693 (10Aklapper) 05Open→03Resolved [18:26:20] 10Phabricator, 10Release-Engineering-Team (Priority Backlog 📥), 10Wikimedia-Phabricator-Extensions, 07Technical-Debt: Stop writing "std:maniphest:security_topic" in "maniphest_customfieldstorage" DB table - https://phabricator.wikimedia.org/T388590 (10Aklapper) 03NEW p:05Triage→03Low [18:31:10] 10Release-Engineering-Team (Priority Backlog 📥), 13Patch-For-Review, 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10625723 (10Pppery) >>! In T386215#10614636, @Pppery wrote: > Just a heads up: some time shortly after the group0 deploy I'm goi... [18:37:41] 10Release-Engineering-Team (Priority Backlog 📥), 10Less.php, 10MediaViewer, 06MediaWiki-Platform-Team, and 3 others: MediaViewer broken in wmf.20 due to LESS change: Less_Exception_Compiler: error evaluating function `floor` math functions take numbers as... - https://phabricator.wikimedia.org/T388526#10625759 [18:45:10] 06Project-Admins, 06Security-Team, 10Vulnerability Management, 07SecTeam-Processed, 07Security: Proposed Security Tags for Phabricator - https://phabricator.wikimedia.org/T387508#10625781 (10sbassett) >>! In T387508#10625524, @Aklapper wrote: > All these project tags now violate https://www.mediawiki... [18:55:41] 10Release-Engineering-Team (Priority Backlog 📥), 05Release, 05Train Deployments: 1.44.0-wmf.20 deployment blockers - https://phabricator.wikimedia.org/T386215#10625862 (10thcipriani) >>! In T386215#10625723, @Pppery wrote: >>>! In T386215#10614636, @Pppery wrote: >> Just a heads up: some time shortly after t... [18:59:29] (03open) 10dancy: monitoring: Set Grafana default dashboard to Reggie [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/442 (https://phabricator.wikimedia.org/T363919) [18:59:30] (03update) 10dancy: monitoring: Set Grafana default dashboard to Reggie [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/442 (https://phabricator.wikimedia.org/T363919) [19:00:24] (03merge) 10dancy: monitoring: Set Grafana default dashboard to Reggie [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/442 (https://phabricator.wikimedia.org/T363919) [19:15:03] (03open) 10dancy: monitoring: Set Grafana default dashboard to Reggie (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/443 (https://phabricator.wikimedia.org/T363919) [19:15:05] (03update) 10dancy: monitoring: Set Grafana default dashboard to Reggie (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/443 (https://phabricator.wikimedia.org/T363919) [19:16:07] (03merge) 10dancy: monitoring: Set Grafana default dashboard to Reggie (followup) [repos/releng/gitlab-cloud-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-cloud-runner/-/merge_requests/443 (https://phabricator.wikimedia.org/T363919) [19:27:23] (03open) 10dancy: Chart 1.5.1: Fix typo in dashboard.yaml [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/94 [19:27:26] (03update) 10dancy: Chart 1.5.1: Fix typo in dashboard.yaml [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/94 [19:28:16] (03merge) 10dancy: Chart 1.5.1: Fix typo in dashboard.yaml [repos/releng/reggie] - 10https://gitlab.wikimedia.org/repos/releng/reggie/-/merge_requests/94 [20:10:10] (03Abandoned) 10Subramanya Sastry: Use CSS to style selflink-fragments in Parsoid HTML to match legacy [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/1112292 (https://phabricator.wikimedia.org/T384060) (owner: 10Subramanya Sastry) [20:27:38] hashar: yes - that patch had already been put up to gerrit & then locally I had another patch on top of that which was also accepted into git - I was sure we had push set to deny because I can't believe this is the first time I have accidentally run `git push` instead of `git review` cos I switch constantly between github & gerrit [20:30:04] (03open) 10aklapper: Include login.wm.o and auth.wm.o in OAuth CSP rule [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/49 (https://phabricator.wikimedia.org/T376803) [20:30:06] 10Phabricator, 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions, 07Browser-Support-Google-Chrome, 13Patch-For-Review: Phab login via SUL works only on second time with Chrome (due to CSP and redirect) - https://phabricator.wikimedia.org/T376803#10626168 (10Aklapper) a:03Aklapper [20:34:15] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10626198 (10Arendpieter) @bd808, thank you for the explanation! A possible outcome of th... [20:45:35] (03CR) 10Jforrester: [C:03+2] "Yup, works great, thanks!" [integration/config] - 10https://gerrit.wikimedia.org/r/1125557 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [20:46:25] (03CR) 10Jforrester: [C:03+2] Docker: [php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125558 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [20:47:16] (03Merged) 10jenkins-bot: Docker: [quibble-bullseye-php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125557 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [20:48:15] (03Merged) 10jenkins-bot: Docker: [php81] Use PCRE2 backport from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1125558 (https://phabricator.wikimedia.org/T386006) (owner: 10Scott French) [20:48:54] !log Docker: [quibble-bullseye-php81 & php81] Use PCRE2 backport from component/php81, for T386006 [20:48:56] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [20:48:56] T386006: Update PCRE in PHP 8.1 images to PCRE 10.39 or newer - https://phabricator.wikimedia.org/T386006 [20:58:55] (03PS1) 10Jforrester: jjb: Update PHP 8.1-based images to PCRE2 from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1126657 (https://phabricator.wikimedia.org/T386006) [21:01:00] (03CR) 10Hashar: [C:03+1] "👌" [integration/config] - 10https://gerrit.wikimedia.org/r/1126657 (https://phabricator.wikimedia.org/T386006) (owner: 10Jforrester) [21:01:07] 10Phabricator, 10Bitu, 10CAS-SSO, 06Infrastructure-Foundations, 10Striker: Inconsistent mapping of Developer accounts and SUL accounts across Phabricator, Bitu, and Striker - https://phabricator.wikimedia.org/T388498#10626311 (10bd808) >>! In T388498#10626198, @Arendpieter wrote: > @bd808, thank you for... [21:01:32] James_F: thank you for the updates! [21:02:09] 10Phabricator, 10Release-Engineering-Team (Doing 😎), 10Wikimedia-Phabricator-Extensions, 07Technical-Debt: SecurityPolicyEnforcerAction: !empty($forced_policies) is always falsy - https://phabricator.wikimedia.org/T385872#10626312 (10Aklapper) a:03Aklapper [21:02:18] (03open) 10aklapper: Remove unused code for $forced_policies [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/50 (https://phabricator.wikimedia.org/T385872) [21:03:00] hashar: It's a delight. [21:03:04] eileen: I have the same issue when switching to Gitlab/Github :b If it can help, we can surely deny push to the branches (`refs/heads/*`) [21:03:12] (03CR) 10Jforrester: [C:03+2] jjb: Update PHP 8.1-based images to PCRE2 from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1126657 (https://phabricator.wikimedia.org/T386006) (owner: 10Jforrester) [21:04:11] 10Beta-Cluster-Infrastructure, 10Cloud-VPS (Debian Buster Deprecation): Replace or remove deployment-echostore02.deployment-prep.eqiad1.wikimedia.cloud - https://phabricator.wikimedia.org/T361383#10626329 (10Eevans) 05Open→03Resolved I recreated the VM (using the old name). `lang=sh-session $ curl -D... [21:04:19] hashar: thanks - I think ejegg is changing that? Or maybe he doesn't have permission ? I'm just confused that I was sure pushes were already blocked [21:04:25] eileen: I briefly dreamed of setting git aliases to keep `git review` and `git push` from doing the wrong thing in the wrong style of repo, but sadly "aliases that hide existing git commands are ignored" per https://www.kernel.org/pub/software/scm/git/docs/git-config.html [21:04:27] (03Merged) 10jenkins-bot: jjb: Update PHP 8.1-based images to PCRE2 from component/php81 [integration/config] - 10https://gerrit.wikimedia.org/r/1126657 (https://phabricator.wikimedia.org/T386006) (owner: 10Jforrester) [21:04:38] :-) [21:05:15] to be fair when you do a `git push origin HEAD:refs/heads/master` Gerrit should act as if you have used a git-review (or a push to the magic refs/for/master [21:05:25] bd808: Just use `gitrev` as a local script then. [21:05:26] and frankly that will make life easier if the default is to create or update changes [21:06:08] but instead to accommodate for the non obvious `refs/for/` we have to resort to `git-review` tooling [21:07:05] and whatever repo you had the issue with inherit from `wikimedia/fundraising`. It grants the `Push` permission to `refs/*` to members of the `fundraising` group (ref: https://gerrit.wikimedia.org/r/admin/repos/wikimedia/fundraising,access ) [21:07:21] so essentially you can always push [21:07:34] so I guess yes use git-review :] [21:07:44] James_F: if brains are being reprogrammed the problem is moot :) [21:08:10] I think Gerrit upstream had a RFC/spec to change the push behavio [21:08:11] r [21:08:39] Fair. [21:08:42] so that if you really wanted to do a direct push you would have to do something like `git push origin master:refs/direct/master` [21:08:45] something like that [21:09:03] or maybe that was someone ranting about Gerrit silliness on the upstream mailing list [21:09:32] hey maybe it is not that hard to implement and maybe it can be a per user configuration setting [21:09:47] I personally use `git r` as my alias for `git review --verbose --no-rebase`. I haven't yet found myself mixed up about `git r` vs `git push` based on the upstream code forge, but its probably just a matter of time. [21:10:16] Default push behavior: ( ) fast-forward branch (X) create/update change(s) [21:14:55] (03open) 10aklapper: Draft: Stop setting std:maniphest:security_topic [repos/phabricator/extensions] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/extensions/-/merge_requests/51 (https://phabricator.wikimedia.org/T388590) [21:15:28] I feel I am not alone struggling with this :-) [21:15:44] yup :) [21:15:50] YOU ARE NOT ALONE! :b [21:16:03] I am personally using `git-review` [21:16:25] and I get entirely confused whenever I contribute to Github/Gitlab, but since I use them less, it is less often a problem =) [21:17:08] (for them I resort to use explicit references: `git push origin master:refs/heads/whatever-funny-name` ) [21:17:37] and I get lost when I have to later amend a commit but that is another story. I gues sI should learn to create a branch locally [21:20:46] * hashar sleeps [21:22:38] 10Beta-Cluster-Infrastructure, 10Cloud-VPS (Debian Buster Deprecation): Replace or remove deployment-echostore02.deployment-prep.eqiad1.wikimedia.cloud - https://phabricator.wikimedia.org/T361383#10626463 (10matmarex) Thanks! [21:23:19] ah here is the email https://groups.google.com/g/repo-discuss/c/LIFavLUiTfs/m/wPaeJjawAAAJ [21:23:54] which argues to reverse the behavior of push [21:30:46] (03CR) 10Hashar: [C:03+2] zuul: Add CommunityConfiguration as phan dependency of CampaignEvents [integration/config] - 10https://gerrit.wikimedia.org/r/1126616 (https://phabricator.wikimedia.org/T387966) (owner: 10Daimona Eaytoy) [21:31:59] (03Merged) 10jenkins-bot: zuul: Add CommunityConfiguration as phan dependency of CampaignEvents [integration/config] - 10https://gerrit.wikimedia.org/r/1126616 (https://phabricator.wikimedia.org/T387966) (owner: 10Daimona Eaytoy) [21:34:21] <-- sleep [21:34:42] Daimona: I have deployed your CI config change for CampaignEvents [21:34:54] ZzzZz [21:35:17] (03PS1) 10Ebernhardson: Add debian-glue CI for opensearch plugins repo [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) [21:37:11] (03CR) 10Hashar: [C:03+2] Add debian-glue CI for opensearch plugins repo [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) (owner: 10Ebernhardson) [21:37:23] I lied, but I cant resist ebernhardson changes :b [21:39:09] (03CR) 10Ryan Kemper: [C:03+1] Add debian-glue CI for opensearch plugins repo [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) (owner: 10Ebernhardson) [21:39:38] (03Merged) 10jenkins-bot: Add debian-glue CI for opensearch plugins repo [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) (owner: 10Ebernhardson) [21:40:57] (03CR) 10Hashar: [C:03+2] "Deployed! Note the job only triggers for changes touching a file under the `debian/` directory." [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) (owner: 10Ebernhardson) [21:42:16] !log Deleted unused "deployment-parsoid" Prefix Puppet configuration [21:42:17] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:50:44] !log Deleted dangling w-beta.wmflabs.org web proxy [21:50:44] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [21:51:52] Thank you! [21:53:20] !log Deleted dangling prometheus-beta.wmcloud.org web proxy [21:53:21] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [22:04:29] !log Deleted orphan parsoid-external-ci-access.beta.wmflabs.org. DNS record [22:04:29] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [22:08:15] !log Updated mail.beta.wmflabs.org service name to point to 185.15.56.115 [22:08:16] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [22:12:51] (03CR) 10Ebernhardson: "Thanks Antoine!" [integration/config] - 10https://gerrit.wikimedia.org/r/1126665 (https://phabricator.wikimedia.org/T388611) (owner: 10Ebernhardson) [22:28:14] 10Continuous-Integration-Infrastructure, 06Diffusion-Repository-Administrators, 10Projects-Cleanup, 06translatewiki.net, and 2 others: Archive the DoubleWiki extension - https://phabricator.wikimedia.org/T344544#10626700 (10Pppery) Adding #continuous-integration-infrastructure as that's the last step that... [22:37:15] 10Continuous-Integration-Infrastructure, 06Diffusion-Repository-Administrators, 10MediaWiki-extensions-CollaborationKit, 10Projects-Cleanup, and 2 others: Archive the CollaborationKit extension - https://phabricator.wikimedia.org/T368092#10626729 (10MGChecker) Adding #continuous-integration-infrastructure... [22:38:19] 10Continuous-Integration-Infrastructure, 06Diffusion-Repository-Administrators, 10MediaWiki-extensions-CollaborationKit, 10Projects-Cleanup, and 2 others: Archive the CollaborationKit extension - https://phabricator.wikimedia.org/T368092#10626732 (10Pppery) [22:39:57] !log Released unused floating IPs 185.15.56.9 and 185.15.56.97 back to global pool [22:39:58] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [22:40:16] 10Continuous-Integration-Infrastructure, 06Diffusion-Repository-Administrators, 10MediaWiki-extensions-CollaborationKit, 10Projects-Cleanup, and 2 others: Archive the CollaborationKit extension - https://phabricator.wikimedia.org/T368092#10626736 (10Pppery) (GitHub also needs to be done) [22:42:19] oh https://github.com/wikimedia/mediawiki/blob/master/tests/phpunit/structure/PerformanceBudgetTest.php#L122 presumes you are setting resource modules via extension.json. [22:42:28] I think if I'm reading it correctly James_F ^? [22:46:20] @seen paladox [22:46:32] hi mutante [22:46:47] * mutante waves :) [22:49:43] paladox: you should have waved you hand in front of your face and said "You Can't See Me" ;) [22:49:56] heh [22:54:05] ;) [22:54:17] !log Deleted unattached volumes: alert01, db09, deploy03, mwmaint, ores02, parsoid14-srv, prometheus05 [22:54:18] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [23:00:07] is there a way to make it overridable. I guess similar to BundleSizeTestBase? I guess making the function public will do it? [23:02:54] 10Continuous-Integration-Infrastructure, 06Diffusion-Repository-Administrators, 10MediaWiki-extensions-CollaborationKit, 10Projects-Cleanup, and 2 others: Archive the CollaborationKit extension - https://phabricator.wikimedia.org/T368092#10626758 (10A_smart_kitten) (Also Phabricator? Unless I'm misundersta...