[03:35:43] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team (Radar), 10Parsoid: beta-scap-sync-world failing - https://phabricator.wikimedia.org/T317759 (10hashar) [03:37:46] 10Beta-Cluster-Infrastructure, 10Release-Engineering-Team (Radar), 10Parsoid: beta-scap-sync-world failing - https://phabricator.wikimedia.org/T317759 (10hashar) That was the same issue as T310069 albeit this time you got it fixed by blanking https://en.wikipedia.beta.wmflabs.org/wiki/Data:DutchMuni-json :]... [07:32:20] (03CR) 10Jforrester: "Hoping that we won't need this, indeed." [integration/config] - 10https://gerrit.wikimedia.org/r/831644 (https://phabricator.wikimedia.org/T316080) (owner: 10Brian Wolff) [08:22:30] Jdlrobson: Is there any reason you created https://phabricator.wikimedia.org/project/view/6153/ which is a duplicate of https://phabricator.wikimedia.org/project/manage/2740/ (and was already documented on the MW extension page)? [09:00:01] (03CR) 10Jforrester: [C: 03+1] Remove references to refreshMessageBlobs.php [tools/scap] - 10https://gerrit.wikimedia.org/r/831628 (https://phabricator.wikimedia.org/T314947) (owner: 10Ahmon Dancy) [09:33:17] (03CR) 10Krinkle: [C: 03+1] Remove references to refreshMessageBlobs.php [tools/scap] - 10https://gerrit.wikimedia.org/r/831628 (https://phabricator.wikimedia.org/T314947) (owner: 10Ahmon Dancy) [12:20:31] (03PS1) 10Hashar: Disable editorconfig indentation check for python [tools/train-dev] - 10https://gerrit.wikimedia.org/r/832635 [12:45:45] 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10Patch-For-Review, 10Release, 10Train Deployments: 1.39.0-wmf.27 deployment blockers - https://phabricator.wikimedia.org/T314188 (10kostajh) [13:26:37] (03CR) 10Hashar: "I have a few nitpicks and suggestion, nothing worth blocking really. I really like this port to python specially the queue/worker implem" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 (owner: 10Ahmon Dancy) [13:26:47] (03CR) 10Hashar: [C: 03+1] Replace mirror-repos.sh with mirror-repos.py [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 (owner: 10Ahmon Dancy) [14:50:20] (03CR) 10Ahmon Dancy: [C: 03+2] Disable editorconfig indentation check for python [tools/train-dev] - 10https://gerrit.wikimedia.org/r/832635 (owner: 10Hashar) [14:50:50] dancy: I will be off very soon, I send a few remarks here and there on your mirror repo.py script. "It is great" is the tldr ;) [14:50:58] (03Merged) 10jenkins-bot: Disable editorconfig indentation check for python [tools/train-dev] - 10https://gerrit.wikimedia.org/r/832635 (owner: 10Hashar) [14:51:07] Thanks hashar. I will process your comments today. [14:51:15] I had a few trivial suggestions here and there feel free to dismiss/amend your patch with them and then self +2 [14:51:26] else I might +2 later tonight [14:51:37] I very welcome that rewrite for sure ;) [14:51:45] and the Queue / workers is very clear [14:53:27] Does a tuple comparison like `sys.version_info >= (3, 6)` really work? [14:53:41] (that should be <=, btw) [14:53:51] I'll figure it out. [14:54:03] that review comment must have been booby trapped [14:54:25] so yeah surprisingly the tuple comparison works [14:54:30] at least with the few test case I did [14:54:32] cool [14:54:58] I think the comparison process them one after the other and short circuit as soon as the comparison operator is matched/mismatched [14:55:18] Makes sense.. just like string comparisons [14:55:18] so even though the rest of the parameters are missing (like patch version) , it is not a problem [14:55:31] yeah [14:56:00] I still had to manually verify it in the python repl just to be sure [14:56:02] or [14:56:08] I could have written a test covering the feat [14:56:09] ;D [14:56:34] Ah, I see sys.version_info is an object that defines a comparison operator. [14:56:41] hmm [14:56:47] it should be a named tuple [14:57:09] oh interesting. [15:08:34] (03PS7) 10Ahmon Dancy: Replace mirror-repos.sh with mirror-repos.py [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 [15:11:53] (03CR) 10Ahmon Dancy: Replace mirror-repos.sh with mirror-repos.py (034 comments) [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 (owner: 10Ahmon Dancy) [15:15:39] (03CR) 10Hashar: [C: 03+2] "Well done ;)" [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 (owner: 10Ahmon Dancy) [15:15:57] Thanks for the review Hashar! [15:15:59] for `git remote set-head origin --auto` I am not entirely sure it works [15:16:09] (03Merged) 10jenkins-bot: Replace mirror-repos.sh with mirror-repos.py [tools/train-dev] - 10https://gerrit.wikimedia.org/r/831940 (owner: 10Ahmon Dancy) [15:16:11] I must have faced that problem at some point in the past [15:16:14] hmm [15:16:22] how does CI merged that in less than 30 Seconds? [15:16:38] oh it just lints [15:16:47] nod [15:17:00] Running full tests would require access to docker [15:17:21] and having the train branch cutter to create a symbolic branch `wmf/latest` might solve a bunch of issues [15:17:39] yeah docker access, I really have to tackle that problem [15:17:49] I think I will eventually end up being bold [15:17:57] we will see [15:18:14] I think I will look at using Bazel for mediawiki over the week-end [15:18:37] speaking of week-end, it is time! Kudos on the bash > python conversion [15:18:44] Thanks. have a great weekend! [15:18:53] you too! [15:21:48] (03CR) 10Ahmon Dancy: [C: 03+2] Remove references to refreshMessageBlobs.php [tools/scap] - 10https://gerrit.wikimedia.org/r/831628 (https://phabricator.wikimedia.org/T314947) (owner: 10Ahmon Dancy) [15:26:51] (03Merged) 10jenkins-bot: Remove references to refreshMessageBlobs.php [tools/scap] - 10https://gerrit.wikimedia.org/r/831628 (https://phabricator.wikimedia.org/T314947) (owner: 10Ahmon Dancy) [15:47:11] !log Upgrading scap to latest code revision in beta cluster [15:47:12] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [15:47:45] (03PS1) 10Ahmon Dancy: Release 4.20.0-1 [tools/scap] - 10https://gerrit.wikimedia.org/r/832659 [15:47:47] (03CR) 10Ahmon Dancy: [C: 03+2] Release 4.20.0-1 [tools/scap] - 10https://gerrit.wikimedia.org/r/832659 (owner: 10Ahmon Dancy) [15:50:50] (03Merged) 10jenkins-bot: Release 4.20.0-1 [tools/scap] - 10https://gerrit.wikimedia.org/r/832659 (owner: 10Ahmon Dancy) [15:53:38] 10Scap, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-WikimediaMaintenance, 10Performance-Team, 10Technical-Debt: Remove old refreshMessageBlobs.php script from WikimediaMaintenance - https://phabricator.wikimedia.org/T314947 (10dancy) [15:54:02] 10Scap, 10MediaWiki-ResourceLoader, 10MediaWiki-extensions-WikimediaMaintenance, 10Performance-Team, 10Technical-Debt: Remove old refreshMessageBlobs.php script from WikimediaMaintenance - https://phabricator.wikimedia.org/T314947 (10dancy) The changes to scap have been deployed to production and tested. [15:57:18] (03PS1) 10Ahmon Dancy: Don't delete certs during train-dev clean [tools/train-dev] - 10https://gerrit.wikimedia.org/r/832662 [16:04:49] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Blocking 🧱), 10Wikimedia-Fundraising-CiviCRM: CI has somehow completely stopped for a pr at wikimedia/fundraising/crm master - https://phabricator.wikimedia.org/T317928 (10AnnWF) @dancy Yeah, I noticed this error while the pr has nothing to... [16:36:19] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Blocking 🧱), 10Wikimedia-Fundraising-CiviCRM: CI has somehow completely stopped for a pr at wikimedia/fundraising/crm master - https://phabricator.wikimedia.org/T317928 (10dancy) Hi @AnnWF I don't think the problem is specific to your accou... [16:36:23] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (Blocking 🧱), 10Wikimedia-Fundraising-CiviCRM: CI has somehow completely stopped for a pr at wikimedia/fundraising/crm master - https://phabricator.wikimedia.org/T317928 (10AnnWF) It's working now, thanks ! [16:48:56] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Support http_proxy, https_proxy and other proxy `build-arg:` options in blubber buildkit frontend - https://phabricator.wikimedia.org/T317997 (10dduvall) [18:26:43] (03CR) 10Thcipriani: [C: 03+2] scap backport: Allow URLs to have a trailing slash [tools/scap] - 10https://gerrit.wikimedia.org/r/832574 (owner: 10Ahmon Dancy) [18:31:58] (03Merged) 10jenkins-bot: scap backport: Allow URLs to have a trailing slash [tools/scap] - 10https://gerrit.wikimedia.org/r/832574 (owner: 10Ahmon Dancy) [18:33:06] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Support http_proxy, https_proxy and other proxy `build-arg:` options in blubber buildkit frontend - https://phabricator.wikimedia.org/T317997 (10dduvall) 05Open→03In progress p:05Triage→03Medium [18:33:10] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ), 10Patch-For-Review, 10User-brennen: Deploy buildkitd to trusted GitLab runners - https://phabricator.wikimedia.org/T308271 (10dduvall) [18:33:13] (03PS1) 10Dduvall: buildkit: Support Docker's build-arg options [blubber] - 10https://gerrit.wikimedia.org/r/832699 (https://phabricator.wikimedia.org/T317997) [18:36:26] (03PS2) 10Dduvall: buildkit: Support Docker's build-arg options [blubber] - 10https://gerrit.wikimedia.org/r/832699 (https://phabricator.wikimedia.org/T317997) [18:37:06] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ), 10Patch-For-Review: Explicitly config buildkitd with internal DNS nameserver - https://phabricator.wikimedia.org/T317904 (10dduvall) 05Open→03Resolved [18:37:11] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ), 10Patch-For-Review, 10User-brennen: Deploy buildkitd to trusted GitLab runners - https://phabricator.wikimedia.org/T308271 (10dduvall) [20:40:21] 10Phabricator: H408 doesn't seem to work with milestone board - https://phabricator.wikimedia.org/T318015 (10ldelench_wmf) [22:10:55] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10serviceops, 10serviceops-collab, 10Patch-For-Review: sort out mysql privileges for phab1004/phab2002 - https://phabricator.wikimedia.org/T315713 (10Dzahn) @dduvall Right now I must be missing something because I still get "access denied" for d... [22:13:10] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10serviceops, 10serviceops-collab, 10Patch-For-Review: sort out mysql privileges for phab1004/phab2002 - https://phabricator.wikimedia.org/T315713 (10Dzahn) I can confirm it works (from 2002) with the "phstats" user and the credentials from /etc... [22:14:50] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10Patch-For-Review, 10User-brennen: Deploy Phabricator with scap - https://phabricator.wikimedia.org/T313259 (10Dzahn) Please also see T315713#8243258 and try one more time. [22:29:49] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10serviceops, 10serviceops-collab, 10Patch-For-Review: move phabricator to new hardware generation - https://phabricator.wikimedia.org/T280597 (10Dzahn) [22:30:11] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10serviceops, 10serviceops-collab, 10Patch-For-Review: sort out mysql privileges for phab1004/phab2002 - https://phabricator.wikimedia.org/T315713 (10Dzahn) 05Resolved→03Open @Marostegui Sorry to be a pain, but while the "phstats" user works... [22:30:38] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10Patch-For-Review, 10User-brennen: Deploy Phabricator with scap - https://phabricator.wikimedia.org/T313259 (10Dzahn) Nevermind, I don't think it's going to work yet, reopened T315713#8243271 [22:31:18] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10serviceops, 10serviceops-collab, 10Patch-For-Review: sort out mysql privileges for phab1004/phab2002 - https://phabricator.wikimedia.org/T315713 (10Dzahn) a:05Dzahn→03None [22:32:48] 10Phabricator, 10Release-Engineering-Team (Bonus Level 🕹ī¸), 10DBA, 10serviceops, and 2 others: sort out mysql privileges for phab1004/phab2002 - https://phabricator.wikimedia.org/T315713 (10Dzahn) [22:34:53] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Buildkitd cannot publish image due to `certificate signed by unknown authority` - https://phabricator.wikimedia.org/T318019 (10dduvall) [22:35:32] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Buildkitd cannot publish image due to `certificate signed by unknown authority` - https://phabricator.wikimedia.org/T318019 (10dduvall) p:05Triage→03Medium [22:42:31] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Buildkitd cannot publish image due to `certificate signed by unknown authority` - https://phabricator.wikimedia.org/T318019 (10Dzahn) In class `profile::base::certificates` there is: ` $puppet_ssl_dir = puppet_ssldir()... [22:47:14] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Buildkitd cannot publish image due to `certificate signed by unknown authority` - https://phabricator.wikimedia.org/T318019 (10Dzahn) **./modules/profile/files/puppet/ca.production.pem** [22:48:03] (03PS11) 10Jeena Huneidi: Replace Lock with TimeoutLock [tools/scap] - 10https://gerrit.wikimedia.org/r/828075 (https://phabricator.wikimedia.org/T315531) [22:51:47] (03CR) 10CI reject: [V: 04-1] Replace Lock with TimeoutLock [tools/scap] - 10https://gerrit.wikimedia.org/r/828075 (https://phabricator.wikimedia.org/T315531) (owner: 10Jeena Huneidi) [22:54:36] (03PS12) 10Jeena Huneidi: Replace Lock with TimeoutLock [tools/scap] - 10https://gerrit.wikimedia.org/r/828075 (https://phabricator.wikimedia.org/T315531) [22:58:55] (03CR) 10CI reject: [V: 04-1] Replace Lock with TimeoutLock [tools/scap] - 10https://gerrit.wikimedia.org/r/828075 (https://phabricator.wikimedia.org/T315531) (owner: 10Jeena Huneidi) [23:01:18] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (Priority Backlog đŸ“Ĩ): Buildkitd cannot publish image due to `certificate signed by unknown authority` - https://phabricator.wikimedia.org/T318019 (10Dzahn) @dduvall The file above is the right CA for the cert in question. cert in private repo, verifie... [23:12:40] (03PS13) 10Jeena Huneidi: Replace Lock with TimeoutLock [tools/scap] - 10https://gerrit.wikimedia.org/r/828075 (https://phabricator.wikimedia.org/T315531)