[00:16:07] !log Cherry-picking https://gerrit.wikimedia.org/r/c/operations/puppet/+/639885 and https://gerrit.wikimedia.org/r/c/performance/arc-lamp/+/639886 in beta. Should only affect deployment-webperf12. [00:16:09] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [00:20:50] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team (Code Health), 10Release-Engineering-Team-TODO, 10Code-Stewardship-Reviews: deployment-prep: Code stewardship request - https://phabricator.wikimedia.org/T215217 (10thcipriani) I had a conversation about this ticket with @nskaggs and I felt that I s... [00:23:45] (03PS1) 10Umherirrender: Remove semicolon in UnusedUseStatementSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) [00:35:44] (03CR) 10DannyS712: "Suggest adding a test too" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) (owner: 10Umherirrender) [00:41:35] (03PS1) 10Jforrester: make-release: Stop branching the Petition extension [tools/release] - 10https://gerrit.wikimedia.org/r/639888 (https://phabricator.wikimedia.org/T267450) [00:43:08] 10Release-Engineering-Team (Deployment services), 10VPS-project-codesearch, 10MediaWiki-Release-Tools, 10Patch-For-Review: Stop branching the Petition repo for production (and thus showing up in CodeSearch) - https://phabricator.wikimedia.org/T267450 (10Jdforrester-WMF) [00:43:12] 10Release-Engineering-Team (Deployment services), 10VPS-project-codesearch, 10MediaWiki-Release-Tools, 10Patch-For-Review: Stop branching the Petition repo for production (and thus showing up in CodeSearch) - https://phabricator.wikimedia.org/T267450 (10Jdforrester-WMF) a:03Jdforrester-WMF Oops. [00:44:47] (03CR) 10Jforrester: "Leaving to RelEng to validate and merge, rather than self-merging, as I'm no longer on the team and so it's not my day that gets ruined if" [tools/release] - 10https://gerrit.wikimedia.org/r/639888 (https://phabricator.wikimedia.org/T267450) (owner: 10Jforrester) [00:55:34] (03CR) 10Krinkle: [C: 03+1] "Thanks :)" [tools/release] - 10https://gerrit.wikimedia.org/r/639888 (https://phabricator.wikimedia.org/T267450) (owner: 10Jforrester) [01:38:54] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team (Code Health), 10Release-Engineering-Team-TODO, 10Code-Stewardship-Reviews: deployment-prep: Code stewardship request - https://phabricator.wikimedia.org/T215217 (10dpifke) As someone who considers beta essential to my role, I'll add a data point wi... [04:10:32] 10Release-Engineering-Team, 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)), 10MediaWiki-Docker, 10Developer Productivity, and 2 others: mediawiki-docker exec feature - https://phabricator.wikimedia.org/T262370 (10jeena) [04:36:34] (03CR) 10DannyS712: [C: 03+1] make-release: Stop branching the Petition extension [tools/release] - 10https://gerrit.wikimedia.org/r/639888 (https://phabricator.wikimedia.org/T267450) (owner: 10Jforrester) [08:44:07] 10Continuous-Integration-Config, 10Math, 10Platform Engineering, 10Platform Engineering Roadmap Decision Making, 10Patch-For-Review: Enable mediawiki-quibble-apitests-vendor-docker for extension Math - https://phabricator.wikimedia.org/T254031 (10hashar) > However, there are two new problems both seem to... [08:45:21] (03CR) 10Hashar: [C: 03+2] "Passed as of https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Math/+/639630" [integration/config] - 10https://gerrit.wikimedia.org/r/639651 (https://phabricator.wikimedia.org/T254031) (owner: 10Physikerwelt) [08:46:03] (03Merged) 10jenkins-bot: Revert "layout: [mediawiki/extensions/Math] Disable API tests, not passing yet" [integration/config] - 10https://gerrit.wikimedia.org/r/639651 (https://phabricator.wikimedia.org/T254031) (owner: 10Physikerwelt) [08:47:37] (03CR) 10Hashar: "Thank you, and I have deploy it!" [integration/config] - 10https://gerrit.wikimedia.org/r/639651 (https://phabricator.wikimedia.org/T254031) (owner: 10Physikerwelt) [10:20:06] 10Project-Admins: Create a sub-project for Wikidata primary sources tool - https://phabricator.wikimedia.org/T163231 (10Aklapper) @Hjfocs (and maybe @Lydia_Pintscher): T259410 implies that this project is dead. Is that correct and its code, wiki pages, and tasks should be archived? Please reply in T259410. [11:15:10] 10Project-Admins, 10Igbo-Wikimedians-User-Group: Igbo-Wikimedians-User-Group-Projects - https://phabricator.wikimedia.org/T266781 (10Nintendofan885) [12:40:42] (03CR) 10Umherirrender: "There are tests for this sniff, but currently another sniff removes the semicolon - now this is part of the sniff itself. I have no idea h" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) (owner: 10Umherirrender) [15:29:46] 10Project-Admins: Create a #Wikimedia-User-Group-China group project - https://phabricator.wikimedia.org/T172163 (10Aklapper) @Shizhao: Could you please answer the last comment? Thanks in advance! [15:33:56] (03PS2) 10Umherirrender: Remove semicolon in UnusedUseStatementSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) [16:23:15] (03CR) 10Thiemo Kreuz (WMDE): [C: 04-1] "Most of what this sniff does appears to be unrelated to the semicolon. Can you please split this patch? A first patch could make them more" (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) (owner: 10Umherirrender) [16:36:15] (03PS3) 10Umherirrender: Make tests for Mediawiki.Classes sniff more standalone [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 [16:36:39] (03CR) 10Umherirrender: [C: 04-2] "> Patch Set 2: Code-Review-1" (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (owner: 10Umherirrender) [16:37:34] (03PS4) 10Thiemo Kreuz (WMDE): Make tests for Mediawiki.Classes sniff more standalone [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (owner: 10Umherirrender) [16:40:35] (03CR) 10jerkins-bot: [V: 04-1] Make tests for Mediawiki.Classes sniff more standalone [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (owner: 10Umherirrender) [16:40:57] (03CR) 10Thiemo Kreuz (WMDE): Make tests for Mediawiki.Classes sniff more standalone (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (owner: 10Umherirrender) [16:46:44] 10MediaWiki-Codesniffer, 10Patch-For-Review: UnusedUseStatementSniff leaves behind semicolons - https://phabricator.wikimedia.org/T265740 (10thiemowmde) I did a major rewrite of this sniff in https://gerrit.wikimedia.org/r/592415. The loop at the end was intended to remove the semicolon as well, but stops to e... [17:27:53] (03PS5) 10Umherirrender: Remove semicolon in UnusedUseStatementSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) [17:30:50] (03CR) 10Umherirrender: "Combined the fix and the test changes" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) (owner: 10Umherirrender) [17:32:49] 10Scap (Scap3-MediaWiki-MVP), 10releng-201617-q4, 10scap2, 10Operations: Make scap able to depool/repool servers via the conftool API - https://phabricator.wikimedia.org/T104352 (10Aklapper) >>! In T104352#6012893, @mmodell wrote: > I think this one should be resolved now? ping - do people agree? [18:23:52] (03CR) 10Thiemo Kreuz (WMDE): [C: 03+1] "Thanks! And sorry for my rushed review before." [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/639887 (https://phabricator.wikimedia.org/T265740) (owner: 10Umherirrender)