[01:03:02] 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, 10User-DannyS712, 10ci-test-error (WMF-deployed Build Failure): phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Ammarpad) Modern skin: https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/Mode... [04:56:43] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10Ammarpad) HTTP Error 400: Bad Request because the url is malformed. It's mixing revid with a page it does... [06:31:37] 10Phabricator, 10Operations, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10jcrespo) [06:33:42] 10Phabricator, 10Operations, 10Patch-For-Review, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10jcrespo) [06:44:28] 10Phabricator, 10Operations, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10DannyS712) [06:55:32] 10Phabricator, 10Operations, 10Patch-For-Review, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10jcrespo) p:05Triage→03High 812 reports were sent today 1st June. The scheduling seem... [07:27:45] 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, 10User-DannyS712, 10ci-test-error (WMF-deployed Build Failure): phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Nikerabbit) p:05High→03Unbreak! Also ContentTranslation: https://integrati... [08:29:17] 10Release-Engineering-Team, 10Wikimedia-Extension-setup, 10CPT Initiatives (API Gateway): Determine how to host Chameleon skin and Bootstrap extension in gerrit while primary development happens in github - https://phabricator.wikimedia.org/T252463 (10DannyS712) LibUp would also need to be disabled: https://... [09:49:18] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10MarcoAurelio) This is a trace from Logstash-Beta I spotted and reported here. I did not insert anything.... [09:58:55] 10Phabricator, 10Operations, 10Security-Team, 10Security, 10User-jbond: Adjust onboarding/offboarding logic to accommodate changes to #security (now acl*security) - https://phabricator.wikimedia.org/T245771 (10Aklapper) [10:13:36] 10Differential, 10Phabricator, 10Wikibugs, 10WorkType-NewFunctionality: Create conduit method to query the feed and return records with relevant details populated instead of just a bunch of phids - https://phabricator.wikimedia.org/T123417 (10Aklapper) 05Stalled→03Open I don't see who or what (task?) e... [10:13:39] 10Differential, 10Phabricator: Phabricator does not provide an API to get Differential transaction data, similar to maniphest.gettasktransactions - https://phabricator.wikimedia.org/T123416 (10Aklapper) [10:31:40] 10Release-Engineering-Team, 10Wikimedia-Extension-setup, 10CPT Initiatives (API Gateway): Determine how to host Chameleon skin and Bootstrap extension in gerrit while primary development happens in github - https://phabricator.wikimedia.org/T252463 (10Reedy) >>! In T252463#6181032, @DannyS712 wrote: > LibUp... [10:57:31] 10Phabricator, 10Operations, 10Security-Team, 10Security, 10User-jbond: Adjust onboarding/offboarding logic to accommodate changes to #security (now acl*security) - https://phabricator.wikimedia.org/T245771 (10jbond) 05Open→03Resolved a:03jbond Resolve as per Moritz comment [10:57:38] 10Phabricator, 10Developer-Advocacy (Jul-Sep 2020), 10Epic: (Semi)automatically close Phabricator tickets with status "stalled" after a while - https://phabricator.wikimedia.org/T252522 (10Aklapper) For the records, right now we have 317 stalled tasks which have been stalled for longer than 12 months. I agr... [11:03:14] 10Phabricator, 10Developer-Advocacy (Jul-Sep 2020), 10Epic: (Semi)automatically close Phabricator tickets with status "stalled" after a while - https://phabricator.wikimedia.org/T252522 (10Nikerabbit) [11:22:12] 10Phabricator, 10Operations, 10Security-Team, 10Patch-For-Review, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10chasemp) a:03chasemp Thanks for disabling @jcrespo, sorry for the av... [11:38:08] 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, 10User-DannyS712, 10ci-test-error (WMF-deployed Build Failure): phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) Hmmmm can we have an emergency run of LibUp? Several repos were updat... [12:19:43] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10DannyS712) >>! In T254079#6181580, @Daimona wrote: > Hmmmm can we have an emergency run of LibUp? Several repos were upda... [12:47:12] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Scap: Document that "scap test-patches" may fail, when "scap apply-patches" would work - https://phabricator.wikimedia.org/T253999 (10LarsWirzenius) 05Open→03Resolved [12:52:04] (03CR) 10Lars Wirzenius: [C: 03+2] "I was thinkin I'd do this while adding integration tests for everything, but merging there doesn't prevent me from adding tests later. So," [tools/scap] - 10https://gerrit.wikimedia.org/r/599973 (https://phabricator.wikimedia.org/T248490) (owner: 10Jforrester) [12:53:54] 10Continuous-Integration-Config, 10phan-taint-check-plugin: Drop taint-check jobs for CI once migration to mediawiki-tools-phan is complete - https://phabricator.wikimedia.org/T254152 (10Daimona) [12:54:09] (03Merged) 10jenkins-bot: Move scap plugins from mediawiki-config into this repo [tools/scap] - 10https://gerrit.wikimedia.org/r/599973 (https://phabricator.wikimedia.org/T248490) (owner: 10Jforrester) [13:10:21] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) [13:10:42] PROBLEM - Parsoid on deployment-parsoid11 is CRITICAL: connect to address 172.16.1.115 and port 8000: Connection refused [13:15:41] 10Gerrit: Gerrit 3.x upgrade - https://phabricator.wikimedia.org/T254158 (10QChris) [13:16:11] 10Gerrit: Gerrit 3.x upgrade - https://phabricator.wikimedia.org/T254158 (10QChris) [13:16:15] 10Gerrit, 10Patch-For-Review: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) [13:25:22] 10Release-Engineering-Team-TODO, 10Core Platform Team, 10CPT Initiatives (API Gateway), 10Patch-For-Review, and 4 others: New Public Wiki for the API Portal - https://phabricator.wikimedia.org/T246945 (10WDoranWMF) >>! In T246945#6178987, @Ladsgroup wrote: > okay, to recap: > - The dns and puppet patches... [13:49:53] 10Gerrit, 10Patch-For-Review: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) Code to run backups is in place. Thanks @dzahn! We now need to wait a week for the backup to run. [13:51:43] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) [13:51:47] 10Continuous-Integration-Config, 10LibUp, 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10phan, and 3 others: Upgrade phan to 0.10.2 and remove phan-taint-check-plugin - https://phabricator.wikimedia.org/T248630 (10Daimona) [13:51:51] 10Continuous-Integration-Config, 10LibUp, 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10phan, and 3 others: Upgrade phan to 0.10.2 and remove phan-taint-check-plugin - https://phabricator.wikimedia.org/T248630 (10Daimona) [13:51:53] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) [13:52:05] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) (Marking this as blocked on the taint-check upgrade) [13:56:37] 10Gerrit, 10Patch-For-Review: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10jcrespo) > We now need to wait a week for the backup to run We don't need to wait, I can (or any other SRE can) run it now, but I have not idea or context what is the end goal here. That is why I... [14:15:58] 10Release-Engineering-Team (Pipeline), 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Patch-For-Review, 10Release Pipeline (Blubber): Blubber policy should be verified after expansion - https://phabricator.wikimedia.org/T248927 (10DC_Slagel) Hi thcipriani, dduval, Just checking in on patch-s... [14:34:48] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) The end goal is to be able to avoid data loss if Gerrit hardware dies. Gerrit currently stores data in three places: 1. The main git repositories, 2. the MariaDB database, and 3. (since git in not the best pl... [14:37:07] 10Gerrit: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10QChris) [14:37:45] 10Gerrit: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10QChris) [14:37:47] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) [14:38:48] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10QChris) >>! In T254155#6181909, @QChris wrote: > [...] Let me file a ticket for [initial backup run]. The ticket is at T254162 [15:06:14] 10Phabricator, 10Operations, 10Security-Team, 10Patch-For-Review, 10Security: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10chasemp) p:05High→03Medium [15:09:39] 10Phabricator, 10Operations, 10Peek, 10Security-Team, 10Patch-For-Review: peek is incorrectly configured to run every minute every 1st of the month, creating large amounts of cronspam - https://phabricator.wikimedia.org/T254127 (10chasemp) [15:18:46] 10phan-taint-check-plugin, 10MediaWiki-extensions-Translate, 10Security-Team, 10Patch-For-Review, and 3 others: SpecialLanguageStats.php fails phan tests due to possible XSS - https://phabricator.wikimedia.org/T253383 (10sbassett) Leaving this open until https://gerrit.wikimedia.org/r/598228 gets merged, o... [15:43:00] 10Release-Engineering-Team-TODO, 10Core Platform Team, 10CPT Initiatives (API Gateway), 10Patch-For-Review, and 4 others: New Public Wiki for the API Portal - https://phabricator.wikimedia.org/T246945 (10apaskulin) [15:44:00] 10Release-Engineering-Team-TODO, 10Core Platform Team, 10CPT Initiatives (API Gateway), 10Patch-For-Review, and 4 others: New Public Wiki for the API Portal - https://phabricator.wikimedia.org/T246945 (10apaskulin) Thanks, @Ladsgroup! I've updated the project name to Wikimedia API Portal. [15:45:49] 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team, 10Pywikibot: Jenkins can't merge patchsets sometimes - https://phabricator.wikimedia.org/T252310 (10Xqt) 05Open→03Resolved a:03Xqt [15:45:59] 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team, 10Pywikibot: Jenkins can't merge patchsets sometimes - https://phabricator.wikimedia.org/T252310 (10Xqt) a:05Xqt→03None [16:10:48] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10greg) >>! In T254155#6181858, @jcrespo wrote: >> We now need to wait a week for the backup to run > > We don't need to wait, I can (or any other SRE can) run it now, but I have not idea or context what is the end goal... [16:11:16] 10Gerrit: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10greg) +1, let's kick off an initial backup. [16:12:40] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10jcrespo) a:03jcrespo [16:12:47] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10jcrespo) p:05Triage→03Medium [16:18:36] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10jcrespo) Running: ` 232157 Back Full 0 0 gerrit1001.wikimedia.org-Hourly-Sun-production-gerrit-repo-data is running ` [16:19:59] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Release, 10Train Deployments: 1.35.0-wmf.36 deployment blockers - https://phabricator.wikimedia.org/T254173 (10thcipriani) a:03jeena [16:20:21] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Release, 10Train Deployments: 1.35.0-wmf.36 deployment blockers - https://phabricator.wikimedia.org/T254173 (10thcipriani) p:05Triage→03Medium [16:20:49] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10jcrespo) I have a few things I would like to test/ask about this. * First, I would like to test a recovery, if possible to make sure the backup worked as intended. * I also have a few questions regar... [16:20:51] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Release, 10Train Deployments: 1.35.0-wmf.37 deployment blockers - https://phabricator.wikimedia.org/T254174 (10thcipriani) p:05Triage→03Medium a:03LarsWirzenius [16:21:37] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Release, 10Train Deployments: 1.35.0-wmf.38 deployment blockers - https://phabricator.wikimedia.org/T254175 (10thcipriani) p:05Triage→03Medium a:03brennen [16:22:15] 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Release, 10Train Deployments: 1.35.0-wmf.39 deployment blockers - https://phabricator.wikimedia.org/T254176 (10thcipriani) p:05Triage→03Medium a:03hashar [16:27:20] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10greg) >>! In T254162#6182368, @jcrespo wrote: > * I also have a few questions regarding recovery of gerrit- in terms of consistency and disaster recovery model, but not sure if this task would be the r... [16:34:54] 10Gerrit: Make sure we backup Gerrit's LFS data - https://phabricator.wikimedia.org/T254155 (10jcrespo) ` *llist jobid=232157 JobId: 232,157 Job: gerrit1001.wikimedia.org-Hourly-Sun-production-gerrit-repo-data.2020-06-01_16.14.26_23 Name: gerrit1001.wikimedia.org-Hourly-Su... [16:35:35] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10Ottomata) This is due to a change in {T249261}. I will look into it! [16:40:00] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10jcrespo) ` *llist jobid=232157 JobId: 232,157 Job: gerrit1001.wikimedia.org-Hourly-Sun-production-gerrit-repo-data.2020-06-01_16.14.26_23 Name: gerrit1001.wikime... [16:43:17] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10Ottomata) a:03Ottomata [16:45:59] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10Ottomata) Although, I'm not sure what would be trying to reach searchsatisfaction on meta. The logstash-... [16:54:23] (03PS1) 10Alexandros Kosiaris: Add operations/debs/kubeyaml [integration/config] - 10https://gerrit.wikimedia.org/r/601373 [17:00:31] (03PS2) 10Jforrester: layout: Add CI for operations/debs/kubeyaml using debian-glue [integration/config] - 10https://gerrit.wikimedia.org/r/601373 (owner: 10Alexandros Kosiaris) [17:01:02] (03CR) 10Jforrester: [C: 03+2] layout: Add CI for operations/debs/kubeyaml using debian-glue [integration/config] - 10https://gerrit.wikimedia.org/r/601373 (owner: 10Alexandros Kosiaris) [17:01:38] (03CR) 10Jforrester: "Yeah, let's not do this, please." [integration/config] - 10https://gerrit.wikimedia.org/r/600403 (https://phabricator.wikimedia.org/T254070) (owner: 10DannyS712) [17:02:18] (03Merged) 10jenkins-bot: layout: Add CI for operations/debs/kubeyaml using debian-glue [integration/config] - 10https://gerrit.wikimedia.org/r/601373 (owner: 10Alexandros Kosiaris) [17:02:30] !log layout: Add CI for operations/debs/kubeyaml using debian-glue [17:02:33] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [17:05:04] (03PS2) 10Jforrester: jjb: fail docker run command when it is piped [integration/config] - 10https://gerrit.wikimedia.org/r/599281 (https://phabricator.wikimedia.org/T253731) (owner: 10Hashar) [17:05:55] (03PS3) 10Jforrester: jjb: fail docker run command when it is piped [integration/config] - 10https://gerrit.wikimedia.org/r/599281 (https://phabricator.wikimedia.org/T253731) (owner: 10Hashar) [17:07:23] 10LibUp: LibUp should update phan-taint-check-plugin - https://phabricator.wikimedia.org/T232935 (10Daimona) 05Open→03Declined Declining in favour of T248630. [17:09:35] 10Continuous-Integration-Config, 10phan, 10TestMe: False positive PhanUndeclaredVariable in try-catch - https://phabricator.wikimedia.org/T207835 (10Daimona) [17:33:16] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/599281 (https://phabricator.wikimedia.org/T253731) (owner: 10Hashar) [17:34:14] (03Merged) 10jenkins-bot: jjb: fail docker run command when it is piped [integration/config] - 10https://gerrit.wikimedia.org/r/599281 (https://phabricator.wikimedia.org/T253731) (owner: 10Hashar) [17:35:12] 10Continuous-Integration-Config, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Patch-For-Review: releng/operations-puppet failed to fail on a git error - https://phabricator.wikimedia.org/T253731 (10Jdforrester-WMF) 05Open→03Resolved [17:40:20] (03PS1) 10Ottomata: EventBus now depends on EventStreamConfig [integration/config] - 10https://gerrit.wikimedia.org/r/601381 (https://phabricator.wikimedia.org/T229863) [17:41:05] (03CR) 10Ppchelko: [C: 03+1] EventBus now depends on EventStreamConfig [integration/config] - 10https://gerrit.wikimedia.org/r/601381 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [17:45:09] (03CR) 10Jforrester: [C: 03+2] EventBus now depends on EventStreamConfig [integration/config] - 10https://gerrit.wikimedia.org/r/601381 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [17:45:22] ty! [17:45:23] 10Beta-Cluster-Infrastructure, 10Core Platform Team, 10CPT Initiatives (API Gateway), 10User-Urbanecm, 10Wiki-Setup (Create): Create new wiki on the beta cluster to test API Portal - https://phabricator.wikimedia.org/T254185 (10CCicalese_WMF) [17:46:06] (03Merged) 10jenkins-bot: EventBus now depends on EventStreamConfig [integration/config] - 10https://gerrit.wikimedia.org/r/601381 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [17:47:45] !log Zuul: EventBus now depends on EventStreamConfig T229863 [17:47:47] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [17:47:47] T229863: Refactor EventBus mediawiki configuration - https://phabricator.wikimedia.org/T229863 [17:59:39] 10Project-Admins: Creating Architecture Team project (Group Type) - https://phabricator.wikimedia.org/T254187 (10kchapman) [18:07:23] (03PS1) 10Ottomata: EventBus should have EventStreamConfig in dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/601384 (https://phabricator.wikimedia.org/T229863) [18:08:02] (03CR) 10Ottomata: "Assuming this is correct, since EventLogging has this in $dependencies too." [integration/config] - 10https://gerrit.wikimedia.org/r/601384 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [18:56:29] 10LibUp, 10Patch-For-Review, 10User-DannyS712: Library upgrader tests not always being run - https://phabricator.wikimedia.org/T254070 (10Jdforrester-WMF) >>! In T254070#6179822, @Reedy wrote: > I think the obvious issue this time around is there is seemingly a lot more that aren't/weren't automatically +2d.... [18:59:32] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Jdforrester-WMF) >>! In T254079#6181580, @Daimona wrote: > Hmmmm can we have an emergency run of LibUp? Several repos wer... [19:21:06] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Jdlrobson) [19:25:07] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) >>! In T254079#6182907, @Jdforrester-WMF wrote: >>>! In T254079#6181580, @Daimona wrote: >> Hmmmm can we have an... [19:37:32] 10Gerrit, 10Operations: Initial backup run for Gerrit LFS data - https://phabricator.wikimedia.org/T254162 (10QChris) @jcrespo Thanks for the initial run! > We should try restoring it, give me a path you have access to to do so. gerrit1002 (gerrit-test) is a bit scarce on disk space, so what about gerrit2001... [19:43:42] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Jdforrester-WMF) >>! In T254079#6182958, @Daimona wrote: >>>! In T254079#6182907, @Jdforrester-WMF wrote: >>>>! In T25407... [19:52:17] 10LibUp, 10phan-taint-check-plugin, 10MW-1.35-notes (1.35.0-wmf.35; 2020-06-02), 10Patch-For-Review, and 2 others: phan-seccheck typeerror - https://phabricator.wikimedia.org/T254079 (10Daimona) It's fine, I'll check them manually tomorrow (especially XSSs) [20:07:47] 10LibUp, 10phan, 10Patch-For-Review: Add "composer phan" type command to composer.json - https://phabricator.wikimedia.org/T253477 (10Jdforrester-WMF) a:03Jdforrester-WMF [20:09:35] 10LibUp, 10Patch-For-Review, 10User-DannyS712: Library upgrader tests not always being run - https://phabricator.wikimedia.org/T254070 (10Legoktm) >>! In T254070#6179673, @DannyS712 wrote: > Could libup be told to post a comment (same account) or give a +1 instead of nothing? If shouldn't need to be a separa... [20:23:22] (03Abandoned) 10Jforrester: Whitelist: Add library upgrader [integration/config] - 10https://gerrit.wikimedia.org/r/600403 (https://phabricator.wikimedia.org/T254070) (owner: 10DannyS712) [20:30:12] (03PS2) 10Jforrester: layout: [wikidata/query/gui-deploy] Drop unused experimental node 6 CI [integration/config] - 10https://gerrit.wikimedia.org/r/599971 (https://phabricator.wikimedia.org/T254027) [20:30:19] (03CR) 10Jforrester: [C: 03+2] layout: [wikidata/query/gui-deploy] Drop unused experimental node 6 CI [integration/config] - 10https://gerrit.wikimedia.org/r/599971 (https://phabricator.wikimedia.org/T254027) (owner: 10Jforrester) [20:31:20] (03Merged) 10jenkins-bot: layout: [wikidata/query/gui-deploy] Drop unused experimental node 6 CI [integration/config] - 10https://gerrit.wikimedia.org/r/599971 (https://phabricator.wikimedia.org/T254027) (owner: 10Jforrester) [20:31:41] (03CR) 10Jforrester: [C: 03+2] EventBus should have EventStreamConfig in dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/601384 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [20:31:58] !log Zuul: EventBus now depends on EventStreamConfig for non-phan too T229863 [20:32:00] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [20:32:00] T229863: Refactor EventBus mediawiki configuration - https://phabricator.wikimedia.org/T229863 [20:32:12] thank you again! [20:32:30] No worries. Sorry configuring this is such a mess. :-( [20:32:39] (03Merged) 10jenkins-bot: EventBus should have EventStreamConfig in dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/601384 (https://phabricator.wikimedia.org/T229863) (owner: 10Ottomata) [20:33:39] 10Continuous-Integration-Config, 10Discovery-Search, 10Wikidata, 10Wikidata-Campsite, and 2 others: Can we scrap the `wikidata/query/gui-deploy` repo, or is it being used? - https://phabricator.wikimedia.org/T254027 (10Jdforrester-WMF) 05Open→03Declined [20:44:25] (03CR) 10Jforrester: "This wasn't ready to land, AFAICT; it's now broken master of a deployed extension. *sighs*" [integration/config] - 10https://gerrit.wikimedia.org/r/599963 (https://phabricator.wikimedia.org/T254031) (owner: 10Physikerwelt) [20:46:38] (03PS1) 10Jforrester: layout: [mediawiki/extensions/Math] Disable API tests, not passing yet [integration/config] - 10https://gerrit.wikimedia.org/r/601427 (https://phabricator.wikimedia.org/T254031) [20:48:13] (03CR) 10Jforrester: [C: 03+2] layout: [mediawiki/extensions/Math] Disable API tests, not passing yet [integration/config] - 10https://gerrit.wikimedia.org/r/601427 (https://phabricator.wikimedia.org/T254031) (owner: 10Jforrester) [20:49:19] (03Merged) 10jenkins-bot: layout: [mediawiki/extensions/Math] Disable API tests, not passing yet [integration/config] - 10https://gerrit.wikimedia.org/r/601427 (https://phabricator.wikimedia.org/T254031) (owner: 10Jforrester) [20:49:45] !log Zuul: Disabled new mediawiki-quibble-apitests-vendor-docker for Math as it doesn't pass T254031 [20:49:48] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [20:49:49] T254031: Enable mediawiki-quibble-apitests-vendor-docker for extension Math - https://phabricator.wikimedia.org/T254031 [20:49:58] 10Continuous-Integration-Config, 10Math, 10Patch-For-Review: Enable mediawiki-quibble-apitests-vendor-docker for extension Math - https://phabricator.wikimedia.org/T254031 (10Jdforrester-WMF) 05Resolved→03Stalled Undeployed. This job isn't passing yet. Please un-mark as stalled if and when the experiment... [20:54:07] 10LibUp, 10MediaWiki-Configuration: Create and maintain/encourage/enforce a sort order for extension/skin.json - https://phabricator.wikimedia.org/T252567 (10Legoktm) https://github.com/json-schema-org/json-schema-vocabularies/issues/7 is the schema tracking ticket, but I don't think we should wait for that.... [20:57:28] 10LibUp: Use packaged composer - https://phabricator.wikimedia.org/T249525 (10Legoktm) My plan right now is just to wait until the bullseye release and switch to the packaged version then. If we want to get rid of integration/composer sooner, we could look into switching to the phar. [20:58:22] 10LibUp, 10MediaWiki-Configuration: Create and maintain/encourage/enforce a sort order for extension/skin.json - https://phabricator.wikimedia.org/T252567 (10Reedy) Would including things like alphasorting hooks at the same time (ie sort the value of Hooks by the key of the array of values) be too noisy? Or is... [20:58:33] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10MarcoAurelio) >>! In T254058#6182454, @Ottomata wrote: > Although, I'm not sure what would be trying to r... [20:59:09] 10LibUp: LibUp fails to run stylelint after upgrade when grunt is missing - https://phabricator.wikimedia.org/T248801 (10Legoktm) p:05Triage→03High This is now causing errors on mediawiki/skins/Vector. [21:00:38] 10Continuous-Integration-Config, 10Math, 10Core Platform Team Workboards (Clinic Duty Team): Enable mediawiki-quibble-apitests-vendor-docker for extension Math - https://phabricator.wikimedia.org/T254031 (10Pchelolo) [21:01:04] 10Beta-Cluster-Infrastructure, 10Analytics, 10Analytics-EventLogging, 10Wikimedia-production-error: [beta] EventLogging trying to fetch wrong Schema title - https://phabricator.wikimedia.org/T254058 (10Ottomata) Ah! I see. This is eventlogging-processor trying to parse an event that it shouldn't. https:/... [21:01:49] hey ottomata - logstash-beta link working now? :-) [21:02:32] 10LibUp, 10Patch-For-Review, 10User-DannyS712: Library upgrader tests not always being run - https://phabricator.wikimedia.org/T254070 (10Reedy) Cases like this are a good reason why it'd be useful if CI checks were run *if* C+2 wasn't explicitly passed >>! In T254079#6183004, @Jdforrester-WMF wrote: > Note... [21:02:36] title=searchsatisfaction does not exist; SearchSatisfaction do [21:02:44] But not sure where that is configured [21:03:00] hauskatze: t hank you! [21:03:17] it looks like eventlogging-processor server side in beta is using some configuration I didn't expect it to [21:03:20] will look into that tomorrow [21:03:22] thank you very much! [21:03:36] ottomata: np, just running like a headless chicken looking where that thing is configured heh [21:03:43] me I mean [21:03:58] thanks for looking into it [21:18:43] 10Continuous-Integration-Config, 10Math, 10Core Platform Team Workboards (Clinic Duty Team): Enable mediawiki-quibble-apitests-vendor-docker for extension Math - https://phabricator.wikimedia.org/T254031 (10Pchelolo) The search tests are failing because Math depends on Wikibase in CI, which in turn brings in... [21:40:43] 10LibUp, 10User-DannyS712: Library upgrader tests not always being run - https://phabricator.wikimedia.org/T254070 (10DannyS712) >>! In T254070#6183078, @Legoktm wrote: >>>! In T254070#6179673, @DannyS712 wrote: >> Could libup be told to post a comment (same account) or give a +1 instead of nothing? If shouldn... [22:13:23] legoktm: Thanks; have given up trying to work out why the phan-seccheck removal isn't passing. [22:14:22] James_F: I think you have a typo [22:14:27] Eurgh. [22:14:37] If it's that, after the amount of time I've been staring at it… [22:14:42] .get_extra('/phan-taint-check... [22:15:04] why is the / there? [22:15:12] Oh, FFS. [22:15:46] Because daimona insisted I should call it mediawiki/phan-taint-check and then magically know to strip the first bit, and I typoed when reverting my change there. [22:16:37] Yeah, now passes. [22:16:38] Oy. [22:16:41] Thank you, legoktm. :-) [22:16:45] :D [22:17:10] (It's amazing how much more productive I can be in LibUp now that I can actually run the tests locally.) [22:17:42] 10LibUp, 10phan, 10Patch-For-Review: Add "composer phan" type command to composer.json - https://phabricator.wikimedia.org/T253477 (10Jdforrester-WMF) 05Open→03Resolved [22:18:24] legoktm: Want to merge it and fix the universe? ;-) [22:28:59] James_F: it's deploying right now [22:29:18] Awesome. [22:31:36] James_F: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageViewInfo/+/601455 [22:31:54] Nice. [22:32:08] Always good to get confirmation that it's real. :-) [22:32:38] legoktm: Worth restarting the run so that we catch the As–Os? [22:32:49] it should loop [22:33:05] Oh, I thought it was one-and-done. Cool. [22:35:30] I queued another run of everything so it'll start processing everything again [22:35:37] Excellent. [22:35:42] but it won't remove the "extra" key if there are no packages to change [22:35:53] Oh, right, 'cos it's only a misc fix. [22:35:55] er, it won't submit a patch to *just* remove the "extra" key [22:35:56] yeah [22:36:37] Once everything is running phan 0.10.2 I'm just going to drop the seccheck job, so it's not the end of the world. [22:36:56] https://codesearch.wmflabs.org/deployed/?q=mediawiki%2Fmediawiki-phan-config%22%3A%20%220%5C.9&i=nope&files=composer%5C.json&repos= still has 144 repos to go. [22:37:14] (Delayed from reality, but not much.) [22:51:31] 10LibUp: LibUp fails to run stylelint after upgrade when grunt is missing - https://phabricator.wikimedia.org/T248801 (10Legoktm) 05Open→03Resolved a:03Legoktm