[01:49:28] Deployed https://gerrit.wikimedia.org/r/c/performance/debs/xhgui/+/621855 on deployment-xhgui03. [01:49:41] !log Deployed https://gerrit.wikimedia.org/r/c/performance/debs/xhgui/+/621855 on deployment-xhgui03. [01:49:42] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [02:35:13] 10Diffusion, 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO, 10Wikimedia-GitHub: Create and maintain somehow a list of repos mastered in GitHub (and in Phabricator Diffusion) - https://phabricator.wikimedia.org/T237470 (10Krinkle) >>! In T237470#6407537, @has... [04:20:40] (03CR) 10Hashar: [C: 03+1] "Looks fine to me ;)" [tools/scap-vagrant] - 10https://gerrit.wikimedia.org/r/621839 (owner: 10Dduvall) [04:34:24] (03CR) 10Hashar: [C: 03+2] "Job deployed:" [integration/config] - 10https://gerrit.wikimedia.org/r/622167 (https://phabricator.wikimedia.org/T260975) (owner: 10Tarrow) [04:35:31] (03Merged) 10jenkins-bot: phpmetrics: add WikibaseManifest [integration/config] - 10https://gerrit.wikimedia.org/r/622167 (https://phabricator.wikimedia.org/T260975) (owner: 10Tarrow) [07:11:16] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) [07:11:47] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) p:05Triage→03High Setting priority to high as it's a blocker for m... [07:34:01] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) Apparently the problem is not a firewall: when firewalling the google... [07:34:07] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) a:05Joe→03None [07:34:31] <_joe_> any help on ^^ would be appreciated [07:52:39] (03PS1) 10Legoktm: rust-coverage: Use newer tarpaulin to stop loading external resources [integration/config] - 10https://gerrit.wikimedia.org/r/622305 (https://phabricator.wikimedia.org/T260786) [07:53:37] (03PS2) 10Legoktm: rust-coverage: Use newer tarpaulin to stop loading external resources [integration/config] - 10https://gerrit.wikimedia.org/r/622305 (https://phabricator.wikimedia.org/T260786) [07:55:16] _joe_: what's the command you're using to run it locally? [07:57:48] (03CR) 10Legoktm: [C: 03+2] rust-coverage: Use newer tarpaulin to stop loading external resources [integration/config] - 10https://gerrit.wikimedia.org/r/622305 (https://phabricator.wikimedia.org/T260786) (owner: 10Legoktm) [07:58:50] (03Merged) 10jenkins-bot: rust-coverage: Use newer tarpaulin to stop loading external resources [integration/config] - 10https://gerrit.wikimedia.org/r/622305 (https://phabricator.wikimedia.org/T260786) (owner: 10Legoktm) [07:59:29] !log rebuilding rust-coverage image for https://gerrit.wikimedia.org/r/c/integration/config/+/622305 [07:59:31] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [08:03:16] _joe_: I'm able to replicate the error locally fwiw [08:04:55] <_joe_> legoktm: oh? interesting [08:05:16] <_joe_> legoktm: can you comment on the task with your steps to repro? [08:05:25] <_joe_> I'm suspecting I might have a locally built image [08:05:29] if I had to guess you have some local helm thing running [08:05:32] because in the output I see [08:05:33] ...Unable to get an update from the "local" chart repository (http://127.0.0.1:8879/charts): [08:05:33] Get "http://127.0.0.1:8879/charts/index.yaml": dial tcp 127.0.0.1:8879: connect: connection refused [08:05:43] <_joe_> that's normal [08:05:52] <_joe_> but the googleapis thing isn't [08:06:30] it's intentional that https://gerrit.wikimedia.org/r/plugins/gitiles/integration/config/+/refs/heads/master/dockerfiles/helm-linter/Dockerfile.template only adds the wikimedia repo and not the google one? [08:07:01] <_joe_> uhm... I think I know what happened [08:07:24] <_joe_> the change for uploading 0.2.5 was merged before I could submit my latest change [08:07:59] <_joe_> that would explain the difference [08:08:06] <_joe_> and yes, that's what happened [08:10:10] sorry I'm not following...what needs fixing? [08:10:39] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) a:03Joe The issue was found, thanks to @Legoktm for finding it: th... [08:10:53] ok :) [08:11:06] 10Continuous-Integration-Infrastructure, 10Code-Review-Workgroup, 10Quality-and-Test-Engineering-Team (QTE), 10Quibble: Lightweight preview environment for gerrit changes - https://phabricator.wikimedia.org/T241140 (10kostajh) cc @hashar and @awight with regards to some of the upcoming #Quibble work. [08:11:15] (03PS1) 10Giuseppe Lavagetto: helm-linter: add the google incubator charts to the image [integration/config] - 10https://gerrit.wikimedia.org/r/622307 (https://phabricator.wikimedia.org/T261182) [08:11:27] (03CR) 10jerkins-bot: [V: 04-1] helm-linter: add the google incubator charts to the image [integration/config] - 10https://gerrit.wikimedia.org/r/622307 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:11:34] <_joe_> legoktm: while I was firefighting an outage, someone merged my previous change [08:11:52] <_joe_> and I had a further patch to add [08:12:53] (03PS2) 10Giuseppe Lavagetto: helm-linter: add the google incubator charts to the image [integration/config] - 10https://gerrit.wikimedia.org/r/622307 (https://phabricator.wikimedia.org/T261182) [08:13:58] _joe_: ack, let me know when it's ready to merge + build/deploy and I can take care of that [08:14:13] (03PS1) 10Giuseppe Lavagetto: Update jjb to use helm-linter 0.2.6 [integration/config] - 10https://gerrit.wikimedia.org/r/622308 (https://phabricator.wikimedia.org/T261182) [08:14:27] <_joe_> legoktm: I think I can too? anyways, they're ready :) [08:15:03] (03CR) 10Legoktm: [C: 03+2] helm-linter: add the google incubator charts to the image [integration/config] - 10https://gerrit.wikimedia.org/r/622307 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:16:15] (03Merged) 10jenkins-bot: helm-linter: add the google incubator charts to the image [integration/config] - 10https://gerrit.wikimedia.org/r/622307 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:16:40] !log rebuilding helm-linter image for https://gerrit.wikimedia.org/r/622307 [08:16:42] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [08:18:09] (03PS1) 10Legoktm: docker: Bump rust-coverage image to 0.1.3 [integration/config] - 10https://gerrit.wikimedia.org/r/622309 (https://phabricator.wikimedia.org/T260786) [08:18:10] (03CR) 10Legoktm: [C: 03+2] docker: Bump rust-coverage image to 0.1.3 [integration/config] - 10https://gerrit.wikimedia.org/r/622309 (https://phabricator.wikimedia.org/T260786) (owner: 10Legoktm) [08:18:59] 10Continuous-Integration-Infrastructure, 10ContentSecurityPolicy, 10Patch-For-Review, 10Upstream: Rust CI coverage report isn't visible because of loading external resources - https://phabricator.wikimedia.org/T260786 (10Legoktm) 05Open→03Resolved a:03Legoktm [08:19:18] (03Merged) 10jenkins-bot: docker: Bump rust-coverage image to 0.1.3 [integration/config] - 10https://gerrit.wikimedia.org/r/622309 (https://phabricator.wikimedia.org/T260786) (owner: 10Legoktm) [08:23:19] 10Release-Engineering-Team (Kanban), 10Scap, 10Patch-For-Review: Scap: TypeError: error: (not all arguments converted during string formatting); format string: (Passed unrecognized git_binary_manager {}); arguments: ((u'fat',)) - https://phabricator.wikimedia.org/T184882 (10hashar) @elukey has kindly merged... [08:23:20] (03PS2) 10Legoktm: Update jjb to use helm-linter 0.2.6 [integration/config] - 10https://gerrit.wikimedia.org/r/622308 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:23:22] (03CR) 10Legoktm: [C: 03+2] Update jjb to use helm-linter 0.2.6 [integration/config] - 10https://gerrit.wikimedia.org/r/622308 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:23:44] _joe_: deployed [08:24:04] <_joe_> legoktm: <3 [08:24:19] (03Merged) 10jenkins-bot: Update jjb to use helm-linter 0.2.6 [integration/config] - 10https://gerrit.wikimedia.org/r/622308 (https://phabricator.wikimedia.org/T261182) (owner: 10Giuseppe Lavagetto) [08:26:09] :-] [08:26:24] legoktm: thank for the rust/doc content security policy patch [08:26:42] :D [08:26:49] np, now I have to write some tests :p [08:44:00] 10Release-Engineering-Team, 10Prod-Kubernetes, 10Release Pipeline, 10serviceops, 10Patch-For-Review: Blocker: helm fails to update the kubernetes-charts-incubator.storage.googleapis.com when running in CI - https://phabricator.wikimedia.org/T261182 (10Joe) 05Open→03Resolved Thanks @Legoktm for the help! [08:53:51] (03CR) 10Hashar: deployment-calendar: alternative wikitext output (032 comments) [tools/release] - 10https://gerrit.wikimedia.org/r/621793 (owner: 10Thcipriani) [08:59:48] <_joe_> hashar: say I want to change something in the deployment calendar (remove puppet-swat), how do I do that? [09:00:10] <_joe_> just modify that repo? [09:00:26] _joe_: I have absolutely no idea. I just pipe those requests to thcipriani (and previously greg) [09:00:36] <_joe_> ahaha ok [09:01:03] I guess they manually run the command that is listed at https://gerrit.wikimedia.org/r/c/mediawiki/tools/release/+/621793/ [09:01:13] and upload the content to wikitech [09:02:36] _joe_: so I guess it is all about removing the entry from deployments-calendar.json [09:02:42] (03PS1) 10Giuseppe Lavagetto: Remove puppet request window [tools/release] - 10https://gerrit.wikimedia.org/r/622315 [09:02:48] <_joe_> yes :) [09:08:23] (03CR) 10Filippo Giunchedi: [C: 03+1] Remove puppet request window [tools/release] - 10https://gerrit.wikimedia.org/r/622315 (owner: 10Giuseppe Lavagetto) [09:14:17] (03CR) 10Hashar: Remove puppet request window (032 comments) [tools/release] - 10https://gerrit.wikimedia.org/r/622315 (owner: 10Giuseppe Lavagetto) [09:14:25] _joe_: there is one extra entry that needs to be removed [09:15:20] <_joe_> hashar: thanks, I'll look [09:16:03] _joe_: namely drop another entry refering to that "puppet" window [09:16:23] <_joe_> hashar: ack, thanks [09:16:35] <_joe_> I'm writing to ops@ to see if someone wants to pick the process up [09:17:20] as far as i am concerned, clunic duty has replaced my use case for the puppet swat window [09:17:31] (which I never actively used since it tends to be too late for me) [09:18:54] or I poke my usual "victims" ;D [09:22:02] (03PS2) 10Giuseppe Lavagetto: Remove puppet request window [tools/release] - 10https://gerrit.wikimedia.org/r/622315 [09:24:54] (03PS2) 10Hashar: build: helpers to start php builtin server [integration/docroot] - 10https://gerrit.wikimedia.org/r/622140 [09:28:42] (03CR) 10Giuseppe Lavagetto: Remove puppet request window (032 comments) [tools/release] - 10https://gerrit.wikimedia.org/r/622315 (owner: 10Giuseppe Lavagetto) [09:34:12] (03CR) 10Awight: "> Nitpick: I find the "webserver" argparse option / variable name to be a bit misleading" (032 comments) [integration/quibble] - 10https://gerrit.wikimedia.org/r/590651 (https://phabricator.wikimedia.org/T225218) (owner: 10Awight) [10:26:14] (03PS3) 10Hashar: build: helpers to start php builtin server [integration/docroot] - 10https://gerrit.wikimedia.org/r/622140 (https://phabricator.wikimedia.org/T149924) [10:26:16] (03PS1) 10Hashar: shared: abstract out realpath check [integration/docroot] - 10https://gerrit.wikimedia.org/r/622323 (https://phabricator.wikimedia.org/T149924) [10:57:36] (03CR) 10Hashar: [C: 04-1] shared: abstract out realpath check (031 comment) [integration/docroot] - 10https://gerrit.wikimedia.org/r/622323 (https://phabricator.wikimedia.org/T149924) (owner: 10Hashar) [10:58:11] 10Phabricator: Spaces request for ...Tech Sr Leadership - https://phabricator.wikimedia.org/T259156 (10Aklapper) 05Open→03Resolved a:03Aklapper >>! In T259156#6407109, @debt wrote: > I've closed all the tickets that were open on all 3 boards. Thank you! > However, when I tried to archive the 3 projects... [11:11:03] (03PS2) 10Hashar: shared: abstract out realpath check [integration/docroot] - 10https://gerrit.wikimedia.org/r/622323 (https://phabricator.wikimedia.org/T149924) [11:11:04] (03PS4) 10Hashar: Support published documents outside of the git checkout [integration/docroot] - 10https://gerrit.wikimedia.org/r/621720 (https://phabricator.wikimedia.org/T149924) [11:11:14] too many patchsets bah :-\ [11:12:24] (03CR) 10Jbond: Remove puppet request window (031 comment) [tools/release] - 10https://gerrit.wikimedia.org/r/622315 (owner: 10Giuseppe Lavagetto) [11:14:02] 10Gerrit, 10MediaWiki-extensions-Examples: Rename repo name on Gerrit from "examples" to "Examples" - https://phabricator.wikimedia.org/T261199 (10MarcoAurelio) [11:23:02] 10Gerrit, 10MediaWiki-extensions-Examples: Rename repo name on Gerrit from "examples" to "Examples" - https://phabricator.wikimedia.org/T261199 (10Aklapper) It's still unclear to me which problem is solved by renaming. :) If there are current standards and conventions could someone link to them, please? [11:46:11] 10Gerrit, 10MediaWiki-extensions-Examples: Rename repo name on Gerrit from "examples" to "Examples" - https://phabricator.wikimedia.org/T261199 (10xSavitar) Thanks for the question @Aklapper. Most of our extensions are named in the format "Xyz" or "XyzAbc" etc. I've seen very few extensions using "ABC" (extens... [11:47:44] 10Gerrit, 10MediaWiki-extensions-Examples: Rename repo name on Gerrit from "examples" to "Examples" - https://phabricator.wikimedia.org/T261199 (10xSavitar) [12:15:03] (03CR) 10CDanis: Remove puppet request window (031 comment) [tools/release] - 10https://gerrit.wikimedia.org/r/622315 (owner: 10Giuseppe Lavagetto) [12:21:54] (03CR) 10Hashar: [C: 03+2] "Awesome, a good step forward." [integration/quibble] - 10https://gerrit.wikimedia.org/r/591826 (https://phabricator.wikimedia.org/T225218) (owner: 10Awight) [12:37:29] (03Merged) 10jenkins-bot: Configurable web host and port [integration/quibble] - 10https://gerrit.wikimedia.org/r/591826 (https://phabricator.wikimedia.org/T225218) (owner: 10Awight) [12:41:21] (03PS1) 10Gehel: Ryan is now in charge of WDQS deployments. [tools/release] - 10https://gerrit.wikimedia.org/r/622341 [12:41:26] (03CR) 10Kosta Harlan: "This looks good, though I'm curious what Apache config Krinkle would propose to use instead to work around this breakage." [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [12:43:11] 10Gerrit, 10MediaWiki-extensions-Examples: Rename repo name on Gerrit from "examples" to "Examples" - https://phabricator.wikimedia.org/T261199 (10hashar) [12:43:13] 10Gerrit, 10Release-Engineering-Team (Development services): Support renaming repositories in Gerrit - https://phabricator.wikimedia.org/T239693 (10hashar) [13:00:16] 10Continuous-Integration-Infrastructure, 10Wikidata: Authenticate github API requests in CI - https://phabricator.wikimedia.org/T254319 (10Lydia_Pintscher) [13:21:19] (03PS5) 10Hashar: Support published documents outside of the git checkout [integration/docroot] - 10https://gerrit.wikimedia.org/r/621720 (https://phabricator.wikimedia.org/T149924) [13:21:21] (03PS1) 10Hashar: dev: provide dummy data for Zuul status page [integration/docroot] - 10https://gerrit.wikimedia.org/r/622346 [13:21:59] (03CR) 10Hashar: "The original change was way too large so I have split it in several smaller changes :]" [integration/docroot] - 10https://gerrit.wikimedia.org/r/621720 (https://phabricator.wikimedia.org/T149924) (owner: 10Hashar) [13:35:00] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team-TODO, 10Zuul, 10Documentation: Document Zuul problems caused by force merge - https://phabricator.wikimedia.org/T225955 (10hashar) a:05hashar→03None Reflecting the reality. [13:39:01] thcipriani: Hello! I've just created https://gerrit.wikimedia.org/r/c/mediawiki/tools/release/+/622341 to update the deployment calendar. Is that sufficient? [13:39:53] the documented process seems to be "ping thcipriani on IRC" : https://wikitech.wikimedia.org/wiki/Deployments [13:47:21] gehel: yes that is the process afaik [13:47:37] (03CR) 10Hashar: [C: 03+1] Ryan is now in charge of WDQS deployments. [tools/release] - 10https://gerrit.wikimedia.org/r/622341 (owner: 10Gehel) [13:48:07] gehel: I have a checkin with Tyler in a couple hours and there is another change pending on that repo so I will mention them to him [13:58:03] 10Release-Engineering-Team-TODO, 10Platform Engineering, 10MW-1.35-notes (1.35.0-wmf.40; 2020-07-07), 10Patch-For-Review, and 6 others: New Public Wiki for the API Portal - https://phabricator.wikimedia.org/T246945 (10WDoranWMF) [14:01:05] hashar: thanks! [14:02:12] (03CR) 10Krinkle: "> I'm curious what Apache config Krinkle would propose to use" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:05:38] (03CR) 10Krinkle: "https://www.mediawiki.org/wiki/Manual:Short_URL/Apache#Simple_instructions" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:08:15] (03CR) 10Krinkle: [C: 04-1] "See also ?demo=basic or ?source_url=, which exist for this purpose." [integration/docroot] - 10https://gerrit.wikimedia.org/r/622346 (owner: 10Hashar) [14:11:18] (03PS1) 10Krinkle: README: Document composer commands from 2474b797 [integration/docroot] - 10https://gerrit.wikimedia.org/r/622360 [14:11:51] (03CR) 10Krinkle: "This can be reduced to a one-line change in composer.json (4000 -> 4001) instead." [integration/docroot] - 10https://gerrit.wikimedia.org/r/622140 (https://phabricator.wikimedia.org/T149924) (owner: 10Hashar) [14:14:11] (03CR) 10Krinkle: Support published documents outside of the git checkout (031 comment) [integration/docroot] - 10https://gerrit.wikimedia.org/r/621720 (https://phabricator.wikimedia.org/T149924) (owner: 10Hashar) [14:21:17] (03CR) 10Dbarratt: "> Patch Set 2:" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:26:24] (03CR) 10Krinkle: "By adding the route in the Apache config as-needed. We do the same for ShortURL in prod." [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:38:05] (03CR) 10Dbarratt: "> Patch Set 2:" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:40:35] 10Continuous-Integration-Config, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)): CI fetching tags should also create the local tag references - https://phabricator.wikimedia.org/T186496 (10hashar) I had all Stretch based images rebuild with... [14:45:10] (03CR) 10Krinkle: "Yes, this apache file should only configure /wiki if it also injects/provides MW settings that configure wgArticlePath. I was assuming it " [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [14:45:45] (03CR) 10Thcipriani: [C: 03+2] "Thanks!" [tools/release] - 10https://gerrit.wikimedia.org/r/622341 (owner: 10Gehel) [14:46:30] (03Merged) 10jenkins-bot: Ryan is now in charge of WDQS deployments. [tools/release] - 10https://gerrit.wikimedia.org/r/622341 (owner: 10Gehel) [15:00:00] 10Phabricator: Spaces request for ...Tech Sr Leadership - https://phabricator.wikimedia.org/T259156 (10debt) Thanks again for all your expert advice and help, @Aklapper :) [15:02:29] (03CR) 10Kosta Harlan: "> Patch Set 2:" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [15:04:06] (03CR) 10Dbarratt: "> Patch Set 2:" [releng/dev-images] - 10https://gerrit.wikimedia.org/r/621896 (https://phabricator.wikimedia.org/T261051) (owner: 10Dbarratt) [15:04:23] (03PS1) 10Hashar: dockerfiles: refresh images to use ci-src-setup.sh [integration/config] - 10https://gerrit.wikimedia.org/r/622374 (https://phabricator.wikimedia.org/T141607) [15:07:23] (03CR) 10Hashar: [C: 03+2] "Rebuilding the fleet." [integration/config] - 10https://gerrit.wikimedia.org/r/622374 (https://phabricator.wikimedia.org/T141607) (owner: 10Hashar) [15:08:27] (03Merged) 10jenkins-bot: dockerfiles: refresh images to use ci-src-setup.sh [integration/config] - 10https://gerrit.wikimedia.org/r/622374 (https://phabricator.wikimedia.org/T141607) (owner: 10Hashar) [15:13:53] (03PS1) 10Lars Wirzenius: update debian/changelog for release [tools/scap] - 10https://gerrit.wikimedia.org/r/622377 [15:14:09] (03CR) 10Lars Wirzenius: [V: 03+2 C: 03+2] update debian/changelog for release [tools/scap] - 10https://gerrit.wikimedia.org/r/622377 (owner: 10Lars Wirzenius) [15:44:35] Project beta-scap-eqiad build #314406: 04FAILURE in 0.76 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314406/ [15:46:00] (03PS1) 10Lars Wirzenius: fix: undo use of yaml.SafeLoader [tools/scap] - 10https://gerrit.wikimedia.org/r/622385 [15:47:27] 16:44:34 [ERROR] "scap sync" has been renamed to "scap sync-world". [15:47:27] 16:44:34 Use "scap sync-world" or "scap sync-file" instead, [15:49:40] (03CR) 10Lars Wirzenius: [C: 03+2] "Self-reviewing undoing my own change." [tools/scap] - 10https://gerrit.wikimedia.org/r/622385 (owner: 10Lars Wirzenius) [15:54:43] Project beta-scap-eqiad build #314407: 04STILL FAILING in 0.85 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314407/ [15:56:43] Project beta-scap-eqiad build #314408: 04STILL FAILING in 0.66 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314408/ [16:04:45] Project beta-scap-eqiad build #314409: 04STILL FAILING in 0.76 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314409/ [16:11:30] (03PS1) 10Lars Wirzenius: fix: use scap sync-world instead of scap sync [integration/config] - 10https://gerrit.wikimedia.org/r/622388 [16:11:58] (03CR) 10Lars Wirzenius: "This is to fix scap on beta for new release testing." [integration/config] - 10https://gerrit.wikimedia.org/r/622388 (owner: 10Lars Wirzenius) [16:14:36] Project beta-scap-eqiad build #314410: 04STILL FAILING in 0.75 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314410/ [16:24:33] I'll deploy it. [16:24:44] Project beta-scap-eqiad build #314411: 04STILL FAILING in 0.71 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314411/ [16:24:50] Unless liw desperately wants to? [16:26:12] James_F, I'm in the middle of making a scap release and trying to get CI to use sync-world instead of sync [16:26:24] James_F, but my first time changing JJB [16:26:25] Doing now. [16:26:34] !log tox -e jenkins-jobs -- --conf jenkins_jobs.ini update ./jjb/ 'beta-scap-eqiad' [16:26:36] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [16:26:53] (03PS2) 10Jforrester: jjb: [beta-scap-eqiad] Use scap sync-world instead of scap sync [integration/config] - 10https://gerrit.wikimedia.org/r/622388 (owner: 10Lars Wirzenius) [16:27:03] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/622388 (owner: 10Lars Wirzenius) [16:27:16] liw: Should now be deployed and working. [16:28:32] (03Merged) 10jenkins-bot: jjb: [beta-scap-eqiad] Use scap sync-world instead of scap sync [integration/config] - 10https://gerrit.wikimedia.org/r/622388 (owner: 10Lars Wirzenius) [16:29:33] Project beta-scap-eqiad build #314412: 04STILL FAILING in 17 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314412/ [16:30:23] James_F, thanks [16:30:32] Any time. [16:34:41] Project beta-scap-eqiad build #314413: 04STILL FAILING in 5.9 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314413/ [16:40:09] I'm fixing that now, ready in a minute I hope; if not, I'll revert Scap on beta to earlier version [16:41:13] (03PS1) 10Lars Wirzenius: fix: use of super [tools/scap] - 10https://gerrit.wikimedia.org/r/622393 [16:42:42] 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Scap, 10EngProd-Virtual-Hackathon, 10Patch-For-Review, 10Sustainability (Incident Followup): Rename `scap sync` to `scap sync-world` - https://phabricator.wikimedia.org/T250302 (10LarsWirzenius) beta-scap-eqiad was running scap sync, now... [16:43:56] (03CR) 10Lars Wirzenius: [C: 03+2] fix: use of super [tools/scap] - 10https://gerrit.wikimedia.org/r/622393 (owner: 10Lars Wirzenius) [16:44:15] (03CR) 10Lars Wirzenius: [V: 03+2 C: 03+2] fix: use of super [tools/scap] - 10https://gerrit.wikimedia.org/r/622393 (owner: 10Lars Wirzenius) [16:44:45] Project beta-scap-eqiad build #314414: 04STILL FAILING in 4 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314414/ [16:50:36] Project beta-scap-eqiad build #314415: 04STILL FAILING in 3.6 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314415/ [16:54:11] !log Build Docker images to use a common ci-src-setup.sh script instead of copy pasted shells # https://gerrit.wikimedia.org/r/c/integration/config/+/622374 [16:54:13] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [16:54:41] Project beta-scap-eqiad build #314416: 04STILL FAILING in 6 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314416/ [16:58:03] !log deployment-deploy01: upgraded scap (3.15.0-1+0~20200825155225.104~1.gbp6cd96d) over (3.15.0-1+0~20200825151423.100~1.gbpec9bb3) [16:58:04] liw: ^^ [16:58:05] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [16:58:26] Project beta-scap-eqiad build #314417: 04STILL FAILING in 3.7 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314417/ [17:00:07] from /var/log/apt/history.log the upgrade that broke it: Upgrade: scap:amd64 (3.14.0-1+0~20200402164156.62~1.gbp529a68, 3.15.0-1+0~20200825151423.100~1.gbpec9bb3) [17:00:16] which is hmm lot of changes :\ [17:00:21] Yeah. [17:00:36] All of them small, but they add up. [17:00:46] and I have no idea why scap doesn't magically upgade itself [17:01:05] hashar, er, I'm _trying_ to do some Scap release tesing, and this is just making it harder [17:02:08] yeah I just thought that a latest version might have fixed that ongoing issue [17:02:27] I guess we want to rollback to last known version ( 3.14.0-1+0~20200402164156.62~1.gbp529a68 ) [17:02:34] and move the jenkins job to use scap sync again [17:02:49] please don't mess with this now [17:03:49] yeah yeah i am not touching it anymore [17:04:05] just hinting how to restore to last known state. Cause beta is no more updating as a result [17:04:35] that's what he's working on :) [17:05:50] Yippee, build fixed! [17:05:50] Project beta-scap-eqiad build #314418: 09FIXED in 3 min 18 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/314418/ [17:05:57] :]]]]] [17:05:58] Hurrah. [17:06:24] It even vaguely looks like it actually worked, rather than just exit 0-ing wrongly. [17:06:36] magiiiic [17:07:01] thcipriani: are there 82 owners on wikimedia github org or is the link on your wikitech-l email broken? [17:07:01] 10Release-Engineering-Team, 10MediaWiki-Vagrant, 10Parsoid: Decide the fate of Parsoid/JS Puppet in Mediawiki-Vagrant - https://phabricator.wikimedia.org/T259988 (10ssastry) I've moved this to the non-parsing-team tasks column on the parsoid board since we haven't maintained this in vagrant before. But, if y... [17:07:21] I am off for dinner, will update some CI jobs later tonight [17:07:44] Majavah: https://github.com/orgs/wikimedia/people says 82 [17:08:09] hrmph, another safeloader problem [17:09:04] Reedy: does https://github.com/orgs/wikimedia/people?query=role%3Aowner say 82 or something else for you? [17:09:13] 82 [17:09:17] oh [17:09:32] doesn't actually say [17:09:33] I see 220 members and no count for role:owner afaict [17:09:45] 23 outside collaborators [17:10:06] I see 82 members total and 82 owners [17:10:28] GH offer an option to hide org membership [17:10:32] So that will explain some of the discrepency [17:11:11] Page 1 of that link (for me) has 12 private, 18 public for example [17:11:16] 10Phabricator (Upstream), 10Developer Productivity, 10Upstream: Disable parent task metadata by default for new sub tasks - https://phabricator.wikimedia.org/T239378 (10Jhernandez) I was talking with Andre the other day about this and he pointed me to this task. I want to show support for //Don't inherit by... [17:12:07] we non-members probably can't see owners then [17:12:29] so both links in that wikitech-l email are broken :/ [17:12:36] I think you can see a subset of owners [17:12:40] the non-private ones [17:12:48] (03PS1) 10Lars Wirzenius: undo: yaml safe loading [tools/scap] - 10https://gerrit.wikimedia.org/r/622397 [17:12:57] it does not tell me if they are owners or not [17:13:06] (03CR) 10Lars Wirzenius: [C: 03+2] undo: yaml safe loading [tools/scap] - 10https://gerrit.wikimedia.org/r/622397 (owner: 10Lars Wirzenius) [17:13:44] these folks are owners: https://github.com/orgs/wikimedia/people?query=role%3Aowner (although there does not seem to be anything special indicating they are org owners) [17:14:33] here's what it looks for me: https://i.imgur.com/ah6luin.png [17:14:48] for clarity: my attempt at preparing a Scap release continue, but since I don't (for now) have a place other than beta to test, beta will be a little unsteady for tonight [17:17:02] Majavah: sigh, evidently role:owner is only for members? or owners? [17:18:03] only visible for [17:18:35] thcipriani: I believe for members only, I can see who is an owner on another org I am a member in but can't for wikimedia [17:19:09] TIL [17:44:18] 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Scap, 10serviceops: Deploy Scap version 3.15.0-1 - https://phabricator.wikimedia.org/T261234 (10LarsWirzenius) [17:45:02] okay, I'm done messing with beta. new Scap release is tagged. [17:49:22] (03PS2) 10Thcipriani: deployment-calendar: alternative wikitext output [tools/release] - 10https://gerrit.wikimedia.org/r/621793 [17:49:36] (03CR) 10Thcipriani: deployment-calendar: alternative wikitext output (034 comments) [tools/release] - 10https://gerrit.wikimedia.org/r/621793 (owner: 10Thcipriani) [18:00:45] 10Beta-Cluster-Infrastructure, 10Performance-Team: deployment-webperf11 stale due to puppet failing - https://phabricator.wikimedia.org/T261236 (10Krinkle) [18:03:37] !log Restarted puppetdb on deployment-puppetdb03 (T248041) [18:03:39] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [18:03:40] T248041: puppetdb on deployment-puppetdb03 keeps getting OOMKilled - https://phabricator.wikimedia.org/T248041 [18:04:41] 10Beta-Cluster-Infrastructure, 10Performance-Team: deployment-webperf11 stale due to puppet failing - https://phabricator.wikimedia.org/T261236 (10dpifke) 05Open→03Resolved a:03dpifke Failing due to T248041. I restarted puppetdb. [18:08:54] 10MediaWiki-Codesniffer, 10MediaWiki-General, 10TechCom, 10at-ease, 10Proposal: Liberate the @ for AtEase - https://phabricator.wikimedia.org/T253461 (10Krinkle) Updated proposal to reflect feedback: > #### Proposal 2 > > * Keep coding conventions that discourage use of `@`. > * Update coding conventio... [18:09:06] 10MediaWiki-Codesniffer, 10MediaWiki-General, 10TechCom, 10at-ease, 10Proposal: Liberate the @ for AtEase - https://phabricator.wikimedia.org/T253461 (10Krinkle) [18:20:28] (03PS3) 10Umherirrender: Allow `@beforeClass` and `@afterClass` in FunctionAnnotationsSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/619871 (owner: 10Krinkle) [18:20:32] (03CR) 10Umherirrender: [C: 03+2] Allow `@beforeClass` and `@afterClass` in FunctionAnnotationsSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/619871 (owner: 10Krinkle) [18:20:44] 10Release-Engineering-Team-TODO: Get an IDE to run tests against services running in kubernetes - https://phabricator.wikimedia.org/T245656 (10thcipriani) 05Stalled→03Resolved VSCode works, PHPStorm does not have upstream support for this. Rather than leave this languish in our backlog, calling this completed! [18:21:42] (03Merged) 10jenkins-bot: Allow `@beforeClass` and `@afterClass` in FunctionAnnotationsSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/619871 (owner: 10Krinkle) [18:22:38] 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)): deployment-charts: Deploy script failing - kube_env: command not found - https://phabricator.wikimedia.org/T259684 (10thcipriani) 05Open→03Resolved Patch merged, haven't run a deployment using the patched version yet, but can open a new task/re... [18:22:50] (03CR) 10Umherirrender: [C: 03+1] "There are some usages: https://codesearch.wmcloud.org/search/?q=%5Cbis_resource%5Cb&i=nope&files=%5C.php&repos=" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/621948 (https://phabricator.wikimedia.org/T260735) (owner: 10Legoktm) [18:23:57] 10Release-Engineering-Team (Pipeline), 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Release Pipeline: PipelineLib helm test log output - https://phabricator.wikimedia.org/T255596 (10thcipriani) [18:24:40] 10Release-Engineering-Team (Pipeline), 10Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1)), 10Release Pipeline: PipelineLib helm test log output - https://phabricator.wikimedia.org/T255596 (10thcipriani) p:05Triage→03Medium [18:29:07] (03CR) 10Umherirrender: "There is not much impacting ourer standard," [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/619983 (owner: 10Umherirrender) [18:38:30] (03PS4) 10Umherirrender: Sort arrays with new @phpcs-require-sorted-array [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/606799 (https://phabricator.wikimedia.org/T247813) [18:38:45] (03CR) 10Umherirrender: Sort arrays with new @phpcs-require-sorted-array (033 comments) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/606799 (https://phabricator.wikimedia.org/T247813) (owner: 10Umherirrender) [18:43:35] (03CR) 10Thcipriani: "Looks good overall. I have a few inline questions/comments." (033 comments) [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/622231 (https://phabricator.wikimedia.org/T255835) (owner: 10Jeena Huneidi) [19:32:40] (03PS1) 10Hashar: jjb: update jobs to use /utils/ci-src-setup.sh [integration/config] - 10https://gerrit.wikimedia.org/r/622417 (https://phabricator.wikimedia.org/T141607) [19:35:18] (03CR) 10Hashar: [C: 04-1] jjb: update jobs to use /utils/ci-src-setup.sh [integration/config] - 10https://gerrit.wikimedia.org/r/622417 (https://phabricator.wikimedia.org/T141607) (owner: 10Hashar) [20:02:10] (03CR) 10Jeena Huneidi: Enable deployment-charts update for published image (033 comments) [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/622231 (https://phabricator.wikimedia.org/T255835) (owner: 10Jeena Huneidi) [20:48:57] (03CR) 10Hashar: [C: 03+1] "Excellent! Looks like Timo got addressed as well, will let him confirm and +2 :]" (031 comment) [tools/release] - 10https://gerrit.wikimedia.org/r/621793 (owner: 10Thcipriani) [20:49:44] (03PS5) 10Jeena Huneidi: Enable deployment-charts update for published image [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/622231 (https://phabricator.wikimedia.org/T255835) [20:59:47] 10Release-Engineering-Team (Logspam), 10MediaWiki-Patrolling, 10Wikimedia-General-or-Unknown, 10Platform Team Workboards (Clinic Duty Team), and 2 others: DBQueryError when using rollback on incubatorwiki - https://phabricator.wikimedia.org/T260798 (10daniel) [21:01:49] 10Release-Engineering-Team (Logspam), 10MediaWiki-Patrolling, 10Wikimedia-General-or-Unknown, 10Platform Team Workboards (Clinic Duty Team), and 2 others: DBQueryError when using rollback on incubatorwiki - https://phabricator.wikimedia.org/T260798 (10daniel) p:05Triage→03Medium [21:06:16] (03CR) 10Krinkle: [C: 03+1] deployment-calendar: alternative wikitext output [tools/release] - 10https://gerrit.wikimedia.org/r/621793 (owner: 10Thcipriani) [21:19:36] (03PS1) 10Jeena Huneidi: Print logs on helm test [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/622429 (https://phabricator.wikimedia.org/T255596) [22:21:19] 10Release-Engineering-Team, 10MediaWiki-Vagrant, 10Parsoid: Decide the fate of Parsoid/JS Puppet in Mediawiki-Vagrant - https://phabricator.wikimedia.org/T259988 (10Tgr) @ssastry questions for the Parsing team would be: - do you see any problem with Parsoid/JS being completely removed from Vagrant? (Is there... [22:59:59] 10Release-Engineering-Team, 10MediaWiki-General, 10MediaWiki-Stakeholders-Group, 10serviceops, and 4 others: Drop official PHP 7.2 support in MediaWiki 1.35 - https://phabricator.wikimedia.org/T257879 (10Reedy) {T261044} is somewhat amusing... The code in parsoid isn't actually compatible with PHP 7.2.22.... [23:06:34] 10Release-Engineering-Team, 10MediaWiki-Vagrant, 10Parsoid: Decide the fate of Parsoid/JS Puppet in Mediawiki-Vagrant - https://phabricator.wikimedia.org/T259988 (10cscott) You can still use RESTBase with Parsoid/PHP, and I'd suggest configuring things that way in vagrant. I'm fine w/ getting rid of support... [23:08:33] 10Release-Engineering-Team, 10MediaWiki-General, 10MediaWiki-Stakeholders-Group, 10serviceops, and 4 others: Drop official PHP 7.2 support in MediaWiki 1.35 - https://phabricator.wikimedia.org/T257879 (10matmarex) The array_key_first() method is provided in the Composer module 'symfony/polyfill-php73', whi... [23:22:04] 10Release-Engineering-Team, 10MediaWiki-General, 10MediaWiki-Stakeholders-Group, 10serviceops, and 4 others: Drop official PHP 7.2 support in MediaWiki 1.35 - https://phabricator.wikimedia.org/T257879 (10Reedy) ` $ composer depends symfony/polyfill-php73 __root__ dev-master requires symfony/polyf... [23:22:53] 10Release-Engineering-Team, 10MediaWiki-General, 10MediaWiki-Stakeholders-Group, 10serviceops, and 4 others: Drop official PHP 7.2 support in MediaWiki 1.35 - https://phabricator.wikimedia.org/T257879 (10Reedy) >>! In T257879#6410791, @matmarex wrote: > The array_key_first() method is provided in the Compo...