[01:28:36] (03PS1) 10Jeena Huneidi: Use new helm chart museum [releng/local-charts] - 10https://gerrit.wikimedia.org/r/639338 [01:29:46] (03CR) 10Jeena Huneidi: [V: 03+2 C: 03+2] Use new helm chart museum [releng/local-charts] - 10https://gerrit.wikimedia.org/r/639338 (owner: 10Jeena Huneidi) [01:47:38] (03PS1) 10Jeena Huneidi: Linkrecommendation service example [releng/local-charts] - 10https://gerrit.wikimedia.org/r/639347 [01:48:39] (03PS2) 10Jeena Huneidi: Linkrecommendation service example [releng/local-charts] - 10https://gerrit.wikimedia.org/r/639347 [02:20:51] (03PS3) 10Jeena Huneidi: Linkrecommendation service example [releng/local-charts] - 10https://gerrit.wikimedia.org/r/639347 [02:34:02] 10Deployments, 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Sustainability (Incident Followup), 10User-brennen: Avoid unfinished train deploys over holidays, weekends, or other stretches of no-deploy days - https://phabricator.wikimedia.org/T260401 (10Urbanecm) I a... [02:51:44] 10Project-Admins: IgboWiki-UG-Projects - https://phabricator.wikimedia.org/T266781 (10Urbanecm) I would personally prefer something like "Igbo Wikimedians User Group" to make its meaning obvious to everyone. To answer the question: >>! In T266781#6588830, @50019062 wrote: > Hello @Aklapper, > Sorry for the dist... [03:07:57] 10Release-Engineering-Team (Logspam), 10CommonsMetadata, 10Structured Data Engineering, 10Structured-Data-Backlog, 10User-DannyS712: CommonsMetadata bad wfTimestamp call - https://phabricator.wikimedia.org/T267033 (10cscott) I've looked into this. If you install CommonsMetadata locally, and set up some c... [05:41:47] 10Project-Admins: Igbo-Wikimedians-User-Group-Projects - https://phabricator.wikimedia.org/T266781 (1050019062) [05:52:12] 10Project-Admins: Igbo-Wikimedians-User-Group-Projects - https://phabricator.wikimedia.org/T266781 (1050019062) >>! In T266781#6604836, @Urbanecm wrote: > I would personally prefer something like "Igbo Wikimedians User Group" to make its meaning obvious to everyone. To answer the question: > I see your point, t... [07:25:28] (03CR) 10Thiemo Kreuz (WMDE): Report particularly confusing assertNotEquals( false ) (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/576276 (owner: 10Thiemo Kreuz (WMDE)) [08:08:24] 10Gerrit: Gerrit is giving tips on how to write comments, make sure it does not send our comments to some third party - https://phabricator.wikimedia.org/T258102 (10Zacharystokes) Gerrit is giving tips on how to write comments, make sure it does not send our comments to some third party whole methods are here. Y... [08:14:41] ^spam if someone wants to wave the magic wand over it [08:14:46] ac is disabled [09:11:27] 10Gerrit, 10VPS-Projects: Gerrit test instance is down - https://phabricator.wikimedia.org/T240933 (10Ammarpad) 05Openβ†’03Declined [09:19:13] !log fixed kafka broker names (s/.eqiad.wmflabs/.eqiad1.wikimedia.cloud/g) in deployment-prep's hiera global config to unblock puppet [09:19:14] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [09:19:42] I also restarted a kafka jumbo broker to check that everything is fine, lemme know if you see issues [10:00:09] (03PS1) 10Awight: PEP 8: private members for backend [integration/quibble] - 10https://gerrit.wikimedia.org/r/639458 [10:00:11] (03PS1) 10Awight: PEP 8: private members for cmd [integration/quibble] - 10https://gerrit.wikimedia.org/r/639459 [10:00:13] (03PS1) 10Awight: PEP 8: private members in commands [integration/quibble] - 10https://gerrit.wikimedia.org/r/639460 [10:00:15] (03PS1) 10Awight: PEP 8: private members for gitchangedinhead [integration/quibble] - 10https://gerrit.wikimedia.org/r/639461 [10:00:17] (03PS1) 10Awight: PEP 8: private members for mediawiki registry [integration/quibble] - 10https://gerrit.wikimedia.org/r/639462 [10:00:19] (03PS1) 10Awight: PEP 8: private members for util [integration/quibble] - 10https://gerrit.wikimedia.org/r/639463 [10:00:21] (03PS1) 10Awight: PEP 8: private members for zuul [integration/quibble] - 10https://gerrit.wikimedia.org/r/639464 [10:00:53] (03Abandoned) 10Awight: PEP 8: underscore prefix for internal functions [integration/quibble] - 10https://gerrit.wikimedia.org/r/588177 (owner: 10Awight) [10:02:20] (03Abandoned) 10Awight: Mark private interfaces [integration/quibble] - 10https://gerrit.wikimedia.org/r/638704 (owner: 10Awight) [11:25:57] 10Project-Admins: Igbo-Wikimedians-User-Group-Projects - https://phabricator.wikimedia.org/T266781 (10Aklapper) Urbanecm is right, see https://www.mediawiki.org/wiki/Phabricator/Creating_and_renaming_projects for naming guidelines (we already have some projects here like #Wikimedia-Cameroon-User-Group or #Wikim... [11:26:17] 10Project-Admins: Igbo-Wikimedians-User-Group-Projects - https://phabricator.wikimedia.org/T266781 (10Aklapper) 05Openβ†’03Resolved a:03Aklapper Requested public project #igbo-wikimedians-user-group has been created: https://phabricator.wikimedia.org/project/view/5069/ Interested people are welcome to join... [11:48:25] (03CR) 10Thiemo Kreuz (WMDE): [C: 03+2] Allow @inheritDoc to be valid documentation of class properties [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638679 (https://phabricator.wikimedia.org/T266741) (owner: 10Umherirrender) [12:10:16] Hi o/ - We're about to change the way we build kuberneters packages a lot and the new build will require to download stuff from the internet. Currently the repo is debian-glue-non-voting which fails with the new changes which are also not gbp compilant. To what would I change the ci job to in that case? (https://gerrit.wikimedia.org/r/c/operations/debs/kubernetes/+/639192/3) [12:55:25] 10Release-Engineering-Team, 10Commons, 10DBA, 10Operations, and 2 others: Increase on database writes and deletes activity on Commonswiki leads to some replication lag - https://phabricator.wikimedia.org/T266432 (10Marostegui) 05Openβ†’03Resolved a:03Marostegui This has ceased and we are back to normal... [13:59:54] Kibana 5 filterEditor:suggestValues=false isn't respected in my browser. Something to check again after the upgrade... [14:10:32] 10Gerrit, 10VPS-Projects: Gerrit test instance is down - https://phabricator.wikimedia.org/T240933 (10hashar) There is another instance at https://gerrit.devtools.wmflabs.org/ . I have fixed it yesterday (the path to java was wrong due to a lack of configuration). It is not really functional since its confi... [14:32:56] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10cscott) [14:34:07] 10Release-Engineering-Team (Logspam), 10CommonsMetadata, 10Structured Data Engineering, 10Structured-Data-Backlog, and 2 others: CommonsMetadata bad wfTimestamp call - https://phabricator.wikimedia.org/T267033 (10cscott) a:03cscott [15:25:45] (03PS1) 10Lars Wirzenius: fix(train-dev.md): spelling mistake in "successfully" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/639547 [15:43:49] (03CR) 10Ahmon Dancy: [C: 03+2] fix(train-dev.md): spelling mistake in "successfully" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/639547 (owner: 10Lars Wirzenius) [15:49:52] PROBLEM - Work requests waiting in Zuul Gearman server on contint2001 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [150.0] https://www.mediawiki.org/wiki/Continuous_integration/Zuul https://grafana.wikimedia.org/dashboard/db/zuul-gearman?panelId=10&fullscreen&orgId=1 [15:52:09] RECOVERY - Work requests waiting in Zuul Gearman server on contint2001 is OK: OK: Less than 100.00% above the threshold [90.0] https://www.mediawiki.org/wiki/Continuous_integration/Zuul https://grafana.wikimedia.org/dashboard/db/zuul-gearman?panelId=10&fullscreen&orgId=1 [15:52:28] (03PS1) 10Lars Wirzenius: fix: typo in "successfully" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/639567 [15:53:57] (03CR) 10Ahmon Dancy: [C: 03+2] fix: typo in "successfully" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/639567 (owner: 10Lars Wirzenius) [16:06:04] (03CR) 10DannyS712: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 (owner: 10Umherirrender) [16:29:05] (03CR) 10Thcipriani: [V: 03+2 C: 03+2] "> Patch Set 2:" [software/gerrit] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/639121 (owner: 10Hashar) [16:52:16] (03PS2) 10Lars Wirzenius: chore: clean up shell quoting, other things found by linters [tools/train-dev] - 10https://gerrit.wikimedia.org/r/638672 [16:55:53] (03CR) 10Ahmon Dancy: [C: 03+2] chore: clean up shell quoting, other things found by linters [tools/train-dev] - 10https://gerrit.wikimedia.org/r/638672 (owner: 10Lars Wirzenius) [17:01:19] (03PS2) 10Umherirrender: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 [17:01:31] (03CR) 10Umherirrender: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 (owner: 10Umherirrender) [17:14:52] (03CR) 10Dduvall: "> Patch Set 4: Code-Review-1" [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/635886 (owner: 10Dduvall) [17:26:39] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen) [17:27:40] (03PS5) 10Dduvall: Optimize submodules operations [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/635886 [17:28:30] (03CR) 10Dduvall: "> Patch Set 4: Code-Review-1" [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/635886 (owner: 10Dduvall) [17:32:17] (03CR) 10Jeena Huneidi: [C: 03+2] Optimize submodules operations [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/635886 (owner: 10Dduvall) [17:33:03] (03Merged) 10jenkins-bot: Optimize submodules operations [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/635886 (owner: 10Dduvall) [17:39:01] 10Release-Engineering-Team (Logspam), 10CommonsMetadata, 10Structured Data Engineering, 10Structured-Data-Backlog, and 2 others: CommonsMetadata bad wfTimestamp call - https://phabricator.wikimedia.org/T267033 (10brennen) > I think this bug has been there for a long time, we just happened to hit an image t... [17:39:10] 10Release-Engineering-Team (Logspam), 10CommonsMetadata, 10Structured Data Engineering, 10Structured-Data-Backlog, and 2 others: CommonsMetadata bad wfTimestamp call - https://phabricator.wikimedia.org/T267033 (10brennen) [17:39:16] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen) [17:47:53] (03CR) 10DannyS712: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 (owner: 10Umherirrender) [17:49:30] 10Release-Engineering-Team-TODO, 10Release, 10Train Deployments: 1.36.0-wmf.18 deployment blockers - https://phabricator.wikimedia.org/T263184 (10hashar) If at all possible, can I get the train window at 14:00 local time (CET) which would be 13:00 UTC. That will be easier to manage for me. [17:59:36] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen) [18:22:32] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10cscott) [18:25:22] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10cscott) [18:38:46] !log beta has been stuck for ~22H. Killed backlog of jobs, seems to be running now [18:38:50] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:43:17] (03PS1) 10Thcipriani: Review access change [tools/scap] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/639499 [18:44:27] (03PS2) 10Thcipriani: Review access change [tools/scap] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/639499 [19:29:22] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Abstract Wikipedia (Phase Ξ²): mediawiki/extensions/WikiLambda repository has a leading space in the name - https://phabricator.wikimedia.org/T257775 (10DVrandecic) [19:48:50] πŸ‘‹ I have a CI question about mediawiki/services/texvcjs. I +2 a patch on Monday and retried again yesterday and its has just been lingering there. [19:49:17] I see CI has been enabled for it https://gerrit.wikimedia.org/r/c/integration/config/+/628523 [19:49:46] So I’m not really sure why it isn’t working. Any help would be appreciated [19:53:13] (03PS3) 10Umherirrender: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 [19:53:59] (03PS2) 10Umherirrender: Allow @inheritDoc to be valid documentation of class properties [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638679 (https://phabricator.wikimedia.org/T266741) [19:54:06] (03CR) 10Umherirrender: Move PropertyDocumentationSniff EmptySees check to EmptyTagSniff (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/638676 (owner: 10Umherirrender) [19:54:30] clarakosi: It's dependant commits [19:54:36] You +2'd https://gerrit.wikimedia.org/r/c/mediawiki/services/texvcjs/+/628048/1 [19:54:46] but https://gerrit.wikimedia.org/r/c/mediawiki/services/texvcjs/+/628048/1 and https://gerrit.wikimedia.org/r/c/mediawiki/services/texvcjs/+/628048/1 aren't reviewed/merged [19:55:12] You can tell by looking at the "Relation chain" on the right hand side [19:57:35] Ohhh I see πŸ€¦β€β™€οΈ [19:57:52] Reedy: thanks! [20:05:15] Apparently I just pasted the same commit three times :D [20:05:26] https://gerrit.wikimedia.org/r/c/mediawiki/services/texvcjs/+/628045/1 and https://gerrit.wikimedia.org/r/c/mediawiki/services/texvcjs/+/628036/1 [20:06:06] No worries "relation chain" was enough to get me rolling πŸ˜‰ [20:20:38] 10MediaWiki-Codesniffer, 10User-DannyS712: Autofix for switch indentation adds indentation to the wrong places - https://phabricator.wikimedia.org/T266739 (10Umherirrender) >>! In T266739#6601145, @Umherirrender wrote: > upstream reported - https://github.com/squizlabs/PHP_CodeSniffer/issues/3157 Fixed upstre... [20:55:36] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen) [21:02:46] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen) [21:10:09] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10cscott) [21:59:48] 10Release-Engineering-Team, 10Add-Link, 10Growth-Structured-Tasks, 10Growth-Team (Current Sprint), 10Patch-For-Review: Add Link engineering: Codehealth integration - https://phabricator.wikimedia.org/T266110 (10Etonkovidova) 05Openβ†’03Resolved Last three builds [[ https://integration.wikimedia.org/ci/... [21:59:51] 10Release-Engineering-Team, 10Add-Link, 10Growth-Structured-Tasks, 10Growth-Team (Current Sprint), 10Patch-For-Review: Add Link engineering: Deployment Pipeline setup - https://phabricator.wikimedia.org/T265893 (10Etonkovidova) [22:46:57] 10Deployments, 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Sustainability (Incident Followup), 10User-brennen: Avoid unfinished train deploys over holidays, weekends, or other stretches of no-deploy days - https://phabricator.wikimedia.org/T260401 (10Tgr) If the n... [23:51:23] 10Release-Engineering-Team-TODO, 10Patch-For-Review, 10Release, 10Train Deployments, 10User-brennen: 1.36.0-wmf.16 deployment blockers - https://phabricator.wikimedia.org/T263182 (10brennen)