[04:27:51] 10Phabricator, 10Tool-ldap: https://ldap.toolforge.org/ integration assumes that `cn` and `uid` are equivalent - https://phabricator.wikimedia.org/T376769#10212996 (10Legoktm) This is a silly question - is it possible to register "BryanDavis" as a uid? (probably not, because it has to be lower case?) The mai... [06:30:01] (03PS1) 10Kevin Bazira: inference-services: deactivate article-country CI [integration/config] - 10https://gerrit.wikimedia.org/r/1078793 (https://phabricator.wikimedia.org/T371897) [08:12:41] kevinbazira: that is unfortunate :D [08:12:47] the chicken and egg problem [08:12:52] (03CR) 10Hashar: [C:03+2] inference-services: deactivate article-country CI [integration/config] - 10https://gerrit.wikimedia.org/r/1078793 (https://phabricator.wikimedia.org/T371897) (owner: 10Kevin Bazira) [08:14:08] (03Merged) 10jenkins-bot: inference-services: deactivate article-country CI [integration/config] - 10https://gerrit.wikimedia.org/r/1078793 (https://phabricator.wikimedia.org/T371897) (owner: 10Kevin Bazira) [08:15:10] hashar: thanks for the merge. :D [08:15:18] kevinbazira: yeah it is a bit of a mess [08:15:22] well not a mess but problematic [08:15:24] ordering issue [08:15:27] something like that [08:16:00] what is possible is that for the first commit at https://gerrit.wikimedia.org/r/c/machinelearning/liftwing/inference-services/+/1075033 [08:16:24] you could integrate to it the blubber.yaml and pipeline config that are currently in https://gerrit.wikimedia.org/r/c/machinelearning/liftwing/inference-services/+/1077391 [08:16:32] this way that change 1075033 would pass CI and you can +2 it [08:16:44] else if you want to keep those separate: [08:17:33] 1) amend https://gerrit.wikimedia.org/r/c/machinelearning/liftwing/inference-services/+/1077391 to remove the `Depends-On`, get it merged [08:17:33] 2) revert revert the CI config and have it deployed [08:17:33] 3) `recheck` https://gerrit.wikimedia.org/r/c/machinelearning/liftwing/inference-services/+/1075033 [08:17:36] which should pass :) [08:17:52] it might be easier to use a single change though [08:18:02] kevinbazira: I have deployed the CI config change [08:41:08] 14Phabricator (2024-06-25), 10Release-Engineering-Team (Priority Backlog πŸ“₯), 10Wikimedia-Phabricator-Extensions: Phabricator profile "LDAP User" link goes to Wikitech, but users no longer need to have a Wikitech account - https://phabricator.wikimedia.org/T366766#10213255 (10Aklapper) Followup issue to "... [09:14:10] (03CR) 10Hashar: [C:04-1] "The intentional globbing caught my attention (thanks for finding that out). I think we can get rid of it and find the files when setting " [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) (owner: 10Arthur taylor) [09:15:56] codders: I have reviewed your change for jjb/assert-no-mediawiki-errors.bash [09:16:24] saw that - thank you! I'll make those changes shortly [09:16:24] I tried to avoid having to disable shellcheck and I found that we can ask bash to find the files for us (using globbing) instead of relying on ls [09:16:32] yeah - I prefer your way [09:16:38] ah cool :] [09:16:58] and today I have learned about bash `nullglob` option which fit our need pretty well [09:18:26] I also read about nullglob for the first time yesterday... and then stopped reading - you obviously have more determination than me to find the nice solutions :) [09:18:46] yeah maybe [09:19:12] I felt like using a pattern was an opportunity to get rid of the cd / ls [09:19:28] overall the number of lines / complexity would be reduced [09:19:42] at the price of introducing a barely known option `nullglob`, but a comment can cover that [09:20:12] anyway, it is great to see that script is being adjusted, it is quite critical [09:24:15] (03PS3) 10Arthur taylor: Match `split-group` log files when checking for mediawiki errors [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) [09:25:25] (03CR) 10Arthur taylor: Match `split-group` log files when checking for mediawiki errors (034 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) (owner: 10Arthur taylor) [09:25:53] (03CR) 10CI reject: [V:04-1] Match `split-group` log files when checking for mediawiki errors [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) (owner: 10Arthur taylor) [09:29:35] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Seen), 06cloud-services-team, 10Cloud-VPS, 10ci-test-error (WMF-deployed Build Failure): Various CI jobs failing with: Could not resolve host: gerrit.wikimedia.org - https://phabricator.wikimedia.org/T374830#10213430 (10hashar) There wa... [09:29:53] @hashar: meh. still get SC2206 because those globs are not enclosed in quotes. And when I put them in quotes the script doesn't work any more [09:30:15] I'll have a play around and see if I can make it behave [09:32:40] codders: here is why I have left in my local workspace while I was doing the review: https://phabricator.wikimedia.org/P69505 [09:33:02] thanks! [09:33:30] in lot of case I write the code then convert the change I have made to human friendly reviews [09:33:38] instead of doing a git amend/push/+2 :) [09:34:15] ERROR_FILES=("$WORKSPACE"/log/mw-dberror*.log "$WORKSPACE"/log/mw-error*.log ) [09:34:37] that is the only place with globs, my shellcheck 0.9.0 is happy about it [09:35:44] hmm. shellcheck 0.8.0 doesn't seem to like it https://integration.wikimedia.org/ci/job/integration-config-tox/1278/console [09:36:03] ah. but I missed those quotes. So I'll try with that [09:36:37] (03PS4) 10Arthur taylor: Match `split-group` log files when checking for mediawiki errors [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) [09:38:18] looks better - thank you! [09:40:47] maintenance-disconnect-full-disks build 639031 integration-agent-docker-1047 (/: 26%, /srv: 9%, /var/lib/docker: 95%): OFFLINE due to disk space [09:41:12] codders: the alternative is of course to rewrite that script to python :D [09:41:18] (I am kidding) [09:41:39] haha :) [09:43:57] * hashar lunches [10:19:28] 10Phabricator, 06Release-Engineering-Team: Please disable MFA for Phabricator account for WMDE-leszek - https://phabricator.wikimedia.org/T376605#10213563 (10WMDE-leszek) 05Openβ†’03Resolved Thank you @Aklapper. The MFA was disabled, which allowed me to log in and enable MFA again. This request is then d... [10:31:10] 06Gerrit-Privilege-Requests, 13Patch-For-Review: Give Edesol +2 on Extension:Add_HTML_Meta_and_Title in Gerrit - https://phabricator.wikimedia.org/T372073#10213585 (10Aklapper) I'd probably recommend sending a message to the [wikitech-l mailing list](https://www.mediawiki.org/wiki/Mailing_lists/Wikitech-l) aga... [11:01:29] 10Phabricator: Disable personal Herald rules H154? - https://phabricator.wikimedia.org/T374800#10213681 (10Aklapper) [11:02:29] @hashar: I know we discussed before, but was there a reason not to include XDebug in the Quibble docker images? If I make a PR for that can we add XDebug by default? [11:02:57] 10Phabricator: Disable personal Herald rules H154? - https://phabricator.wikimedia.org/T374800#10213692 (10Aklapper) 05Openβ†’03Resolved a:05TerraCodesβ†’03Aklapper @TerraCodes: I disabled your Herald rule H154, as in total, I could only find two matches from more than six years ago so it doesn't look pa... [12:34:52] ahh [12:35:04] codders needs an IRC bouncer :] [12:35:37] for when he comes back, Xdebug got removed in https://gerrit.wikimedia.org/r/c/integration/config/+/866388 because T319495 [12:35:37] T319495: Drop xdebug from all RelEng images - https://phabricator.wikimedia.org/T319495 [12:36:49] (03open) 10awight: Update deployments-calendar.yaml; Move awight to UTC morning window [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/122 [12:41:19] 10Continuous-Integration-Infrastructure, 13Patch-For-Review: Drop xdebug from all RelEng images - https://phabricator.wikimedia.org/T319495#10213923 (10hashar) @ArthurTaylor that is the task which removed XDebug in the CI image. On CI we only used it for generating test coverage and that got replaced by t... [12:47:59] (03Abandoned) 10Hashar: zuul/parameter_functions.py: Remove parsoid extension remapping [integration/config] - 10https://gerrit.wikimedia.org/r/1075299 (owner: 10Reedy) [12:51:10] (03open) 10jforrester: Add fix_stylelint_config to add .stylelintcache as needed [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/62 (https://phabricator.wikimedia.org/T376760) [12:55:33] (03PS1) 10Kosta Harlan: zuul: Add VisualEditor as dependency of Linter [integration/config] - 10https://gerrit.wikimedia.org/r/1078933 (https://phabricator.wikimedia.org/T376783) [13:00:46] (03CR) 10Hashar: [C:03+2] zuul: Add VisualEditor as dependency of Linter [integration/config] - 10https://gerrit.wikimedia.org/r/1078933 (https://phabricator.wikimedia.org/T376783) (owner: 10Kosta Harlan) [13:02:31] (03Merged) 10jenkins-bot: zuul: Add VisualEditor as dependency of Linter [integration/config] - 10https://gerrit.wikimedia.org/r/1078933 (https://phabricator.wikimedia.org/T376783) (owner: 10Kosta Harlan) [13:02:54] (03Abandoned) 10Hashar: Run Browser tests in parallel if QUIBBLE_SELENIUM_PARALLEL is set [integration/quibble] - 10https://gerrit.wikimedia.org/r/1055185 (https://phabricator.wikimedia.org/T370033) (owner: 10Arthur taylor) [13:32:41] 10Phabricator: Logging in - https://phabricator.wikimedia.org/T376803 (10Hdicu) 03NEW [13:36:52] 10Phabricator: Logging in - https://phabricator.wikimedia.org/T376803#10214098 (10Hdicu) [14:02:19] (03CR) 10Hashar: [C:03+2] Match `split-group` log files when checking for mediawiki errors (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) (owner: 10Arthur taylor) [14:03:05] !log Updating MediaWiki CI jobs for https://gerrit.wikimedia.org/r/c/integration/config/+/1078628 [14:03:06] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [14:03:57] 10Release-Engineering-Team (Radar), 06Data-Platform-SRE, 06Java-Scala-Standardization, 03Discovery-Search (Current work): Validate CI integration so that Ci can release Maven artifacts on user's demand - https://phabricator.wikimedia.org/T367403#10214360 (10Ahoelzl) 05Openβ†’03Resolved [14:04:00] (03Merged) 10jenkins-bot: Match `split-group` log files when checking for mediawiki errors [integration/config] - 10https://gerrit.wikimedia.org/r/1078628 (https://phabricator.wikimedia.org/T375852) (owner: 10Arthur taylor) [14:05:08] 06Release-Engineering-Team, 07Spike: [Developer Experience] [SPIKE] Investigate process to automate deployment of folders and artifacts to HDFS - https://phabricator.wikimedia.org/T360968#10214392 (10Ahoelzl) 05Openβ†’03Resolved [14:05:26] 10Phabricator: Logging in - https://phabricator.wikimedia.org/T376803#10214390 (10Aklapper) 05Openβ†’03Stalled I haven't heard this from other folks. Do you use some proxy or VPN setup? What's shown in the Network tab of the developer console of your web browser? Which browser(s) and versions did you test? Doe... [14:10:05] 06Project-Admins, 07Tracking-Neverending: Requests for addition to the #acl*Project-Admins group (in comments) - https://phabricator.wikimedia.org/T706#10214461 (10JTweed-WMF) Hi, I've recently started as Senior Product Manager for the MediaWiki Platform team. Can I please request permissions to create project... [14:16:59] 06Project-Admins, 07Tracking-Neverending: Requests for addition to the #acl*Project-Admins group (in comments) - https://phabricator.wikimedia.org/T706#10214494 (10Ladsgroup) >>! In T706#10214461, @JTweed-WMF wrote: > Hi, I've recently started as Senior Product Manager for the MediaWiki Platform team. Can I pl... [14:27:16] hey [14:27:26] do you have any idea why I get this on the operations/puppet.git repo? [14:27:32] https://www.irccloud.com/pastebin/R04FMy7P/ [14:47:22] (03update) 10dancy: Update deployments-calendar.yaml; Move awight to UTC morning window [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/122 (owner: 10awight) [16:19:56] (03PS1) 10Hashar: jjb: force grep color when searching mw error logs [integration/config] - 10https://gerrit.wikimedia.org/r/1078978 [16:21:20] 14Phabricator (2024-06-25), 10Release-Engineering-Team (Priority Backlog πŸ“₯), 10Wikimedia-Phabricator-Extensions: Phabricator profile "LDAP User" link goes to Wikitech, but users no longer need to have a Wikitech account - https://phabricator.wikimedia.org/T366766#10214978 (10matmarex) Thanks, this is nice. [16:21:52] (03CR) 10Eric Gardner: Codex-PHP: Remove the postmerge tasks for now (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/1078493 (https://phabricator.wikimedia.org/T373940) (owner: 10Eric Gardner) [16:26:15] 10Phabricator, 10Tool-ldap: https://ldap.toolforge.org/ integration assumes that `cn` and `uid` are equivalent - https://phabricator.wikimedia.org/T376769#10214990 (10bd808) >>! In T376769#10212996, @Legoktm wrote: > This is a silly question - is it possible to register "BryanDavis" as a uid? (probably not, be... [16:38:40] (03PS2) 10Hashar: jjb: grep from log dir to have short names [integration/config] - 10https://gerrit.wikimedia.org/r/1078978 [16:41:08] (03CR) 10Hashar: "An example build: https://integration.wikimedia.org/ci/job/mediawiki-quibble-apitests-vendor-php74/21498/console" [integration/config] - 10https://gerrit.wikimedia.org/r/1078978 (owner: 10Hashar) [16:43:01] (03PS1) 10Hashar: jjb: force grep color when searching mw error logs [integration/config] - 10https://gerrit.wikimedia.org/r/1078981 [16:44:12] (03CR) 10CI reject: [V:04-1] jjb: force grep color when searching mw error logs [integration/config] - 10https://gerrit.wikimedia.org/r/1078981 (owner: 10Hashar) [16:47:28] (03PS2) 10Hashar: jjb: force grep color when searching mw error logs [integration/config] - 10https://gerrit.wikimedia.org/r/1078981 [16:49:35] (03CR) 10Hashar: [C:04-1] Codex-PHP: Remove the postmerge tasks for now (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/1078493 (https://phabricator.wikimedia.org/T373940) (owner: 10Eric Gardner) [16:58:01] 10Phabricator, 10Tool-ldap: https://ldap.toolforge.org/ integration assumes that `cn` and `uid` are equivalent - https://phabricator.wikimedia.org/T376769#10215091 (10Legoktm) a:03Legoktm Ack, thanks for the explanation. I'll create a new `/cn/` route then and will send a MR to update Phabricator's links too. [18:00:57] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830 (10bd808) 03NEW [18:01:43] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830#10215273 (10bd808) p:05Triageβ†’03Medium a:03bd808 [18:28:10] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830#10215337 (10bd808) [18:42:31] (03update) 10jforrester: Add fix_stylelint_config to add .stylelintcache as needed [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/62 (https://phabricator.wikimedia.org/T376760) [18:43:06] (03update) 10jforrester: Add fix_stylelint_config to add .stylelintcache as needed [repos/ci-tools/libup] - 10https://gitlab.wikimedia.org/repos/ci-tools/libup/-/merge_requests/62 (https://phabricator.wikimedia.org/T376760) [19:13:36] 10GitLab (Pipeline Services Migration🐀), 06collaboration-services, 10Wikidata, 10Wikidata Query UI, and 3 others: move commons-query.wikimedia.org and query.wikidata.org to kubernetes - https://phabricator.wikimedia.org/T350793#10215478 (10WMDE-leszek) Thanks @Lucas_Werkmeister_WMDE for bringing up the top... [20:09:38] 10Release-Engineering-Team (Priority Backlog πŸ“₯), 10Scap, 10Codex, 10Design-System-Team (DST-Sprint-33 (2024-09-30 to 2024-10-11)), 07Epic: [Spike/Timebox] DST support for web app scaffolding & initial setup - https://phabricator.wikimedia.org/T375826#10215621 (10egardner) [20:20:18] (03open) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [20:20:18] (03update) 10dancy: WIP: Spiderpig HTTP API [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/451 [20:20:18] (03update) 10dancy: WIP: SpiderPig Web UI [repos/releng/scap] (master-I0a50f91db8c9aa9147afdd0468e37e001af86167) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/471 [20:20:19] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [20:20:26] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [20:20:31] (03update) 10dancy: WIP: SpiderPig Web UI [repos/releng/scap] (master-I0a50f91db8c9aa9147afdd0468e37e001af86167) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/471 [20:23:10] 10Release-Engineering-Team (Priority Backlog πŸ“₯), 10Scap, 10Codex, 10Design-System-Team (DST-Sprint-33 (2024-09-30 to 2024-10-11)), 07Epic: [Spike/Timebox] DST support for web app scaffolding & initial setup - https://phabricator.wikimedia.org/T375826#10215646 (10egardner) @dancy and I will meet tomorrow... [22:37:47] (03approved) 10thcipriani: Don't load plugins from user's home directory or current directory [repos/releng/scap] - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/466 (owner: 10dancy) [22:39:11] (03merge) 10thcipriani: Update deployments-calendar.yaml; Move awight to UTC morning window [repos/releng/release] - 10https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/122 (owner: 10awight) [22:44:31] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830#10215841 (10bd808) I created https://toolsadmin.wikimedia.org/tools/id/jenkins-build-stats to run the job from Dan's repo. At the moment... [22:44:44] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830#10215842 (10bd808) 05Openβ†’03In progress [23:02:59] 10Continuous-Integration-Infrastructure, 10Cloud-VPS (Quota-requests): Quota increase for Integration project (Jenkins CI runners) - https://phabricator.wikimedia.org/T376847 (10bd808) 03NEW [23:29:34] !log Setup integration-agent-docker Puppet prefix and removed instance Puppet configuration for all integration-agent-docker-* instances. [23:29:35] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [23:31:26] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [23:32:32] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [23:33:19] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [23:33:41] (03update) 10dancy: WIP: spiderpig auth [repos/releng/scap] (master-I6b866887f1fb30397d9cb409a4f0844cc11235c2) - 10https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/472 [23:35:39] 10Gerrit, 06collaboration-services, 10VPS-project-Codesearch, 10VPS-project-Extdist, 13Patch-For-Review: Move clients off of gerrit-replica.wikimedia.org back to gerrit.wikimedia.org - https://phabricator.wikimedia.org/T336710#10215968 (10Dzahn) Hello Antoine, so the reason I hadn't just merged that was... [23:37:50] 10Gerrit, 06collaboration-services, 10VPS-project-Codesearch, 10VPS-project-Extdist, 13Patch-For-Review: Move clients off of gerrit-replica.wikimedia.org back to gerrit.wikimedia.org - https://phabricator.wikimedia.org/T336710#10215974 (10Ladsgroup) BTW, last time we did the change, the hound storage fil... [23:42:02] 10Continuous-Integration-Infrastructure: Setup CI build stats reporting tool to produce ongoing reports on capacity and utilization - https://phabricator.wikimedia.org/T376830#10215979 (10bd808) > Dan said he remembers needing authentication to access something in the Jenkins API, but the details have been lost...