[00:07:10] 10Release-Engineering-Team (Deployment services), 10Patch-For-Review, 10SDAW-MediaSearch (MediaSearch-ReleaseCandidate), 10Structured-Data-Backlog (Current Work), 10Wikimedia-extension-review-queue: Split MediaSearch out into its own extension - https://phabricator.wikimedia.org/T265939 (10AnneT) I think... [00:21:34] 10Release-Engineering-Team (Pipeline), 10MW-on-K8s, 10Operations, 10serviceops, 10Release Pipeline (Blubber): Deployment infrastructure for PHP microservices - https://phabricator.wikimedia.org/T261369 (10Krinkle) Some kind of `/deploy` repo seems needed, I think, as otherwise we would be deploying unaud... [00:22:27] Krinkle: The whole point of k8s was to avoid the deploy repo by having it in the k8s image. [00:22:47] Krinkle: It's not "some", it's all but two or so services that haven't migrated yet. [00:24:15] James_F: ack, but there's not shortcut for security here. we have to do *something*. whether a vendor repo, deploy repo, checked-in, cherry-picked local composer registry, something. [00:24:42] Pinned dependencies and security isolation was agreed as sufficient beforehand. [00:24:52] If you think that's wrong, file a task with Security to review? [00:25:28] pinned to what, where do we review what we are pinning to? [00:25:40] Pinned in package-lock.json. [00:26:22] Obviously humans can still C+2 commits they shouldn't, but that was thought sufficient. [00:26:40] lock files mean nothing, that just prevents it from changing unexpectedly. It doesn't tell us what the code actually is. [00:26:48] even cryptominers have a sha1 hash [00:27:13] Yes, but they rarely have the same sha1 hash as what we originally installed and have posioned npmjs.com's registry to take it over. [00:27:29] we're talking about composer right, not npm [00:27:41] OK, composer.lock and packagist.com. [00:27:43] Same thing. [00:27:45] I assume we still don't deploy random php libs that weren't audited first. [00:27:51] I hope not. [00:28:00] pinnig a version is just a defence against accidental changes. [00:28:01] Certainly, not in /vendor. [00:28:19] (Speaking of, please +2 the less.php bump. ;-)) [00:28:21] it's not a review. I"m just asking where the review would happen for a php service's dependencies, if not via a vendor/ [00:28:35] In the service's repo? [00:28:38] Where else? [00:28:49] The point of services is to isolated from each other. [00:28:56] so we check-in vendor in the same repo, and install with --no-dev? (no deploy repo) [00:28:57] that works [00:28:59] See "+2 is a big deal". [00:29:17] Yeah, the prod image of the service must be built with --no-dev. [00:29:23] that woudl meaen the docker build doesn't have to run composer, and we always have a copy in git. [00:29:36] I'm not agreeing to that part of your statement. :-) [00:29:38] I don't think that's what was discussed so far by others, but that seems good indeed. [00:30:15] if composer only runs during the build, then where is the lib's actual code seen by a human at least once? [00:30:40] Composer's run is to fetch and validate that the lock file matches the fetched code. [00:30:46] how is that differnet from deploying a random php lib that we haven't audited. where was it audited? [00:30:57] By the code reviewers of said library. [00:31:15] I'm refering to dependencies, not our own code. [00:31:19] Yes. [00:32:03] If you C+2 a patch in mw-libs-ShellBox or whatever, you're responsible for making sure that any change to the composer lock file is OK with Security etc. [00:32:23] ok, and how would someone go about making sure that it is OK, what would security do? [00:32:42] do we store someone's local vendor dir from the previous commit somewhere and diff against it? [00:32:55] No? [00:33:02] I'm sure you think you're right. [00:33:03] do we keep a list somewhere of audited versions/checksums? [00:33:09] But I don't, and I'm not the one you have to convince. [00:33:29] so that means the libs aren't audited anywhere [00:33:57] we just download unknown code, store its checksum, and the next update replace its checksum. We just validate that whatever I downloaded and didnt' review from composer is the the same as what docker will download. [00:34:53] sure, I'll esclalate. It wasnt understood about npm either until last month. [00:34:57] If you're not carefully reviewing changes to composer.lock that seems to be a you problem. [00:35:26] there is no code in composer.lock, ther eis nothing meaningful to review beyond it stating the same package name and version, and some unknown hash you've never seen before. [00:35:37] Anyway, I'll be interested to see how the task goes. [00:36:05] If it changes you review what packages were added and check if they have been accepted in that version. [00:36:43] Ideally there'd be a central list and a CI service validating against that, but of course this is all new. [00:36:50] All of that is good, and is part of validating. It is not part of auditing. I'm talking about auditing. [00:38:06] * Krinkle emails security, will follow-up [02:17:18] 10Beta-Cluster-Infrastructure, 10Growth-Team (Current Sprint), 10User-Urbanecm: Create beta bnwiki - https://phabricator.wikimedia.org/T270165 (10Etonkovidova) >>! In T270165#6693593, @Tgr wrote: > (Note the wiki does not have Growth features enabled ATM.) Will they be enabled soon? [02:27:21] 10Beta-Cluster-Infrastructure, 10Growth-Team (Current Sprint), 10User-Urbanecm: Create beta bnwiki - https://phabricator.wikimedia.org/T270165 (10Tgr) >>! In T270165#6693810, @Etonkovidova wrote: > Will they be enabled soon? I wasn't planning to (it would get it along with production bnwiki of course) but w... [04:01:44] how long does it take for a new wikitech account to propagate to gerrit? [04:02:58] (03PS7) 10Gergő Tisza: Add Gerrit report format [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647594 [04:03:00] (03PS7) 10Gergő Tisza: Add fix reporting to Gerrit robot comment reporter [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647595 [04:25:44] Shouldn't it be mostly instant? [04:38:12] Reedy: it should, but after 45 minutes it's still not working. I suppose at this point it is unlikely to be a lag issue. [04:38:39] I'm not sure we do much replication stuff with ldap [04:39:57] Using the correct username etc? [06:25:19] 10Beta-Cluster-Infrastructure, 10Growth-Team (Current Sprint), 10User-Urbanecm: Create beta bnwiki - https://phabricator.wikimedia.org/T270165 (10Urbanecm_WMF) I personally think it's useful to enable them prior to production, so we (especially Ankan) can actually test the features work correctly, since it's... [07:11:45] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikitech account with non-unique password - https://phabricator.wikimedia.org/T270233 (10Tgr) [07:12:19] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikitech account with non-unique password - https://phabricator.wikimedia.org/T270233 (10Tgr) We should probably rejects such registration on wikitech where it is easy to provide a non-useless error message. [07:12:20] Reedy: had to make sure I use a unique email address. Filed https://phabricator.wikimedia.org/T270233 [07:13:48] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikitech account with non-unique password - https://phabricator.wikimedia.org/T270233 (10Majavah) [07:58:07] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikitech account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10Peachey88) [08:23:38] 10Release-Engineering-Team, 10WVUI, 10Release Pipeline (Blubber): Use npm version 6.x on blubber-generated docker images - https://phabricator.wikimedia.org/T269957 (10hashar) **TLDR** do request SRE to import the npm 6.14.8 package from `buster-backports` to `apt.wikimedia.org` under a specific component.**... [08:28:56] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Operations, 10serviceops: replace doc1001.eqiad.wmnet with a buster VM - https://phabricator.wikimedia.org/T247653 (10hashar) [08:43:44] (03PS8) 10Gergő Tisza: Add Gerrit report format [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647594 [08:43:46] (03PS8) 10Gergő Tisza: Add fix reporting to Gerrit robot comment reporter [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647595 [08:49:04] (03CR) 10Gergő Tisza: Add Gerrit report format (032 comments) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647594 (owner: 10Gergő Tisza) [08:53:39] (03PS9) 10Gergő Tisza: Add fix reporting to Gerrit robot comment reporter [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647595 [08:58:02] (03CR) 10jerkins-bot: [V: 04-1] Add fix reporting to Gerrit robot comment reporter [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647595 (owner: 10Gergő Tisza) [09:03:50] (03CR) 10Hashar: "Nice thanks for completing! :)" [integration/config] - 10https://gerrit.wikimedia.org/r/628589 (owner: 10Jforrester) [09:20:00] (03CR) 10Hashar: Add mediawiki project pipeline wmf-publish (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/649441 (https://phabricator.wikimedia.org/T269617) (owner: 10Dduvall) [09:30:44] (03CR) 10Hashar: "Hi Kunal, I would like to avoid introducing yet another programming language to the stack. We have already problems keeping up with the d" [integration/utils-rs] - 10https://gerrit.wikimedia.org/r/647995 (owner: 10Legoktm) [10:10:42] 10Release-Engineering-Team (Deployment services), 10Patch-For-Review, 10SDAW-MediaSearch (MediaSearch-ReleaseCandidate), 10Structured-Data-Backlog (Current Work), 10Wikimedia-extension-review-queue: Split MediaSearch out into its own extension - https://phabricator.wikimedia.org/T265939 (10matthiasmullie... [10:19:51] 10Continuous-Integration-Infrastructure, 10Operations: Have linters/tests results show up as comments in files on gerrit - https://phabricator.wikimedia.org/T209149 (10kostajh) @hashar @Tgr @awight I was sitting down to look at Quibble to see how we could add support for the Gerrit reporting / fix format that... [10:47:16] 10Continuous-Integration-Infrastructure, 10Operations: Have linters/tests results show up as comments in files on gerrit - https://phabricator.wikimedia.org/T209149 (10Tgr) We could also make a report which inherits from the default one and outputs the POST body into a file as a side effect. Note that GerritR... [10:59:29] (03PS10) 10Gergő Tisza: Add fix reporting to Gerrit robot comment reporter [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647595 [11:55:16] 10Continuous-Integration-Infrastructure: Phase out python3.4 from CI infrastructure - https://phabricator.wikimedia.org/T270268 (10hashar) [11:55:41] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Patch-For-Review: Provide python 3.8 in CI test images - https://phabricator.wikimedia.org/T268850 (10hashar) [11:55:46] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services): Phase out python3.4 from CI infrastructure - https://phabricator.wikimedia.org/T270268 (10hashar) [12:18:31] 10Release-Engineering-Team (Pipeline), 10MW-on-K8s, 10Operations, 10serviceops, 10Release Pipeline (Blubber): Deployment infrastructure for PHP microservices - https://phabricator.wikimedia.org/T261369 (10akosiaris) >>! In T261369#6693707, @Krinkle wrote: > Some kind of `/deploy` repo seems needed, I thi... [13:46:50] 10Phabricator, 10Developer-Advocacy (Jan-Mar 2021): Review the default options in Phab's Favorites menu; offer dedicated forms for bug reports vs feature requests? - https://phabricator.wikimedia.org/T253744 (10Aklapper) [15:37:36] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Operations, 10serviceops, and 2 others: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10hnowlan) mw1265 is now reimaged and pooled with weight 5 (as opposed to its pre... [15:38:27] 10Release-Engineering-Team (Deployment services), 10Release-Engineering-Team-TODO, 10Operations, 10serviceops, and 2 others: Upgrade MediaWiki clusters to Debian Buster (debian 10) - https://phabricator.wikimedia.org/T245757 (10jijiki) [16:25:51] (03CR) 10Ahmon Dancy: [C: 03+1] php.go: Pass --no-scripts to composer for extra hardening [blubber] - 10https://gerrit.wikimedia.org/r/649738 (https://phabricator.wikimedia.org/T270207) (owner: 10Legoktm) [16:28:30] (03CR) 10Ahmon Dancy: [C: 03+1] Fix documentation for deploy step [integration/pipelinelib] - 10https://gerrit.wikimedia.org/r/649445 (owner: 10Dduvall) [16:30:10] (03CR) 10Ahmon Dancy: [C: 03+2] Update Mediawiki Chart [releng/local-charts] - 10https://gerrit.wikimedia.org/r/649458 (owner: 10Jeena Huneidi) [16:39:10] (03CR) 10Jeena Huneidi: [C: 03+2] "LGTM, thanks!" [blubber] - 10https://gerrit.wikimedia.org/r/649738 (https://phabricator.wikimedia.org/T270207) (owner: 10Legoktm) [16:43:04] (03Merged) 10jenkins-bot: php.go: Pass --no-scripts to composer for extra hardening [blubber] - 10https://gerrit.wikimedia.org/r/649738 (https://phabricator.wikimedia.org/T270207) (owner: 10Legoktm) [16:46:19] 10Phabricator, 10Security: if phabricator users are blocked due to IP range blocks send a 4xx rather than 500 - https://phabricator.wikimedia.org/T229620 (10Dzahn) 05Open→03Resolved a:03Dzahn Apparently this has been fixed by someone somewhere (Nice, but would have been great to get an update here). T27... [16:53:45] (03PS10) 10Brennen Bearnes: WIP: Move xdebug build & php entrypoint to stretch; xdebug 3.0.1 [releng/dev-images] - 10https://gerrit.wikimedia.org/r/640567 [17:08:38] (03CR) 10Ammarpad: "What is the problem caused by $this->assertTrue( true )? The task just link to another task where one extension chose to not use it for wh" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647071 (https://phabricator.wikimedia.org/T268509) (owner: 10DannyS712) [17:12:08] 10Release-Engineering-Team, 10WVUI, 10Release Pipeline (Blubber): Use npm version 6.x on blubber-generated docker images - https://phabricator.wikimedia.org/T269957 (10nnikkhoui) `On Debian that is two separate packages and they are not kept in sync.` I was actually really curious about that, I was originall... [17:23:24] (03CR) 10DannyS712: "> Patch Set 5:" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647071 (https://phabricator.wikimedia.org/T268509) (owner: 10DannyS712) [17:28:25] 10Release-Engineering-Team, 10WVUI, 10Release Pipeline (Blubber): Use npm version 6.x on blubber-generated docker images - https://phabricator.wikimedia.org/T269957 (10hashar) > Thanks a ton for all the background @hashar, your suggestion all makes sense. Glad it helped :] > Will put in a request to add 6.... [17:31:14] legoktm: So… when we get codesniffer and phan fully working on PHP 8.0, we'll need LibUp to try to apply them to REL1_35 branches. Any chance you could have a look at Reedy's branch support patch? :-) [17:31:40] (03CR) 10Jforrester: [C: 03+1] Configure pipeline for mediawiki/libs/Shellbox [integration/config] - 10https://gerrit.wikimedia.org/r/649745 (owner: 10Legoktm) [17:34:26] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikitech account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10bd808) >>! In T270233#6694081, @Tgr wrote: > We should probably reject such registration on wikitech where it is easy to provide a non-useless error m... [17:36:37] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikimedia Developer account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10bd808) [17:42:09] 10Release-Engineering-Team, 10WVUI, 10Release Pipeline (Blubber): Use npm version 6.x on blubber-generated docker images - https://phabricator.wikimedia.org/T269957 (10nnikkhoui) AHHHHH. Got it. Thanks!! [17:53:58] James_F: ack [17:54:02] marxarelli: in case you were wondering what I was working towards with the Blubber patch -- https://gerrit.wikimedia.org/r/c/wikimedia/toolhub/+/648395/5/.pipeline/blubber.yaml [17:54:09] and it is working :) [17:56:55] bd808: \o/ [17:56:58] that's great! [17:57:12] also, you win the award for largest blubber config :) [17:57:22] Is that an award? :-) [17:57:32] I haven't even gotten to the complicated bits yet ;) [17:58:14] James_F: 🏆 <- award, yes (you can't read the inscription but it's very kind [17:58:27] * James_F grins. [17:58:51] I think the plaque should mostly give this url: https://en.wikipedia.org/wiki/Exploding_whale [17:59:06] haha [17:59:11] that's perfect [18:00:55] bd808: i've updated the on-wiki docs for builder.requirements https://wikitech.wikimedia.org/wiki/Blubber/User_Guide#Variant_Config [18:01:23] it looks like your feature might already be useful to others. legoktm, i left a comment about it on your shellbox ps ^ [18:01:55] oh cool [18:01:59] i'm going to see about refactoring the other requirements fields based on it [18:02:13] * bd808 hopes that will be fairly easy [18:27:14] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikimedia Developer account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10Tgr) Thanks for fixing the task description, clearly I was half asleep. @bd808 is a wikitech fix worth it separately or should it be fixed... [19:11:00] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikimedia Developer account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10bd808) I'm not sure that preventing duplicate emails in the LDAP directory is actually valuable at all. It seems more like an implementatio... [19:33:33] 10Gerrit, 10wikitech.wikimedia.org: Can't login into Gerrit with a Wikimedia Developer account with non-unique email address - https://phabricator.wikimedia.org/T270233 (10hashar) [19:34:32] 10Gerrit, 10Trust-and-Safety, 10cloud-services-team (Kanban): Cannot login gerrit.wikimedia.org with wikitech.mediawiki.org account - https://phabricator.wikimedia.org/T270064 (10hashar) I am very happy you have managed to reset the password. Thank you for your patience while it was being figured out. [19:35:37] (03CR) 10Jeena Huneidi: [C: 03+1] "LGTM" [integration/config] - 10https://gerrit.wikimedia.org/r/649745 (owner: 10Legoktm) [19:45:34] 10Release-Engineering-Team (Logspam), 10Growth-Team, 10Wikimedia-production-error: Flow: could not locate workflow for revision {rev} - https://phabricator.wikimedia.org/T270334 (10thcipriani) [19:59:29] (03CR) 10Jforrester: [C: 03+2] "Nice improvement, thank you." [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647070 (https://phabricator.wikimedia.org/T242762) (owner: 10DannyS712) [20:01:09] (03CR) 10Jforrester: [C: 03+1] Create PHPUnitAssertEmpty sniff to warn against using assertEmpty [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/591242 (https://phabricator.wikimedia.org/T246674) (owner: 10DannyS712) [20:01:19] (03Merged) 10jenkins-bot: New sniff: MediaWiki.PHPUnit.AssertCount [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/647070 (https://phabricator.wikimedia.org/T242762) (owner: 10DannyS712) [20:01:46] 10MediaWiki-Codesniffer, 10Patch-For-Review, 10User-DannyS712, 10User-thiemowmde: Expand PHPUnitAssertEqualsSniff to better auto-fix comparisons with count() - https://phabricator.wikimedia.org/T242762 (10DannyS712) 05Open→03Resolved [20:01:53] 10MediaWiki-Codesniffer, 10User-DannyS712, 10User-thiemowmde: Expand PHPUnitAssertEqualsSniff to better auto-fix comparisons with count() - https://phabricator.wikimedia.org/T242762 (10DannyS712) [20:02:13] 10MediaWiki-Codesniffer, 10User-DannyS712, 10User-thiemowmde: Expand PHPUnitAssertEqualsSniff to better auto-fix comparisons with count() - https://phabricator.wikimedia.org/T242762 (10DannyS712) ->accepted based on the fact that it was merged [20:06:13] 10Release-Engineering-Team, 10Release-Engineering-Team-TODO (2020-10-01 to 2020-12-31 (Q2)), 10MediaWiki-Docker, 10User-brennen: Set up distribution of MediaWiki-Docker CLI - https://phabricator.wikimedia.org/T250241 (10jeena) a:03jeena [20:41:26] 10MediaWiki-Releasing, 10Security: Write and send pre-release announcements for MediaWiki 1.31.11/1.35.1 - https://phabricator.wikimedia.org/T263804 (10Reedy) 05Open→03Resolved p:05Triage→03Medium https://lists.wikimedia.org/pipermail/mediawiki-announce/2020-December/000267.html [20:41:32] 10MediaWiki-Releasing, 10Security: Write and send pre-release announcements for MediaWiki 1.31.11/1.35.1 - https://phabricator.wikimedia.org/T263804 (10Reedy) [21:55:59] 10Release-Engineering-Team (Logspam), 10Growth-Team, 10StructuredDiscussions, 10Wikimedia-production-error: Flow: could not locate workflow for revision {rev} - https://phabricator.wikimedia.org/T270334 (10Aklapper) [22:16:13] 10Release-Engineering-Team, 10Gerrit-Privilege-Requests, 10TechCom: Create WikiTeq group on Gerrit - https://phabricator.wikimedia.org/T267213 (10tstarling) 05Open→03Resolved a:03tstarling Done. Please review the [[https://gerrit.wikimedia.org/r/admin/groups/7badf5846dad8ef6b9101094b2d5c80f6d08b66f,mem... [22:22:44] James_F: I was just about to ask you about that :) [22:23:02] * James_F grins. [22:23:34] I didn't realize how special this repo was, I'm trying to figure out how to adequately test 8.0 support [22:24:18] The 'replacement' for npm-php is node10-test-browser-php72-composer. [22:24:44] But I've not added the php{7{3,4},80} variants of that image. [22:24:49] Because it only exist(ed) for OOUI. [22:26:24] Feel free. ;-) [22:26:26] I think we also want to configure php-parallel-lint and the composer-package-* jobs too [22:26:54] node10-test-browser-php72-composer lets you test whatever composer jobs you want. [22:28:45] (03PS1) 10Jforrester: jjb: [wikipeg-npm-php-docker] Stop using ancient npm-php image [integration/config] - 10https://gerrit.wikimedia.org/r/649968 [22:28:47] (03PS1) 10Jforrester: dockerfiles: Drop npm-php, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649969 [22:28:58] legoktm: ^^ Untested, but at lest that's a start. [22:29:56] I don't know the library well enough to know if we could get away with distinct npm test and composer test jobs, like everything else has. [22:30:15] Or if, like OOUI, it has composer and npm calling each other and so both need to be present simultaneously. [22:30:39] Let me peruse. [22:31:17] I think the nodejs code generates PHP? not exactly sure [22:36:22] Well, it fails in the basic node10-test-browser image, so let's go with this change for now. [22:36:53] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/649968 (owner: 10Jforrester) [22:38:04] legoktm: Also the repo needs adding to LibUp's list of libraries that aren't under mediawiki/libs/ but should be. :-( [22:38:15] (03Merged) 10jenkins-bot: jjb: [wikipeg-npm-php-docker] Stop using ancient npm-php image [integration/config] - 10https://gerrit.wikimedia.org/r/649968 (owner: 10Jforrester) [22:41:32] (03PS2) 10Jforrester: dockerfiles: Drop npm-php, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649969 [22:41:34] (03PS1) 10Jforrester: jjb: [wikipeg-npm-php-docker] Define entrypoint (oops), use npm cit [integration/config] - 10https://gerrit.wikimedia.org/r/649974 [22:43:38] (03CR) 10Jforrester: [C: 03+2] jjb: [wikipeg-npm-php-docker] Define entrypoint (oops), use npm cit [integration/config] - 10https://gerrit.wikimedia.org/r/649974 (owner: 10Jforrester) [22:43:42] (03CR) 10Jforrester: [C: 03+2] dockerfiles: Drop npm-php, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649969 (owner: 10Jforrester) [22:45:05] (03Merged) 10jenkins-bot: jjb: [wikipeg-npm-php-docker] Define entrypoint (oops), use npm cit [integration/config] - 10https://gerrit.wikimedia.org/r/649974 (owner: 10Jforrester) [22:45:07] (03Merged) 10jenkins-bot: dockerfiles: Drop npm-php, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649969 (owner: 10Jforrester) [22:45:42] legoktm: https://gerrit.wikimedia.org/r/c/labs/libraryupgrader/+/649977 [22:46:01] legoktm: So, want me to build a php80 version of the image? [22:56:01] (03PS1) 10Jforrester: jjb: Drop mwgate-npm-browser-node-6-docker, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649978 [22:56:28] (03CR) 10Jforrester: [C: 03+2] "One down, 628 remaining." [integration/config] - 10https://gerrit.wikimedia.org/r/649978 (owner: 10Jforrester) [22:57:51] (03Merged) 10jenkins-bot: jjb: Drop mwgate-npm-browser-node-6-docker, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649978 (owner: 10Jforrester) [22:59:06] James_F: yes please, but idk if I'm going to get to more wikipeg stuff today so no rush [22:59:21] (03PS1) 10Jforrester: jjb: Drop npm-node-6-docker, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649980 [23:29:34] (03PS1) 10Jforrester: dockerfiles: Provide node10-test-browser-php80-composer [integration/config] - 10https://gerrit.wikimedia.org/r/649984 [23:29:36] (03PS1) 10Jforrester: jjb: Provide PHP-versioned names of OOUI and Wikipeg specials [integration/config] - 10https://gerrit.wikimedia.org/r/649985 [23:29:38] (03PS1) 10Jforrester: Zuul: Switch OOUI and Wikipeg over to versioned special jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649986 [23:29:38] legoktm: Tah-dah. [23:29:40] (03PS1) 10Jforrester: jjb: Drop unversioned OOUI and Wikipeg special jobs, unused [integration/config] - 10https://gerrit.wikimedia.org/r/649987 [23:29:42] (03PS1) 10Jforrester: jjb: [wikipeg-special-node10-plus-php72-composer-docker] Use same image [integration/config] - 10https://gerrit.wikimedia.org/r/649988 [23:29:44] (03PS1) 10Jforrester: jjb: Provide PHP 8.0 variants of OOUI and Wikipeg jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649989 [23:29:46] (03PS1) 10Jforrester: Zuul: Add experimental PHP 8.0 jobs for OOUI and Wikipeg [integration/config] - 10https://gerrit.wikimedia.org/r/649990 [23:30:07] <3 [23:30:29] (Assuming that works, of course. ;-)) [23:31:43] (03CR) 10jerkins-bot: [V: 04-1] jjb: Provide PHP-versioned names of OOUI and Wikipeg specials [integration/config] - 10https://gerrit.wikimedia.org/r/649985 (owner: 10Jforrester) [23:31:45] (03CR) 10jerkins-bot: [V: 04-1] jjb: [wikipeg-special-node10-plus-php72-composer-docker] Use same image [integration/config] - 10https://gerrit.wikimedia.org/r/649988 (owner: 10Jforrester) [23:31:47] (03CR) 10jerkins-bot: [V: 04-1] dockerfiles: Provide node10-test-browser-php80-composer [integration/config] - 10https://gerrit.wikimedia.org/r/649984 (owner: 10Jforrester) [23:31:49] (03CR) 10jerkins-bot: [V: 04-1] jjb: Provide PHP 8.0 variants of OOUI and Wikipeg jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649989 (owner: 10Jforrester) [23:31:51] (03CR) 10jerkins-bot: [V: 04-1] Zuul: Switch OOUI and Wikipeg over to versioned special jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649986 (owner: 10Jforrester) [23:31:56] (03CR) 10jerkins-bot: [V: 04-1] jjb: Drop unversioned OOUI and Wikipeg special jobs, unused [integration/config] - 10https://gerrit.wikimedia.org/r/649987 (owner: 10Jforrester) [23:31:58] Psh, that jerkins, such a critic. [23:32:07] (03CR) 10jerkins-bot: [V: 04-1] Zuul: Add experimental PHP 8.0 jobs for OOUI and Wikipeg [integration/config] - 10https://gerrit.wikimedia.org/r/649990 (owner: 10Jforrester) [23:32:41] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/649980 (owner: 10Jforrester) [23:34:26] (03Merged) 10jenkins-bot: jjb: Drop npm-node-6-docker, now unused [integration/config] - 10https://gerrit.wikimedia.org/r/649980 (owner: 10Jforrester) [23:35:38] (03PS2) 10Jforrester: dockerfiles: Provide node10-test-browser-php80-composer [integration/config] - 10https://gerrit.wikimedia.org/r/649984 [23:35:39] (03PS2) 10Jforrester: jjb: Provide PHP-versioned names of OOUI and Wikipeg specials [integration/config] - 10https://gerrit.wikimedia.org/r/649985 [23:35:41] (03PS2) 10Jforrester: Zuul: Switch OOUI and Wikipeg over to versioned special jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649986 [23:35:43] (03PS2) 10Jforrester: jjb: Drop unversioned OOUI and Wikipeg special jobs, unused [integration/config] - 10https://gerrit.wikimedia.org/r/649987 [23:35:45] (03PS2) 10Jforrester: jjb: [wikipeg-special-node10-plus-php72-composer-docker] Use same image [integration/config] - 10https://gerrit.wikimedia.org/r/649988 [23:35:49] (03PS2) 10Jforrester: jjb: Provide PHP 8.0 variants of OOUI and Wikipeg jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649989 [23:35:51] (03PS2) 10Jforrester: Zuul: Add experimental PHP 8.0 jobs for OOUI and Wikipeg [integration/config] - 10https://gerrit.wikimedia.org/r/649990 [23:37:42] (03CR) 10jerkins-bot: [V: 04-1] jjb: [wikipeg-special-node10-plus-php72-composer-docker] Use same image [integration/config] - 10https://gerrit.wikimedia.org/r/649988 (owner: 10Jforrester) [23:37:55] (03CR) 10jerkins-bot: [V: 04-1] jjb: Provide PHP 8.0 variants of OOUI and Wikipeg jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649989 (owner: 10Jforrester) [23:38:07] (03CR) 10Jforrester: [C: 03+2] dockerfiles: Provide node10-test-browser-php80-composer [integration/config] - 10https://gerrit.wikimedia.org/r/649984 (owner: 10Jforrester) [23:38:16] (03CR) 10jerkins-bot: [V: 04-1] Zuul: Add experimental PHP 8.0 jobs for OOUI and Wikipeg [integration/config] - 10https://gerrit.wikimedia.org/r/649990 (owner: 10Jforrester) [23:39:25] (03Merged) 10jenkins-bot: dockerfiles: Provide node10-test-browser-php80-composer [integration/config] - 10https://gerrit.wikimedia.org/r/649984 (owner: 10Jforrester) [23:40:56] !log Docker: Publishing node10-test-browser-php80-composer 0.0.1 [23:40:59] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [23:41:16] (03CR) 10Jforrester: [C: 03+2] jjb: Provide PHP-versioned names of OOUI and Wikipeg specials [integration/config] - 10https://gerrit.wikimedia.org/r/649985 (owner: 10Jforrester) [23:42:39] (03Merged) 10jenkins-bot: jjb: Provide PHP-versioned names of OOUI and Wikipeg specials [integration/config] - 10https://gerrit.wikimedia.org/r/649985 (owner: 10Jforrester) [23:43:32] (03CR) 10Jforrester: [C: 03+2] Zuul: Switch OOUI and Wikipeg over to versioned special jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649986 (owner: 10Jforrester) [23:44:53] (03Merged) 10jenkins-bot: Zuul: Switch OOUI and Wikipeg over to versioned special jobs [integration/config] - 10https://gerrit.wikimedia.org/r/649986 (owner: 10Jforrester) [23:48:47] !log Zuul: Switch OOUI and Wikipeg over to versioned special jobs [23:48:49] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [23:53:05] (03CR) 10Jforrester: [C: 03+2] "Deployed." [integration/config] - 10https://gerrit.wikimedia.org/r/649987 (owner: 10Jforrester) [23:54:30] (03Merged) 10jenkins-bot: jjb: Drop unversioned OOUI and Wikipeg special jobs, unused [integration/config] - 10https://gerrit.wikimedia.org/r/649987 (owner: 10Jforrester) [23:55:36] 10Release-Engineering-Team, 10Gerrit-Privilege-Requests, 10TechCom: Create WikiTeq group on Gerrit - https://phabricator.wikimedia.org/T267213 (10DannyS712) >>! In T267213#6697149, @tstarling wrote: > Done. Please review the [[https://gerrit.wikimedia.org/r/admin/groups/7badf5846dad8ef6b9101094b2d5c80f6d08b6... [23:55:40] (03PS3) 10Jforrester: jjb: [wikipeg-special-node10-plus-php72-composer-docker] Use same image [integration/config] - 10https://gerrit.wikimedia.org/r/649988