[02:32:21] (03CR) 10Krinkle: [C: 04-1] Allow customizing allowed phpdoc tags [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/663969 (owner: 10C. Scott Ananian) [04:20:25] 10Release-Engineering-Team (Logspam), 10wikitech.wikimedia.org, 10Wikimedia-production-error: labswiki: Field 'notification_bundle_display_hash' doesn't have a default value (10.64.0.98) - https://phabricator.wikimedia.org/T262033 (10Krinkle) @Marostegui How does this compare to the main wiki clusters such a... [04:42:05] 10Continuous-Integration-Infrastructure, 10SRE, 10serviceops: legoktm can't build CI docker images without using root because he's no longer in contint-admins - https://phabricator.wikimedia.org/T275731 (10Legoktm) [04:59:30] 10VPS-project-Codesearch, 10Patch-For-Review: Codesearch: Index schema/events/* repos - https://phabricator.wikimedia.org/T275705 (10Legoktm) a:03Legoktm Will be added in the next 24 hours. [05:01:00] 10VPS-project-Codesearch: Codesearch: Index wikidata/query/* repos - https://phabricator.wikimedia.org/T275706 (10Legoktm) Services makes sense. Do we need the deploy repositories though? I was actually thinking of removing them from analytics, they were starting to get annoying (for me at least) [07:48:57] 10Continuous-Integration-Infrastructure, 10SRE, 10serviceops: legoktm can't build CI docker images without using root because he's no longer in contint-admins - https://phabricator.wikimedia.org/T275731 (10Joe) I agree that it would make sense for anyone with global root to also be able to manage CI, but it... [07:52:44] James_F: for the record, we figured out the issue :) it was https://gerrit.wikimedia.org/r/666787 [08:39:37] 10phan: Phan crash: ERROR: Phan\Exception\InvalidFQSENException: Invalid namespaced name for FQSEN '\message-bundle' - https://phabricator.wikimedia.org/T275742 (10Nikerabbit) [08:50:16] 10phan: Phan crash: ERROR: Phan\Exception\InvalidFQSENException: Invalid namespaced name for FQSEN '\message-bundle' - https://phabricator.wikimedia.org/T275742 (10Nikerabbit) It looks like Phan is incorrectly assuming that the key in HookHandlers is a class name: ` "HookHandlers": { "message-bundle": { "f... [09:01:15] 10Phabricator, 10WMSE-Wikispeech-Speech-Data-Collector-2019, 10User-Sebastian_Berlin-WMSE, 10Wikispeech-Jobrunner (Sprint): Automatically add tasks in some Wikispeech projects to column of another Wikispeech project - https://phabricator.wikimedia.org/T239390 (10Sebastian_Berlin-WMSE) @Aklapper, is there a... [10:29:37] (03PS1) 10Xqt: [tests] enable doctests with Pywikibot [integration/config] - 10https://gerrit.wikimedia.org/r/666857 (https://phabricator.wikimedia.org/T275750) [10:30:53] (03PS2) 10Xqt: [tests] enable doctests with Pywikibot [integration/config] - 10https://gerrit.wikimedia.org/r/666857 (https://phabricator.wikimedia.org/T275750) [10:55:55] 10Phabricator, 10WMSE-Wikispeech-Speech-Data-Collector-2019, 10User-Sebastian_Berlin-WMSE, 10Wikispeech-Jobrunner (Sprint): Automatically add tasks in some Wikispeech projects to column of another Wikispeech project - https://phabricator.wikimedia.org/T239390 (10Aklapper) @Sebastian_Berlin-WMSE: No, there... [11:02:54] 10Release-Engineering-Team (CI & Testing services), 10Pywikibot, 10Patch-For-Review, 10Pywikibot-tests: doctest is not processed with jenkins CI tests - https://phabricator.wikimedia.org/T275750 (10Xqt) [11:11:36] 10phan-taint-check-plugin: Phan crash: ERROR: Phan\Exception\InvalidFQSENException: Invalid namespaced name for FQSEN '\message-bundle' - https://phabricator.wikimedia.org/T275742 (10Daimona) This is a taint-check bug, not phan. [11:18:01] 10Continuous-Integration-Config, 10Code-Health, 10Discovery-Search (Current work): Ensure that SonarQube is commenting on gerrit code reviews of the Search Platform team - https://phabricator.wikimedia.org/T264873 (10Gehel) [11:21:12] 10phan-taint-check-plugin: Phan crash: ERROR: Phan\Exception\InvalidFQSENException: Invalid namespaced name for FQSEN '\message-bundle' - https://phabricator.wikimedia.org/T275742 (10Daimona) Related to T250499, taint-check doesn't have support for HookHandlers. It should copy what ExtensionProcessor does: extra... [11:22:05] (03PS1) 10Gehel: Replace the java-codehealth-patch by adding step in the standard job [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) [11:23:51] (03PS2) 10Gehel: [DNM] Replace the java-codehealth-patch by adding step in the standard job [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) [11:23:59] (03CR) 10Gehel: [C: 04-1] "[DNM] java projects need to be updated to all include sonar configuration" [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) (owner: 10Gehel) [11:25:54] 10phan-taint-check-plugin: Phan crash: ERROR: Phan\Exception\InvalidFQSENException: Invalid namespaced name for FQSEN '\message-bundle' - https://phabricator.wikimedia.org/T275742 (10Daimona) Oh, also, a temporary workaround would be to avoid using the dash in the handler name. Taint-check will still try and get... [11:33:56] (03PS3) 10Gehel: [DNM] Replace the java-codehealth-patch by adding step in the standard job [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) [12:13:31] (03CR) 10Jakob: [C: 03+1] "Looks reasonable to me!" (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/664848 (https://phabricator.wikimedia.org/T210286) (owner: 10Lucas Werkmeister (WMDE)) [12:50:13] 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10GitLab (Initialization), 10User-brennen: Define auth strategy for GitLab - https://phabricator.wikimedia.org/T274461 (10jbond) Thanks for the write up some minor comments inline > Wikitech developer accounts are how users create an LDAP obje... [13:07:25] (03PS3) 10Lucas Werkmeister (WMDE): Reinstate wikidata-query-gui-build job [integration/config] - 10https://gerrit.wikimedia.org/r/664848 (https://phabricator.wikimedia.org/T210286) [13:08:13] (03CR) 10Lucas Werkmeister (WMDE): "> maybe we can first make it experimental in zuul/layout.yaml, rather than postmerge, and then run `check experimental` on a wikidata-quer" [integration/config] - 10https://gerrit.wikimedia.org/r/664848 (https://phabricator.wikimedia.org/T210286) (owner: 10Lucas Werkmeister (WMDE)) [13:15:44] (03CR) 10Gehel: [C: 04-1] [DNM] Replace the java-codehealth-patch by adding step in the standard job [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) (owner: 10Gehel) [13:44:07] (03PS1) 10Lucas Werkmeister (WMDE): Include class names in UnusedUse messages [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/666893 [13:58:21] (03PS4) 10Gehel: [DNM] Replace the java-codehealth-patch by adding step in the standard job [integration/config] - 10https://gerrit.wikimedia.org/r/666869 (https://phabricator.wikimedia.org/T264873) [14:10:34] 10Project-Admins, 10Growth-Team: Create a Phabricator project for mentorship dashboard - https://phabricator.wikimedia.org/T275771 (10Urbanecm_WMF) [14:34:41] 10Project-Admins, 10Growth-Team: Create a Phabricator project for mentorship dashboard - https://phabricator.wikimedia.org/T275771 (10RHo) Good idea @Urbanecm_WMF - maybe `GrowthExperiments-MentorDashboard`? And any existing related tasks can be like T239234 have this tag appended to it. [15:09:51] 10Continuous-Integration-Infrastructure, 10SRE, 10serviceops: legoktm can't build CI docker images without using root because he's no longer in contint-admins - https://phabricator.wikimedia.org/T275731 (10jbond) p:05Triage→03Medium [15:26:33] 10Continuous-Integration-Infrastructure, 10SRE, 10serviceops: legoktm can't build CI docker images without using root because he's no longer in contint-admins - https://phabricator.wikimedia.org/T275731 (10thcipriani) >>! In T275731#6859724, @Joe wrote: > I agree that it would make sense for anyone with glob... [15:52:18] (03CR) 10Ahmon Dancy: [C: 03+2] test: use Debian packages for dependencies, not tox/pip [tools/scap] - 10https://gerrit.wikimedia.org/r/665316 (owner: 10Lars Wirzenius) [15:53:15] (03Merged) 10jenkins-bot: test: use Debian packages for dependencies, not tox/pip [tools/scap] - 10https://gerrit.wikimedia.org/r/665316 (owner: 10Lars Wirzenius) [15:53:48] (03CR) 10Lars Wirzenius: "> Will this have any impact on packaging via dh-python?" [tools/scap] - 10https://gerrit.wikimedia.org/r/665316 (owner: 10Lars Wirzenius) [16:02:33] 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10Patch-For-Review, 10Release, 10Train Deployments: 1.36.0-wmf.32 deployment blockers - https://phabricator.wikimedia.org/T274936 (10dancy) [16:11:38] 10Continuous-Integration-Infrastructure, 10SRE, 10serviceops, 10Patch-For-Review: legoktm can't build CI docker images without using root because he's no longer in contint-admins - https://phabricator.wikimedia.org/T275731 (10jbond) > would adding *contint_roots_members explicitly to contint-admin with a c... [16:13:47] (03PS1) 10Dduvall: Make deletion of temporary files configurable [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666944 [16:19:39] 10Release-Engineering-Team (Deployment services), 10Phatality, 10observability: An alternative to Phatality extending Kibana plugin api - https://phabricator.wikimedia.org/T275788 (10mmodell) [16:48:58] 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10GitLab (Initialization), 10User-brennen: Remove Speed & Function blockers for GitLab work - https://phabricator.wikimedia.org/T274458 (10brennen) [17:10:01] 10Continuous-Integration-Config, 10Code-Health-Metrics, 10Product-Infrastructure-Team-Backlog, 10Services: Enable codehealth pipeline for more node services maintained by Product Infrastructure - https://phabricator.wikimedia.org/T240989 (10MSantos) [17:30:54] (03PS2) 10Dduvall: Make deletion of temporary files configurable [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666944 [17:35:49] (03CR) 10Ahmon Dancy: [C: 03+2] Make deletion of temporary files configurable [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666944 (owner: 10Dduvall) [17:36:28] (03Merged) 10jenkins-bot: Make deletion of temporary files configurable [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666944 (owner: 10Dduvall) [18:02:10] 10LibUp: libup: switch to composer v2 API - https://phabricator.wikimedia.org/T275802 (10Legoktm) [18:16:47] (03PS1) 10Dduvall: Fix blubber application/yaml media-type detection [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666972 [18:26:01] (03CR) 10Ahmon Dancy: [C: 03+2] Fix blubber application/yaml media-type detection [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666972 (owner: 10Dduvall) [18:26:38] (03Merged) 10jenkins-bot: Fix blubber application/yaml media-type detection [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/666972 (owner: 10Dduvall) [19:41:15] thcipriani: hey.. i've updated the mw-client-error board to have a specific widget for group0 wikis. When triaging you can now click enable all to filter out all triaged things https://usercontent.irccloud-cdn.com/file/BAv6kN24/Screen%20Shot%202021-02-25%20at%2011.41.00%20AM.png [19:41:33] above should help with train log triage [20:01:34] Jdlrobson: nice :) [20:02:01] 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10Patch-For-Review, 10Release, 10Train Deployments: 1.36.0-wmf.32 deployment blockers - https://phabricator.wikimedia.org/T274936 (10dancy) [20:04:05] the first time I learned about the "CHANGE ALL FILTERS" button was yesterday [20:04:33] don't even know if that's a new feature for kibana [20:14:22] 10Release-Engineering-Team (Logspam), 10JavaScript, 10MediaWiki-Interface (Tables), 10Readers-Web-Backlog (Kanbanana-FY-2020-21), 10Wikimedia-production-error: Some tables cannot be sorted (TypeError: $nextRows[i] is undefined in table sorting / Uncaught TypeErr... - https://phabricator.wikimedia.org/T265503 [22:01:18] I could use help with changing the docker-registry credentials in jenkins: https://phabricator.wikimedia.org/T275559#6862829 [22:01:27] mostly my question is, can I just do it? or are there other steps? [23:05:04] legoktm: Maybe that's sufficient? [23:05:11] * James_F shrugs in unhelpfulness. [23:36:45] I think there is a private repo somewhere to add the credentials to also...I'll see if I can get more info [23:39:42] 10Deployments, 10Release-Engineering-Team-TODO: L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10dduvall) [23:40:33] 10Deployments, 10Release-Engineering-Team: L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10dduvall) [23:45:06] 10Deployments, 10Release-Engineering-Team: L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10Dzahn) The `/usr/local/bin/scap-master-sync` command syncs `/srv/mediawiki-staging` and `/srv/patches` _from_ another deployment server. The following rsync options ar... [23:48:12] 10Deployments, 10Release-Engineering-Team: L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10Dzahn) This came up in the context of creating new deployment servers on buster (deploy1002, deploy2002) and wanting to keep all the /srv/ data in sync before switching... [23:52:05] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO (2021-01-01 to 2021-03-31 (Q3)), 10serviceops, 10Patch-For-Review: Replace production deployment servers and update them to Buster - https://phabricator.wikimedia.org/T265963 (10Dzahn) [23:52:10] 10Deployments, 10Release-Engineering-Team: L10n cache files building up on backup deploy hosts - https://phabricator.wikimedia.org/T275826 (10Dzahn) [23:52:12] 10Scap: scap deploy --init on deployment server fails on first puppet run - https://phabricator.wikimedia.org/T257317 (10Dzahn) [23:54:22] 10Release-Engineering-Team (Logspam), 10JavaScript, 10MediaWiki-Interface (Tables), 10Patch-For-Review, and 2 others: Some tables cannot be sorted (TypeError: $nextRows[i] is undefined in table sorting / Uncaught TypeError: Cannot read property 'type' of undefine... - https://phabricator.wikimedia.org/T265503