[03:36:42] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [03:46:31] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [06:20:58] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [06:36:43] (03PS2) 10Abijeet Patro: Translate: Add dependency on Scribunto [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) [06:36:55] (03CR) 10Abijeet Patro: "Yes, please! Thanks" [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) (owner: 10Abijeet Patro) [06:37:16] (03PS3) 10Abijeet Patro: Translate: Add dependency on Scribunto [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) [07:29:15] (03open) 10kbach: Update allowed_projects [repos/releng/docpub] - 10https://gitlab.wikimedia.org/repos/releng/docpub/-/merge_requests/22 [08:02:09] 10Gerrit, 10Tools: Gerrit reviewer bot should have an option to add people as CC instead of reviewers - https://phabricator.wikimedia.org/T363290#9793349 (10hashar) Thus it looks like we can mark this as a duplicate of T334118? The resolution would be to decline it since the reviewer bot is in maintenance mode. [08:05:16] (03CR) 10Hashar: [C:03+2] Translate: Add dependency on Scribunto [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) (owner: 10Abijeet Patro) [08:06:42] (03Merged) 10jenkins-bot: Translate: Add dependency on Scribunto [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) (owner: 10Abijeet Patro) [08:08:07] (03CR) 10Hashar: [C:03+2] Keep UserMerge phan dependencies for production-deployed extensions [integration/config] - 10https://gerrit.wikimedia.org/r/1030979 (https://phabricator.wikimedia.org/T364610) (owner: 10Umherirrender) [08:09:05] !log Reloaded Zuul for https://gerrit.wikimedia.org/r/1025313 [08:09:07] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [08:09:26] (03Merged) 10jenkins-bot: Keep UserMerge phan dependencies for production-deployed extensions [integration/config] - 10https://gerrit.wikimedia.org/r/1030979 (https://phabricator.wikimedia.org/T364610) (owner: 10Umherirrender) [08:09:31] (03PS1) 10Majavah: zuul: [mediawiki/extensions/WikiLove] Add back UserMerge Phan dep [integration/config] - 10https://gerrit.wikimedia.org/r/1031386 [08:09:58] (03Abandoned) 10Majavah: zuul: [mediawiki/extensions/WikiLove] Add back UserMerge Phan dep [integration/config] - 10https://gerrit.wikimedia.org/r/1031386 (owner: 10Majavah) [08:10:00] (03CR) 10Hashar: [C:03+2] "I have deployed the change. If Phan static analysis fails, you will have to add the dependency further below in the same file in the `phan" [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) (owner: 10Abijeet Patro) [08:10:33] (03CR) 10Hashar: [C:03+2] "Deployed!" [integration/config] - 10https://gerrit.wikimedia.org/r/1030979 (https://phabricator.wikimedia.org/T364610) (owner: 10Umherirrender) [08:11:35] (03CR) 10Hashar: [C:03+2] zuul: Add Theknightwho to the CI allowlist [integration/config] - 10https://gerrit.wikimedia.org/r/1030907 (owner: 10Novem Linguae) [08:13:12] (03Merged) 10jenkins-bot: zuul: Add Theknightwho to the CI allowlist [integration/config] - 10https://gerrit.wikimedia.org/r/1030907 (owner: 10Novem Linguae) [08:13:34] 10Phabricator: Phabricator should reuse confirmed MediaWiki email during registration - https://phabricator.wikimedia.org/T230590#9793382 (10Aklapper) @Tgr: Thanks for that info. Without it I would not have given this a try! <3 I tried locally (still had a MediaWiki 1.41.0-alpha (`bb40716`) from 18 July 2023 in... [08:14:03] (03CR) 10Hashar: [C:03+2] "I have deployed the change. CI will not trigger when the user send a patchset or when they comment `recheck` on an existing patchset." [integration/config] - 10https://gerrit.wikimedia.org/r/1030907 (owner: 10Novem Linguae) [08:20:28] (03merge) 10aklapper: Make labels readable in report bars (contrast) [repos/phabricator/phabricator] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/phabricator/-/merge_requests/49 (https://phabricator.wikimedia.org/T301024) [08:20:35] 10Phabricator, 10Release-Engineering-Team (Yakisfaction), 07CSS, 13Patch-For-Review: Label cut-off in Phabricator report tables - https://phabricator.wikimedia.org/T301024#9793400 (10CodeReviewBot) aklapper merged https://gitlab.wikimedia.org/repos/phabricator/phabricator/-/merge_requests/49 Make labels r... [08:21:02] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 07CSS: Label cut-off in Phabricator report tables - https://phabricator.wikimedia.org/T301024#9793401 (10Aklapper) [08:29:25] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Configure Docker builder GC settings for CI - https://phabricator.wikimedia.org/T364773#9793431 (10hashar) [08:30:42] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 06Machine-Learning-Team: Python torch fills disk of CI Jenkins instances - https://phabricator.wikimedia.org/T338317#9793440 (10hashar) [08:30:45] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Configure Docker builder GC settings for CI - https://phabricator.wikimedia.org/T364773#9793441 (10hashar) [08:32:38] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 06Machine-Learning-Team: Python torch fills disk of CI Jenkins instances - https://phabricator.wikimedia.org/T338317#9793457 (10hashar) >>! In T338317#9623973, @dancy wrote: > We could configure buildkit gc rules for the Docker daemon: http... [08:33:58] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Configure Docker builder GC settings for CI - https://phabricator.wikimedia.org/T364773#9793466 (10hashar) [08:36:07] (03CR) 10Hashar: [C:03+2] Use new "bluespice" template on other extensions [integration/config] - 10https://gerrit.wikimedia.org/r/1029119 (owner: 10Robert Vogel) [08:37:32] (03Merged) 10jenkins-bot: Use new "bluespice" template on other extensions [integration/config] - 10https://gerrit.wikimedia.org/r/1029119 (owner: 10Robert Vogel) [08:37:59] (03CR) 10Hashar: "It is great to see James Forrester has refined CI for Bluespice ( fc444e207874aa4ff486964146a23b3ba429dda2 ) :)" [integration/config] - 10https://gerrit.wikimedia.org/r/1029119 (owner: 10Robert Vogel) [09:38:39] (03PS2) 10Jack Phoenix: zuul: [mediawiki/extensions/MediaWikiChat] Add phan dependencies for MWC [integration/config] - 10https://gerrit.wikimedia.org/r/1031173 [09:47:01] 10Release-Engineering-Team (Yakisfaction), 05Release, 05Train Deployments: 1.43.0-wmf.5 deployment blockers - https://phabricator.wikimedia.org/T361399#9793686 (10hashar) [09:50:43] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Gerrit ssh host key fails on fedora39 - https://phabricator.wikimedia.org/T364217#9793714 (10Tarrow) >>! In T364217#9781787, @Dzahn wrote: > @Tarrow You could try if it works with `ssh -o RequiredRSASize=1024` just to debug or until we can f... [09:55:47] 10Release-Engineering-Team (Yakisfaction), 13Patch-For-Review, 05Release, 05Train Deployments: 1.43.0-wmf.5 deployment blockers - https://phabricator.wikimedia.org/T361399#9793751 (10Clement_Goubert) [09:58:23] 10Phabricator: Phabricator should reuse confirmed MediaWiki email during registration - https://phabricator.wikimedia.org/T230590#9793757 (10Tgr) You need to confirm the email address (via the button at the bottom of Special:Preferences, or the email you get right after signup, or by using [[https://www.mediawik... [10:19:42] 10Release-Engineering-Team (Yakisfaction), 13Patch-For-Review, 05Release, 05Train Deployments: 1.43.0-wmf.5 deployment blockers - https://phabricator.wikimedia.org/T361399#9793848 (10hashar) @Aklapper and I paired the deployment and quickly noticed some database issues showing up in the error logs and orig... [11:22:06] 10Phabricator, 07Technical-Debt: Investigate / remove custom downstream changes to PhabricatorClientRateLimit.php - https://phabricator.wikimedia.org/T364839 (10Aklapper) 03NEW p:05Triage→03Low [11:30:49] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: gerrit 3.10 will require Java 17+ - https://phabricator.wikimedia.org/T364342#9794066 (10hashar) [11:31:25] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Gerrit 3.10 will require Java 17 - https://phabricator.wikimedia.org/T364342#9794070 (10hashar) [11:33:54] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Gerrit 3.10 will require Java 17 - https://phabricator.wikimedia.org/T364342#9794075 (10hashar) I rewrote the task description to remove mention of upgrading Gerrit and the OS and have the task to focus solely on switching from Java 11 to Java 17. [11:35:11] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Switch Gerrit from Java 11 to Java 17 - https://phabricator.wikimedia.org/T364342#9794086 (10hashar) [11:50:14] 10MediaWiki-Releasing, 05MW-1.42-release: Write release announcement for 1.42.0-rc.0 - https://phabricator.wikimedia.org/T359843#9794145 (10Reedy) [11:56:21] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Switch Gerrit from Java 11 to Java 17 - https://phabricator.wikimedia.org/T364342#9794168 (10hashar) [11:58:40] (03CR) 10Abijeet Patro: "Thanks, I'll check that out." [integration/config] - 10https://gerrit.wikimedia.org/r/1025313 (https://phabricator.wikimedia.org/T359918) (owner: 10Abijeet Patro) [12:06:45] (03PS3) 10Abijeet Patro: Translate: Add phan dependency on Scribunto [integration/config] - 10https://gerrit.wikimedia.org/r/1031178 (https://phabricator.wikimedia.org/T359918) [12:06:59] 10Gerrit: Gerrit email subject begins with “size bucket” ([XS], [S], [M] [L], [XL]) since 6 May 2024 - https://phabricator.wikimedia.org/T364484#9794191 (10hashar) 05Open→03Resolved a:03hashar Fixed by reverting the change [12:22:24] 10Gerrit: Gerrit email subject begins with “size bucket” ([XS], [S], [M] [L], [XL]) since 6 May 2024 - https://phabricator.wikimedia.org/T364484#9794242 (10Lucas_Werkmeister_WMDE) Alright, thanks! [12:29:34] 10Phabricator: Phabricator should reuse confirmed MediaWiki email during registration - https://phabricator.wikimedia.org/T230590#9794267 (10Aklapper) a:03Aklapper Thanks again, you were right with everything in your last comment. :D [12:30:59] 10Continuous-Integration-Config: Display CI errors in the Gerrit UI - https://phabricator.wikimedia.org/T364849 (10Tgr) 03NEW [12:32:34] 10Continuous-Integration-Config: Display CI errors in the Gerrit UI - https://phabricator.wikimedia.org/T364849#9794286 (10Tgr) [12:37:09] 10MediaWiki-Releasing, 05MW-1.42-release: Write release announcement for 1.42.0-rc.0 - https://phabricator.wikimedia.org/T359843#9794292 (10Reedy) @MSantos git tag is pushed, tarballs are uploaded. Just the last step is to send the email to the three mailing lists (mediawiki-announce, wikitech-l and mediawiki-... [12:57:12] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850 (10brennen) 03NEW [13:01:28] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 10Wikimedia-Phabricator-Extensions: Make WMF's Phabricator Translations extension work with an upstream Phorge installation - https://phabricator.wikimedia.org/T364426#9794365 (10Aklapper) [13:13:23] (03open) 10brouberol: Update projects.json [repos/releng/gitlab-trusted-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/merge_requests/73 (https://phabricator.wikimedia.org/T364795) [13:23:24] 10Release-Engineering-Team (Yakisfaction), 05Release, 05Train Deployments: 1.43.0-wmf.5 deployment blockers - https://phabricator.wikimedia.org/T361399#9794558 (10hashar) [13:35:08] (03open) 10brennen: update submodules for 2024-05-14 deploy [repos/phabricator/deployment] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/deployment/-/merge_requests/42 (https://phabricator.wikimedia.org/T364850) [13:36:16] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services, 13Patch-For-Review: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850#9794614 (10CodeReviewBot) brennen opened https://gitlab.wikimedia.org/repos/phabricator/deployment/-/me... [13:52:17] (03CR) 10Hashar: [C:03+2] Zuul: [mediawiki/skins/Vector] Add MobileFrontend phan dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1028589 (owner: 10Fomafix) [13:53:42] (03Merged) 10jenkins-bot: Zuul: [mediawiki/skins/Vector] Add MobileFrontend phan dependency [integration/config] - 10https://gerrit.wikimedia.org/r/1028589 (owner: 10Fomafix) [13:56:16] (03CR) 10Hashar: [C:03+2] "Deployed!" [integration/config] - 10https://gerrit.wikimedia.org/r/1028589 (owner: 10Fomafix) [14:08:21] 10Phabricator: Disable old Phabricator OAuth consumers on mediawiki.org - https://phabricator.wikimedia.org/T364788#9794834 (10Aklapper) Ah, thanks for explaining and disabling! [14:43:23] (03merge) 10dancy: Update allowed_projects [repos/releng/docpub] - 10https://gitlab.wikimedia.org/repos/releng/docpub/-/merge_requests/22 (owner: 10kbach) [14:47:12] (03update) 10esanders: Downgrade errors with translated data (in default config) to warnings [repos/ci-tools/banana-checker] - 10https://gitlab.wikimedia.org/repos/ci-tools/banana-checker/-/merge_requests/13 (https://phabricator.wikimedia.org/T364727) [14:48:46] (03open) 10esanders: Add support for .bananaconfig.json files [repos/ci-tools/banana-checker] (warn) - 10https://gitlab.wikimedia.org/repos/ci-tools/banana-checker/-/merge_requests/15 (https://phabricator.wikimedia.org/T364843) [14:49:12] (03open) 10esanders: Major config refactor [repos/ci-tools/banana-checker] (configfile) - 10https://gitlab.wikimedia.org/repos/ci-tools/banana-checker/-/merge_requests/16 (https://phabricator.wikimedia.org/T364843) [14:52:54] (03merge) 10brennen: update submodules for 2024-05-14 deploy [repos/phabricator/deployment] (wmf/stable) - 10https://gitlab.wikimedia.org/repos/phabricator/deployment/-/merge_requests/42 (https://phabricator.wikimedia.org/T364850) [14:53:02] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services, 13Patch-For-Review: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850#9795029 (10CodeReviewBot) brennen merged https://gitlab.wikimedia.org/repos/phabricator/deployment/-/me... [15:04:08] 10Phabricator (phabricator-next), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services, 13Patch-For-Review: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850#9795116 (10ops-monitoring-bot) Icinga downtime and Alertmanager silence (ID=6e2580b0-999e-4a68-87e7-c37... [15:10:05] 10Gerrit, 06Release-Engineering-Team, 06collaboration-services: Gerrit ssh host key fails on fedora39 - https://phabricator.wikimedia.org/T364217#9795150 (10Dzahn) Thanks for confirming that @Tarrow Good to know we have a workaround. [15:36:39] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07Technical-Debt: Remove custom WMF Phab downstream hacks superseded by upstream Phorge changes - https://phabricator.wikimedia.org/T341799#9795255 (10Aklapper) [15:37:18] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 10Wikimedia-Phabricator-Extensions: Make WMF's Phabricator Translations extension work with an upstream Phorge installation - https://phabricator.wikimedia.org/T364426#9795258 (10Aklapper) 05Open→03Resolved This got deployed today... [15:37:23] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07Technical-Debt: Remove custom WMF Phab downstream hacks superseded by upstream Phorge changes - https://phabricator.wikimedia.org/T341799#9795256 (10Aklapper) 05Open→03Resolved Except for dedicated 364088, all of this got deploy... [15:37:31] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07Technical-Debt: Remove custom "Download Archive" Github tarball link from Diffusion - https://phabricator.wikimedia.org/T364087#9795263 (10Aklapper) 05Open→03Resolved This got deployed today on 2024-05-14. [15:37:34] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850#9795260 (10Aklapper) 05Open→03Resolved a:03Aklapper This got deployed today on 2024-05-14. [15:37:42] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction): Revert to using upstream src/infrastructure/daemon/workers/management/PhabricatorWorkerManagementWorkflow.php - https://phabricator.wikimedia.org/T352721#9795267 (10Aklapper) 05Open→03Resolved This got deployed today on 2024-05-14. [15:37:49] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07Technical-Debt: Remove non-performant custom Conduit API maniphest.getprojecttasktransactions - https://phabricator.wikimedia.org/T360654#9795265 (10Aklapper) 05Open→03Resolved This got deployed today on 2024-05-14. [15:38:11] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07CSS: Label cut-off in Phabricator report tables - https://phabricator.wikimedia.org/T301024#9795274 (10Aklapper) 05Open→03Resolved This got deployed today on 2024-05-14. [15:38:15] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 07Technical-Debt: Remove unneeded custom changes in webroot/rsrc/css/phui/phui-property-list-view.css - https://phabricator.wikimedia.org/T352667#9795269 (10Aklapper) 05Open→03Resolved This got deployed today on 2024-05-14. [15:38:21] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 10Wikimedia-Phabricator-Extensions, 07Technical-Debt: Remove custom go-get / go-import code in Diffusion repositories - https://phabricator.wikimedia.org/T352322#9795272 (10Aklapper) 05Open→03Resolved This got deployed today on... [15:39:57] 10Phabricator (2024-05-14), 10Release-Engineering-Team (Yakisfaction), 06collaboration-services: Deploy Phabricator/Phorge 2024-05-14 - https://phabricator.wikimedia.org/T364850#9795295 (10Aklapper) a:05Aklapper→03brennen [15:44:55] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Configure Docker builder GC settings for CI - https://phabricator.wikimedia.org/T364773#9795321 (10Dzahn) [16:23:47] 10Scap, 06SRE, 13Patch-For-Review: Failed scap check for totoro.wikimedia.org during deployment - https://phabricator.wikimedia.org/T364880#9795831 (10dancy) [16:27:39] 10Scap, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during sacp deployment - https://phabricator.wikimedia.org/T364880#9795868 (10Dzahn) scap runs `httpbb /srv/deployment/httpbb-tests/appserver/* --hosts=mwdebug.discovery.wmnet --https_port=4444 --retry_on_timeout` This... [16:28:52] 10Scap, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during sacp deployment - https://phabricator.wikimedia.org/T364880#9795900 (10dancy) [16:30:20] 10Scap, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during scap deployment - https://phabricator.wikimedia.org/T364880#9795958 (10dancy) [16:30:57] 10Scap, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during scap deployment - https://phabricator.wikimedia.org/T364880#9795971 (10Lucas_Werkmeister_WMDE) > ^ It seems wrong that this doesn't fail. I’m not sure why it should fail? It seem to match the behavior I can see... [16:39:11] 10Scap, 06serviceops-radar, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during scap deployment - https://phabricator.wikimedia.org/T364880#9796100 (10Dzahn) Lucas is right. I can confirm the test passes with any wikimedia.org subdomain as long as the path stays /wiki/M... [16:39:25] 10Scap, 06serviceops-radar, 06SRE, 13Patch-For-Review: Confusing failed httpbb check for totoro.wikimedia.org during scap deployment - https://phabricator.wikimedia.org/T364880#9796101 (10Dzahn) [16:54:16] 10Scap, 06serviceops-radar, 06SRE: httpbb should show more information / details about failed checks - https://phabricator.wikimedia.org/T364886 (10Lucas_Werkmeister_WMDE) 03NEW [17:09:18] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team: Configure Docker builder GC settings for CI - https://phabricator.wikimedia.org/T364773#9796299 (10dancy) 05Open→03Resolved Closing optimistically. [17:32:23] (03PS1) 10Bartosz Dziewoński: Add Wargo to CI allow list [integration/config] - 10https://gerrit.wikimedia.org/r/1031519 [17:49:02] (03CR) 10Majavah: [C:03+2] Add Wargo to CI allow list [integration/config] - 10https://gerrit.wikimedia.org/r/1031519 (owner: 10Bartosz Dziewoński) [17:51:00] (03Merged) 10jenkins-bot: Add Wargo to CI allow list [integration/config] - 10https://gerrit.wikimedia.org/r/1031519 (owner: 10Bartosz Dziewoński) [17:51:20] !log reload zuul for https://gerrit.wikimedia.org/r/c/integration/config/+/1031519 [17:51:22] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [17:51:25] (03CR) 10Majavah: [C:03+2] "Deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/1031519 (owner: 10Bartosz Dziewoński) [18:10:11] (03update) 10dancy: buildkit.Compile: Fix comment [repos/releng/blubber] - 10https://gitlab.wikimedia.org/repos/releng/blubber/-/merge_requests/79 [18:15:23] (03merge) 10dancy: buildkit.Compile: Fix comment [repos/releng/blubber] - 10https://gitlab.wikimedia.org/repos/releng/blubber/-/merge_requests/79 [19:15:41] 10Phabricator (phabricator-next): QUESTION: Transition to Phorge? - https://phabricator.wikimedia.org/T364908 (10Sydbarrett74) 03NEW [19:26:56] 10Release-Engineering-Team (Yakisfaction): Blubber Python builder: Support virtualenv setup - https://phabricator.wikimedia.org/T357548#9796966 (10dancy) p:05Triage→03Medium a:03dancy [19:49:57] 10Phabricator, 06collaboration-services: QUESTION: Transition to Phorge? - https://phabricator.wikimedia.org/T364908#9797054 (10RhinosF1) [19:51:28] 10Phabricator, 06collaboration-services: QUESTION: Transition to Phorge? - https://phabricator.wikimedia.org/T364908#9797061 (10RhinosF1) 05Open→03Invalid We've been running Phorge since {T333885} [19:58:02] Is there any kind of metadata on our docker registry images I can use to learn more about the images? Like I'm wondering what the difference is between 'python3-bookworm' and 'python3-build-bookworm' [20:00:44] hmm, I guess the answer to that is 'check the production-images' repo. sorry ;P [20:01:59] inflatador: serviceops can help :) [20:02:22] but essentially the first is intended to serve as a base image to run an app [20:02:43] while the build one would have gcc / extra dev libs to compile dependencies [20:02:59] and one probably does not need gcc for a python micro service [20:03:13] (unless we one day with a `compileoid` service) [20:03:24] (unless we one day come up with a `compileoid` service) [20:03:30] but yeah read the source :) [20:06:57] LOL [20:09:51] blubber doesn't have a "--no-install-recommends" flag, does it? [20:09:58] for its apt provider [20:11:27] airflow does seem to want to pull in silly amounts of g++/gcc and friends [20:18:09] inflatador: I have no idea :( [20:19:43] np, I'll stop babbling ;) . I think the build image is the right one for now. Thanks for your help! [20:20:32] inflatador: https://gitlab.wikimedia.org/repos/releng/blubber/-/blob/main/config/apt.go?ref_type=heads [20:20:32] that is the source code for Blubber `apt` [20:20:44] then maybe the no install recommends can be set via dpkg configuration [20:20:47] or apt config [20:21:31] inflatador: https://phabricator.wikimedia.org/T289596 might be relevant [20:21:46] "Our containers already disable installing recommended packages via apt configuration:" [20:21:48] and thus be set in a parent image via /etc/apt/apt.conf.d or /etc/dpkg/dpkg.cfg.d [20:22:23] which is probably the case since we have a customization to prevent installing man pages for examples [20:22:54] thanks mutante hashar ...sounds like a non-issue then [21:45:59] (03open) 10eevans: Added repos/sre/data-gateway to trusted runners [repos/releng/gitlab-trusted-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/merge_requests/74 (https://phabricator.wikimedia.org/T364921) [21:56:05] hashar: Please have a look at https://gerrit.wikimedia.org/r/c/integration/config/+/1028900 [21:56:32] 10Gerrit, 07Upstream: Add copy command without "git clone" wrapper - https://phabricator.wikimedia.org/T363646#9797740 (10Reedy) Yeah, but I don't need the `git clone` part; especially when using it with an IDE. Both GitLab and GitHub don't include it... [21:57:53] (03merge) 10dancy: Added repos/sre/data-gateway to trusted runners [repos/releng/gitlab-trusted-runner] - 10https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/merge_requests/74 (https://phabricator.wikimedia.org/T364921) (owner: 10eevans) [22:11:01] 06Release-Engineering-Team, 06collaboration-services: Create test Gerrit instance - https://phabricator.wikimedia.org/T363196#9797784 (10Dzahn) a:03Dzahn [22:19:45] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [22:19:58] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [22:21:54] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [22:23:41] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [22:30:34] (03update) 10jhuneidi: Draft: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [22:30:52] (03update) 10jhuneidi: backport.py: Restore included_in branches check [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/270 (https://phabricator.wikimedia.org/T360291) [23:19:22] Can anyone tell me why this is stuck? https://gitlab.wikimedia.org/repos/sre/data-gateway/-/jobs/261578 [23:19:50] it's the first run of build-and-publish-production-image so I'm sure I must have done something wrong [23:35:46] urandom: I think the issue may be that the commit is on both the "data-gateway" and "main" branches and that "data-gateway" is not a protected branch in that repo. [23:35:55] "Please note Trusted Runners accept jobs from protected branches only. So make sure to protect the main branch and to trigger this job on protected branches only. Otherwise the jobs will get stuck." -- https://wikitech.wikimedia.org/wiki/GitLab/Gitlab_Runner/Trusted_Runners#Dockerfile_support [23:37:38] oh. [23:38:58] that's unfortunate [23:40:38] urandom: do you have rights to mark the data-gateway branch as protected? [23:40:51] that branch was only temporary [23:41:07] I wonder if I can delete it and rerun? [23:41:45] likely, yes [23:42:28] or mark the branch as protected, let things get unstuck, and then delete the feature branch and the protection rule for it [23:43:11] heh [23:43:34] I'm not sure if I could have marked it protected, but I can't now (because I already deleted it) [23:44:37] https://gitlab.wikimedia.org/repos/sre/data-gateway/-/settings/repository is the screen that would do it [23:47:51] I wonder if the rules for the build-and-publish-production-image job should also include `$CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH`? [23:48:34] That might be functionally the same as the $CI_COMMIT_REF_PROTECTED guard that is already there though. Hmmm [23:50:49] anyway... I added that branch back (matching sha), protected it (I could in fact do that), and retried the job... and it is also stuck [23:53:12] if that's why this is stuck, it's unfortunate because merging a branch to main, and then tagging a release would seem like a pretty normal thing to have happen [23:53:53] the job also failed to run with the branch removed? [23:53:59] yes [23:54:13] and then I added it back, marked it protected, and tried again... which also failed [23:54:44] I don't know how to check 2 things: if there are actually trusted runners online, and if this repo is allowed to use trusted runners [23:55:10] it was added to trusted runners [23:55:21] we need a fancier nerd like maybe dancy or brennen at this point I think [23:55:37] https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/commit/35c91ec3792634bf95e0b4532fbfc06b94f27ae2 [23:55:54] heh [23:57:28] urandom: I don't think that config change has actually be applied to the runners yet -- https://gitlab.wikimedia.org/repos/releng/gitlab-trusted-runner/-/pipelines/53058 [23:57:38] that "apply-job" needs to fire [23:58:05] ok, that makes sense [23:58:53] I guess I assumed that it was ran when the request was merged [23:59:05] bd808: see, you're a very fancy nerd!