[00:43:21] (03PS1) 10Legoktm: Add mwbot-rs to doc.wikimedia.org [integration/docroot] - 10https://gerrit.wikimedia.org/r/938025 (https://phabricator.wikimedia.org/T341543) [00:45:12] am I allowed to self-merge docroot changes? [02:39:00] I think changes to integration docroot will neeed a scap deploy. [02:39:18] modules/profile/manifests/ci/website.pp: scap::target { 'integration/docroot': [05:49:16] 10Release-Engineering-Team, 10MediaWiki-General: wikimedia/mediawiki docker image out of date - https://phabricator.wikimedia.org/T274656 (10Legoktm) This isn't about the #docker-hub-mediawiki image, it's about some other image hosted on docker hub. Blindly tagging #release-engineering-team in case they have a... [06:29:52] 10GitLab (Pipeline Services Migration🐤), 10Release-Engineering-Team (Priority Backlog 📥), 10Toolforge, 10cloud-services-team: Move Toolforge PipelineLib repositories to GitLab - https://phabricator.wikimedia.org/T334399 (10dcaro) [07:11:29] legoktm: yes, I guess it is fine for changes made to opensource.yaml. For changes to the php and js apps, it is probably better to loop Timo in :) [07:11:42] or poke folks :] [07:12:19] (03CR) 10Hashar: [C: 03+2] Add mwbot-rs to doc.wikimedia.org [integration/docroot] - 10https://gerrit.wikimedia.org/r/938025 (https://phabricator.wikimedia.org/T341543) (owner: 10Legoktm) [07:12:30] deploying it [07:12:52] (03Merged) 10jenkins-bot: Add mwbot-rs to doc.wikimedia.org [integration/docroot] - 10https://gerrit.wikimedia.org/r/938025 (https://phabricator.wikimedia.org/T341543) (owner: 10Legoktm) [07:14:13] (03CR) 10Hashar: [C: 03+2] "Deployed with scap. Note the site is behind ATS/Varnish with a one hour cache." [integration/docroot] - 10https://gerrit.wikimedia.org/r/938025 (https://phabricator.wikimedia.org/T341543) (owner: 10Legoktm) [07:14:27] {done} [08:10:20] 10GitLab (Project Migration), 10Release-Engineering-Team (Priority Backlog 📥), 10API Platform, 10Anti-Harassment, and 18 others: Migrate PipelineLib repos to GitLab - https://phabricator.wikimedia.org/T332953 (10dcaro) [08:11:45] 10GitLab (Project Migration), 10Release-Engineering-Team (Priority Backlog 📥), 10API Platform, 10Anti-Harassment, and 18 others: Migrate PipelineLib repos to GitLab - https://phabricator.wikimedia.org/T332953 (10dcaro) [08:15:33] 10GitLab (Auth & Access), 10Release-Engineering-Team (They Live 🕶️🧟), 10CAS-SSO, 10Infrastructure-Foundations, and 4 others: migrate gitlab away from the CAS protocol - https://phabricator.wikimedia.org/T320390 (10Jelto) @Arnoldokoth thanks for testing this. You are talking about the your individual profil... [08:28:40] 10Phabricator (Upstream), 10Upstream: When a project is not public, it appears to users outside "Visible To" criteria as "Unknown Object", which is confusing - https://phabricator.wikimedia.org/T302525 (10Aklapper) 05Open→03Declined I assume this task is about not showing the `"Unknown Object"` at all, so... [08:34:13] 10GitLab (Auth & Access), 10Release-Engineering-Team (They Live 🕶️🧟), 10CAS-SSO, 10Infrastructure-Foundations, and 4 others: migrate gitlab away from the CAS protocol - https://phabricator.wikimedia.org/T320390 (10SLyngshede-WMF) @Jelto No not yet, and I just tried to login still the same error. I can see... [08:52:07] 10Phabricator (Upstream), 10Upstream: Incorrect height of class "phui-oi-frame" - https://phabricator.wikimedia.org/T307492 (10Aklapper) 05Stalled→03Open Ahhhh! Nice catch! I can reproduce. Upstreamed as https://we.phorge.it/T15552 [09:06:32] 10Phabricator (Upstream), 10Upstream: Upload popup overflows box with a long filename - https://phabricator.wikimedia.org/T316305 (10Aklapper) [09:07:21] 10Phabricator (Upstream), 10CSS, 10Upstream: Upload popup overflows box with a long filename - https://phabricator.wikimedia.org/T316305 (10Aklapper) Can reproduce in Firefox 115. I do not see any issue in Chromium 114 - seems to be smarter. [09:07:37] 10Phabricator (Upstream), 10CSS, 10Upstream: Upload popup overflows box with a long filename - https://phabricator.wikimedia.org/T316305 (10Aklapper) [09:28:25] 10Phabricator (Upstream), 10CSS, 10Upstream: Upload popup overflows box with a long filename - https://phabricator.wikimedia.org/T316305 (10Aklapper) a:03Aklapper [09:37:23] 10Phabricator (Upstream), 10CSS, 10Upstream: Incorrect height of class "phui-oi-frame" - https://phabricator.wikimedia.org/T307492 (10Aklapper) p:05Triage→03Low a:03Aklapper [10:47:45] 10Phabricator: Tasks created via Conduit API have restricted edit policy (due to general "Default Edit Policy" for Maniphest) - https://phabricator.wikimedia.org/T305998 (10Aklapper) [10:51:20] 10Project-Admins: Requests for addition to the #acl*Project-Admins group (in comments) - https://phabricator.wikimedia.org/T706 (10WMDE-leszek) Kindly requesting to add the following accoutns of my WMDE team mates. @Ifrahkhanyaree is a Product Manager, and @Silvan_WMDE , @WMDE_Norman @Muhammad_Yasser_Jazirahly_W... [10:52:52] 10GitLab (Pipeline Services Migration🐤), 10collaboration-services: Move micro sites from Ganeti to Kubernetes and from Gerrit to GitLab - https://phabricator.wikimedia.org/T300171 (10Jelto) a:03Jelto Thanks for merging the above change @Dzahn . I deployed the change to all wikikube clusters. So we are using... [11:18:25] Hello beautiful release engineering people!!! I hope you are all doing well :) We had a patch merged on dev-images yesterday that we're keen to get it published to the wmf docker registry and I'm wondering do people usually write a phab task for that or just try to charm designated con-int admins on IRC? (hence the attempted charm at the beginning) [11:18:30] the patch that was merged is here https://gitlab.wikimedia.org/repos/releng/dev-images/-/merge_requests/45 [11:19:36] jgleeson: I imagine both work ;P [11:19:47] :D [11:19:55] I'm curious if I have the rights for that repo... [11:20:07] I can build the CI ones, so I must be able to do those.. [11:23:11] !log Updating development images on contint primary for various commits... [11:23:12] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [11:23:34] thank you Reedy! [11:24:00] there's quite a few undeployed stuff... [11:24:28] [contint.wikimedia.org] cat /tmp/docker-pkg-build.log [11:24:28] sudo: cd: command not found [11:24:28] lol [11:24:43] It's doing something [11:25:30] jgleeson: usually the workflow for these kinds of repos is that by merging a patch you indicate that you're going to build it, so if you don't have rights to do so you would add people who do as reviewers to it [11:27:04] jgleeson: in theory... [11:27:05] == Step 2: publishing == [11:27:05] Successfully published image docker-registry.discovery.wmnet/dev/fundraising-tools-bullseye:1.0.0 [11:31:50] ah ok thanks taavi I'll remember that for next time [11:31:59] thanks again Reedy [11:34:00] Not difficult for someone to do though... clone the repo and run fab (if you have the rights) [11:34:57] 10Release-Engineering-Team (Seen), 10MW-on-K8s, 10SRE, 10Traffic, and 2 others: Serve production traffic via Kubernetes - https://phabricator.wikimedia.org/T290536 (10Reedy) [11:36:48] 'if you have the rights' is the hard part here. I'm in contint-admins and can update jenkins and zuul config but can't build docker images [12:19:32] 10Diffusion, 10Phabricator (Upstream), 10Upstream: Error trying to view previous git blame ("Skip past this commit" button): "reset() expects parameter 1 to be array, null given" - https://phabricator.wikimedia.org/T274559 (10Aklapper) ` reset() expects parameter 1 to be array, null given Stack trace: # Phu... [12:26:51] 10Diffusion, 10Phabricator (Upstream), 10Upstream: Error trying to view previous git blame ("Skip past this commit" button): "reset() expects parameter 1 to be array, null given" - https://phabricator.wikimedia.org/T274559 (10Aklapper) p:05Triage→03Low [12:55:36] 10Release-Engineering-Team (Deployment Training Requests): Deployment Training Request for jebe - https://phabricator.wikimedia.org/T341559 (10JEbe-WMF) [13:08:49] hashar: thanks! [13:10:25] !log Dropping default from campaign_events.event_answers_status column in beta wikishared # T341642 [13:10:30] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [13:10:32] T341642: Add the campaign_events.event_answers_status column in beta DB - https://phabricator.wikimedia.org/T341642 [13:11:37] 10Phabricator, 10Release-Engineering-Team, 10User-brennen: Remove unused(?) fixed_settings.yaml from Phabricator puppet config - https://phabricator.wikimedia.org/T331673 (10Aklapper) Is {T239355} a dup to merge into this ticket? [13:11:54] 10Phabricator, 10Release-Engineering-Team, 10User-brennen: Remove unused(?) fixed_settings.yaml from Phabricator puppet config - https://phabricator.wikimedia.org/T331673 (10Aklapper) p:05Triage→03Medium [13:30:02] 10GitLab (Auth & Access), 10Release-Engineering-Team (They Live 🕶️🧟), 10CAS-SSO, 10Infrastructure-Foundations, and 4 others: migrate gitlab away from the CAS protocol - https://phabricator.wikimedia.org/T320390 (10SLyngshede-WMF) For OIDC via CAS in our Python applications we're relying on a special OIDC b... [13:34:31] 10GitLab (Auth & Access), 10Release-Engineering-Team (They Live 🕶️🧟), 10CAS-SSO, 10Infrastructure-Foundations, and 4 others: migrate gitlab away from the CAS protocol - https://phabricator.wikimedia.org/T320390 (10SLyngshede-WMF) The fixes that worked a little, but failed to provide correct user informatio... [15:15:16] 10Continuous-Integration-Config, 10Release-Engineering-Team, 10Fresnel, 10MediaWiki-Platform-Team-Backlog, and 2 others: Make it possible to run Fresnel on MediaWiki extension repos - https://phabricator.wikimedia.org/T220162 (10larissagaulia) @thcipriani If you decide to take the project on we could arran... [15:39:18] Daimona: so for https://gerrit.wikimedia.org/r/c/integration/quibble/+/938254 I'm thinking whether I should squash the two changes together, or temporarily modify the Gerrit ACLs for that repo [15:39:46] I think both options are fine, especially because the CA change is a small, self-contained, test-only change [15:39:58] So whatever you prefer :_) [15:40:10] I don't like either of those, that's the problem :D [15:41:11] I think I'll go with the ACL route [15:41:52] Yeah, circular dependencies suck :/ [15:42:07] and done [15:43:11] so the tests on https://gerrit.wikimedia.org/r/c/integration/quibble/+/938010 should pass now, I'll leave the actual +2'ing to you or someone else who understands that change better than I do [15:56:49] I don't have +2 on that repo [15:57:03] (03PS1) 10Subramanya Sastry: Remove workarounds for Parsoid HTML now that core patches are live [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/938270 [15:58:05] h.ashar reviewed something this morning but it's a holiday in France so I don't think he's around [16:03:22] oh, I thought that was inheriting +2 from mediawiki/*. in that case I hope I haven't stepped on anyone's toes by merging the first patch [16:08:28] You fixed failing CI on master, I don't think that's stepping over anybody's toes ;-) [16:14:51] 10GitLab (Pipeline Services Migration🐤), 10collaboration-services: Move micro sites from Ganeti to Kubernetes and from Gerrit to GitLab - https://phabricator.wikimedia.org/T300171 (10Dzahn) [16:50:14] 10GitLab (Pipeline Services Migration🐤), 10collaboration-services: Move micro sites from Ganeti to Kubernetes and from Gerrit to GitLab - https://phabricator.wikimedia.org/T300171 (10Dzahn) Yep! So regarding static-bugzilla on Gerrit. That content repo is _quite large_ compared to any other microsite. Because... [17:08:05] 10Phabricator, 10DBA, 10Data-Persistence-Backup, 10collaboration-services, 10Patch-For-Review: phabricator->phorge migration - database handling - https://phabricator.wikimedia.org/T335080 (10Dzahn) @Marostegui or @Ladsgroup Could the replication to this host be disabled? ^thanks! [18:03:25] 10Gitlab-Application-Security-Pipeline, 10Security Team AppSec, 10Security-Team, 10SecTeam-Processed, 10Security: Address issues within certain Gitlab CI security templates - https://phabricator.wikimedia.org/T338034 (10sbassett) Ok, so I think the above MR being merged addresses issue (2). The rest are... [18:05:03] 10Gitlab-Application-Security-Pipeline, 10Security Team AppSec, 10Security-Team, 10SecTeam-Processed, 10Security: Address issues within certain Gitlab CI security templates - https://phabricator.wikimedia.org/T338034 (10sbassett) [18:45:00] !log Tag Quibble 1.5.5 @ 85009d62de # T90875 T227900 T341840 [18:45:06] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:45:06] T90875: Use vendor/bin/phpunit instead of tests/phpunit/phpunit.php - https://phabricator.wikimedia.org/T90875 [18:45:06] T341840: CentralAuthExtensionJsonTest fails if SecurePoll is not installed - https://phabricator.wikimedia.org/T341840 [18:45:07] T227900: Phase out usage of tests/phpunit/suite.xml - https://phabricator.wikimedia.org/T227900 [18:49:29] (03PS1) 10Jforrester: Docker [quibble] Upgrade Quibble to 1.5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/938296 (https://phabricator.wikimedia.org/T90875) [18:54:12] (03CR) 10Jforrester: [C: 03+2] Docker [quibble] Upgrade Quibble to 1.5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/938296 (https://phabricator.wikimedia.org/T90875) (owner: 10Jforrester) [18:55:20] (03Merged) 10jenkins-bot: Docker [quibble] Upgrade Quibble to 1.5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/938296 (https://phabricator.wikimedia.org/T90875) (owner: 10Jforrester) [18:56:23] !log Docker: Building and publishing quibble images # T90875 T227900 T341840 [18:56:28] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:56:29] T90875: Use vendor/bin/phpunit instead of tests/phpunit/phpunit.php - https://phabricator.wikimedia.org/T90875 [18:56:29] T341840: CentralAuthExtensionJsonTest fails if SecurePoll is not installed - https://phabricator.wikimedia.org/T341840 [18:56:29] T227900: Phase out usage of tests/phpunit/suite.xml - https://phabricator.wikimedia.org/T227900 [19:04:23] (03PS1) 10Jforrester: jjb: Update quibble jobs to images with Quibble 1.5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/938298 (https://phabricator.wikimedia.org/T90875) [19:13:13] !log jjb: First testing on quibble-vendor-mysql-php74-noselenium-docker [19:13:15] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [19:27:18] (03CR) 10Jforrester: [C: 03+2] "Fully deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/938298 (https://phabricator.wikimedia.org/T90875) (owner: 10Jforrester) [19:28:52] (03Merged) 10jenkins-bot: jjb: Update quibble jobs to images with Quibble 1.5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/938298 (https://phabricator.wikimedia.org/T90875) (owner: 10Jforrester) [20:26:19] (03CR) 10Arlolra: [C: 03+2] Remove workarounds for Parsoid HTML now that core patches are live (032 comments) [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/938270 (owner: 10Subramanya Sastry) [20:26:57] (03Merged) 10jenkins-bot: Remove workarounds for Parsoid HTML now that core patches are live [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/938270 (owner: 10Subramanya Sastry) [20:56:00] 10Release-Engineering-Team, 10Scap, 10Patch-For-Review, 10Wikimedia-Incident: Warn against potentially deploying a config patch NOT specified - https://phabricator.wikimedia.org/T338168 (10CodeReviewBot) dancy merged https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/156 backport: add tests f... [21:55:43] 10Release-Engineering-Team, 10Scap, 10MW-on-K8s, 10Patch-For-Review: Inform deployers of mw-debug deployment - https://phabricator.wikimedia.org/T341798 (10CodeReviewBot) dancy opened https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/162 Mention k8s mw-debug deployment if --pause-after-tests... [22:59:40] 10GitLab (Project Migration), 10Release-Engineering-Team (Priority Backlog 📥), 10Data Engineering and Event Platform Team, 10Data-Platform-SRE: Migrate analytics/datahub pipeline to GitLab - https://phabricator.wikimedia.org/T341194 (10BTullis) p:05Triage→03High