[00:27:55] 10Project-Admins: Create Phabricator project for Abstract Wikipedia - https://phabricator.wikimedia.org/T257007 (10Krinkle) [00:38:05] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: RFC: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10Demian) Yes: https://www.mediawiki.org/wiki/Talk:Wikimedia_Release_Engineering_Team/GitLab >>! In T167547#6273071, @Nemo_bis wrote: > Because it was not mentioned clearly in t... [00:41:54] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: RFC: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10Reedy) >>! In T167547#6276057, @Demian wrote: > Yes: https://www.mediawiki.org/wiki/Talk:Wikimedia_Release_Engineering_Team/GitLab Yes what? https://www.mediawiki.org/wiki/To... [00:55:01] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10Demian) [00:55:09] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10Demian) My bad, I misunderstood the rename and the private discussion on https://www.mediawiki.org/wiki/Talk:Wikimedia_Release_Engineering_Team/GitLab as a preparation for RFC. No R... [04:26:38] (03CR) 10Legoktm: [C: 03+2] shared: Consistently use "Test coverage" [integration/docroot] - 10https://gerrit.wikimedia.org/r/608942 (owner: 10Legoktm) [04:27:32] (03Merged) 10jenkins-bot: shared: Consistently use "Test coverage" [integration/docroot] - 10https://gerrit.wikimedia.org/r/608942 (owner: 10Legoktm) [04:59:41] (03PS1) 10Legoktm: Add CheckUser to Thanks' phan dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/609279 [05:01:03] (03CR) 10DannyS712: [C: 03+1] Add CheckUser to Thanks' phan dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/609279 (owner: 10Legoktm) [05:01:41] (03CR) 10Legoktm: [C: 03+2] Add CheckUser to Thanks' phan dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/609279 (owner: 10Legoktm) [05:02:39] (03Merged) 10jenkins-bot: Add CheckUser to Thanks' phan dependencies [integration/config] - 10https://gerrit.wikimedia.org/r/609279 (owner: 10Legoktm) [05:03:05] !log deploying https://gerrit.wikimedia.org/r/609279 [05:03:07] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [05:16:13] 10Beta-Cluster-Infrastructure, 10Jade, 10MediaWiki-ContentHandler, 10Patch-For-Review, 10User-DannyS712: Beta cluster: The content model 'JadeJudgment' is not registered - https://phabricator.wikimedia.org/T247476 (10DannyS712) Just ran into this again - I suggest simply deleting all pages with the JadeJ... [06:21:43] (03PS4) 10Thiemo Kreuz (WMDE): Add (optional) FullQualifiedClassNameSniff from WikibaseCodeSniffer [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/603496 (https://phabricator.wikimedia.org/T164653) [06:49:33] 10Continuous-Integration-Config, 10MediaWiki-Releasing, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1))), and 2 others: Configure CI to support REL1_35 - https://phabricator.wikimedia.org/T256377 (10hashar) T... [06:59:45] 10Gerrit, 10Regression: Gerrit commit message body is always wrapped with MS Edge - https://phabricator.wikimedia.org/T256964 (10Xqt) >>! In T256964#6274601, @QChris wrote: > I'm lacking access to MS Edge. Could you please post a screenshot? Sure. The commit message: {F31915175} And as it is shown with Gerr... [07:11:31] 10Gerrit, 10Regression: Gerrit commit message body is always wrapped with MS Edge - https://phabricator.wikimedia.org/T256964 (10hashar) If I understood CSS properly, the wrapping rule comes from the first rule in: ` lang=css :host([pre]) span { white-space: var(--linked-text-white-space, pre-wrap); wo... [07:36:32] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10hashar) Can we stop using this task entirely please? It has been filed in 2017 for a project to migrate out of Gerrit to Phabricator Differential. It is no more related to the 20... [07:47:58] 10Gerrit, 10DBA, 10Patch-For-Review: Make sure both `reviewdb-test` (used forgerrit upgrade testing) and `reviewdb` (formerly production) databases get torn down - https://phabricator.wikimedia.org/T255715 (10jcrespo) > If we can still keep the backups for a bit (say ... 1 month?), We keep database backups... [08:31:59] 10Beta-Cluster-Infrastructure, 10Scap: beta-publish-deb failure do to "/srv/packages/db/LOG: permission denied" - https://phabricator.wikimedia.org/T250157 (10hashar) @dancy and I quickly talked about this job failure yesterday. So here is the whole context, it is a bit long cause well I have a tendency to lik... [08:35:06] 10Beta-Cluster-Infrastructure, 10Scap: beta-publish-deb failure do to "/srv/packages/db/LOG: permission denied" - https://phabricator.wikimedia.org/T250157 (10hashar) TLDR: the issue is most probably caused by Puppet running some aptly command as root which thus lead to files changing ownership. The root cau... [09:02:55] 10Gerrit, 10DBA, 10Patch-For-Review: Make sure both `reviewdb-test` (used forgerrit upgrade testing) and `reviewdb` (formerly production) databases get torn down - https://phabricator.wikimedia.org/T255715 (10QChris) >>! In T255715#6276372, @jcrespo wrote: > We keep database backups of m2 currently for a bit... [09:04:41] 10Gerrit: Gerrit: Permission error "use a SHA1 visible to you, or get update permission on the ref" when using git-review ≤1.26 - https://phabricator.wikimedia.org/T256671 (10QChris) Since this seems to be happening again and again, @legoktm and I suggested a change #upstream at https://gerrit-review.googlesourc... [09:05:03] 10Gerrit, 10TechCom-RFC, 10Wikimedia-GitHub: Replace Gerrit & GitHub with GitLab - https://phabricator.wikimedia.org/T167547 (10Aklapper) This task is three years old. And it was declined. This task is not actionable (and in my personal opinion is such a vague idea that it should have never been created). T... [09:11:13] 10Gerrit: Gerrit: Permission error "use a SHA1 visible to you, or get update permission on the ref" or "You need 'Create' rights to create new references." when using git-review ≤1.26 - https://phabricator.wikimedia.org/T256671 (10QChris) [09:22:37] 10Gerrit, 10Browser-Support-Microsoft-Edge, 10Regression: Gerrit commit message body is always wrapped with MS Edge - https://phabricator.wikimedia.org/T256964 (10Aklapper) [09:32:42] (03Abandoned) 10Thiemo Kreuz (WMDE): Use preferred order of tags within codesniffer [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/591501 (https://phabricator.wikimedia.org/T175374) (owner: 10DannyS712) [09:52:59] 10Gerrit, 10Code-Health-Metrics, 10Sonarqubebot: Stop SonarQube spam - https://phabricator.wikimedia.org/T256942 (10kostajh) > – it seems like whoever wrote that special Gerrit integration didn’t think that the bot could ever be wrong about something Yes, we (#code-health-metrics group) have thought about a... [10:24:46] 10phan-taint-check-plugin, 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10MW-1.35-notes (1.35.0-wmf.8; 2019-11-26): Add phan-taint-check-plugin to DonationInterface extension - https://phabricator.wikimedia.org/T202384 (10Daimona) [10:24:49] 10phan, 10MediaWiki-extensions-DonationInterface: Add phan to DonationInterface extension - https://phabricator.wikimedia.org/T224763 (10Daimona) [10:25:04] 10phan-taint-check-plugin, 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10MW-1.35-notes (1.35.0-wmf.8; 2019-11-26): Add phan-taint-check-plugin to DonationInterface extension - https://phabricator.wikimedia.org/T202384 (10Daimona) Closing as duplicate because taint-check is now included... [10:29:54] 10Continuous-Integration-Config, 10phan-taint-check-plugin, 10Wikimedia-General-or-Unknown, 10Patch-For-Review: Enable phan-taint-check-plugin on all Wikimedia-deployed repositories after getting it to pass - https://phabricator.wikimedia.org/T201219 (10Daimona) 05Open→03Resolved Given that all subtask... [11:31:53] 10Gerrit, 10Code-Health-Metrics, 10Sonarqubebot: Stop SonarQube spam - https://phabricator.wikimedia.org/T256942 (10kostajh) > @Krinkle Thanks for this suggestion. I would propose that we wait a week to see how things look with the narrower subset of rules, if that's OK with everyone. Here's a URL we can wa... [11:35:04] 10Gerrit, 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Patch-For-Review, 10ci-test-error (WMF-deployed Build Failure): Jenkins job failing intermittently due to Gerrit HTTP 502 errors when interacting with repos - https://phabricator.wikimedia.org/T246763 (10hashar) @Lucas_Werkmeister_WMDE... [11:58:25] 10phan-taint-check-plugin, 10MW-1.34-notes (1.34.0-wmf.20; 2019-08-27): Sanitizer::stripAllTags() causing double-escape false positive from phan-taint-check - https://phabricator.wikimedia.org/T230234 (10Daimona) 05Open→03Resolved a:03Daimona >>! In T230234#5414404, @Daimona wrote: > I've submitted a tes... [12:05:20] maintenance-disconnect-full-disks build 191586 integration-agent-docker-1011 (/srv: 98%): OFFLINE due to disk space [12:06:27] PROBLEM - Free space - all mounts on integration-agent-docker-1011 is CRITICAL: CRITICAL: integration.integration-agent-docker-1011.diskspace._srv.byte_percentfree (<33.33%) [12:16:26] RECOVERY - Free space - all mounts on integration-agent-docker-1011 is OK: OK: All targets OK [12:25:19] maintenance-disconnect-full-disks build 191590 integration-agent-docker-1011: OFFLINE due to disk space [12:30:56] 10Continuous-Integration-Config, 10Jenkins: mediawiki-fresnel-patch-docker should set jenkins as reviewer - https://phabricator.wikimedia.org/T256919 (10hashar) 05Open→03Declined The first comment comes from the coverage job which merely write a message. The gerrit review command was: ` gerrit.review.--pro... [12:35:34] 10Beta-Cluster-Infrastructure: Please, give me a Steward, Global sysop and Checkuser rights - https://phabricator.wikimedia.org/T257039 (10Tegel_) [12:46:04] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (2020-04 to 2020-06 (Q4)), 10Zuul, and 2 others: pywikibot get merge rejections due to zuul-merger not being able to update tags - https://phabricator.wikimedia.org/T252310 (10hashar... [12:51:12] 10Phabricator, 10Editing-Team-Request, 10Editing-team (Tracking): Custom task form for #Editing-team-request - https://phabricator.wikimedia.org/T256123 (10Aklapper) Heja, I don't know much about Forms but took a quick look at this. The description above says > Please add your team's tag under tags but it al... [12:51:20] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1))), 10Zuul: "systemctl restart zuul" does not gracefully restart Zuul scheduler - https://phabricator.wikimedia.org/T257040... [12:53:27] That's broke. Is it not supposed to cut off project tags if they're too long. [13:13:18] 10Continuous-Integration-Infrastructure, 10Release-Engineering-Team (CI & Testing services), 10Release-Engineering-Team-TODO (Release-Engineering-Team-TODO (2020-07-01 to 2020-09-30 (Q1))), 10Zuul: "systemctl restart zuul" does not gracefully restart Zuul scheduler - https://phabricator.wikimedia.org/T257040... [13:52:46] (03PS1) 10Dbarratt: Allow Article Path to be customized in LocalSettings.php [releng/dev-images] - 10https://gerrit.wikimedia.org/r/609427 [13:57:08] 10Release-Engineering-Team, 10MediaWiki-Docker, 10dev-images: Docker images do not allow for article paths other than /wiki - https://phabricator.wikimedia.org/T257044 (10dbarratt) [13:57:33] (03PS2) 10Dbarratt: Allow Article Path to be customized in LocalSettings.php [releng/dev-images] - 10https://gerrit.wikimedia.org/r/609427 (https://phabricator.wikimedia.org/T257044) [14:03:57] (03PS1) 10QChris: Allow “Gerrit Managers” to import history [extensions/WSOAuth] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/609428 [14:03:59] (03CR) 10QChris: [V: 03+2 C: 03+2] Allow “Gerrit Managers” to import history [extensions/WSOAuth] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/609428 (owner: 10QChris) [14:04:04] (03PS1) 10QChris: Import done. Revoke import grants [extensions/WSOAuth] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/609429 [14:04:06] (03CR) 10QChris: [V: 03+2 C: 03+2] Import done. Revoke import grants [extensions/WSOAuth] (refs/meta/config) - 10https://gerrit.wikimedia.org/r/609429 (owner: 10QChris) [14:15:34] maintenance-disconnect-full-disks build 191612 integration-agent-docker-1013 (/srv: 97%): OFFLINE due to disk space [14:19:22] 10Gerrit: Make Gerrit test instance at https://gerrit.git.wmflabs.org work again or update docs - https://phabricator.wikimedia.org/T256563 (10QChris) 05Open→03Resolved a:03QChris There is a new working Gerrit in labs https://gerrit.devtools.wmflabs.org/ That got set up by @Dzahn and @Paladox I've update... [14:20:07] PROBLEM - Free space - all mounts on integration-agent-docker-1013 is CRITICAL: CRITICAL: integration.integration-agent-docker-1013.diskspace._srv.byte_percentfree (<44.44%) [14:30:04] RECOVERY - Free space - all mounts on integration-agent-docker-1013 is OK: OK: All targets OK [14:30:27] maintenance-disconnect-full-disks build 191615 integration-agent-docker-1013: OFFLINE due to disk space [14:55:20] maintenance-disconnect-full-disks build 191620 integration-agent-docker-1013: OFFLINE due to disk space [15:30:56] 10Project-Admins, 10Analytics, 10Analytics-Cluster: can we rename analytics-cluster to analytics-clusters (plural)? - https://phabricator.wikimedia.org/T257059 (10Nuria) [15:38:40] 10Gerrit: Make Gerrit test instance at https://gerrit.git.wmflabs.org work again or update docs - https://phabricator.wikimedia.org/T256563 (10Zoranzoki21) >>! In T256563#6277055, @QChris wrote: > There is a new working Gerrit in labs https://gerrit.devtools.wmflabs.org/ > That got set up by @Dzahn and @Paladox... [16:34:13] 10Gerrit: Make Gerrit test instance at https://gerrit.git.wmflabs.org work again or update docs - https://phabricator.wikimedia.org/T256563 (10Dzahn) 05Resolved→03Open a:05QChris→03Dzahn [18:04:33] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Quality-and-Test-Engineering-Team (QTE), 10User-Soham, and 2 others: Create MediaWiki-Docker documentation for all repositories with Selenium tests - https://phabricator.wikimedia.org/T256239 (10Vidhi-Mody) [18:08:20] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Wikibugs: jerkins-bot should not post on IRC for Gerrit changes marked 'WIP' - https://phabricator.wikimedia.org/T239928 (10dancy) a:03dancy [18:38:17] 10Project-Admins, 10Analytics, 10Analytics-Cluster: can we rename analytics-cluster to analytics-clusters (plural)? - https://phabricator.wikimedia.org/T257059 (10Aklapper) @Nuria: For future reference, please don't assign tasks to me as I might be on vacation and as there are other folks who could also perf... [18:38:55] 10Project-Admins, 10Analytics, 10Analytics-Cluster: Rename Analytics-Cluster to Analytics-Clusters (plural) and make it a subproject of Analytics - https://phabricator.wikimedia.org/T257059 (10Aklapper) [19:07:58] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Wikibugs: wikibugs2 should not post on IRC for Gerrit changes marked 'WIP' - https://phabricator.wikimedia.org/T239928 (10dancy) [19:09:06] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Quality-and-Test-Engineering-Team (QTE), 10User-Soham, and 2 others: Create MediaWiki-Docker documentation for all repositories with Selenium tests - https://phabricator.wikimedia.org/T256239 (10Vidhi-Mody) [19:10:49] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Wikibugs: wikibugs2 should not post on IRC for Gerrit changes marked 'WIP' - https://phabricator.wikimedia.org/T239928 (10dancy) Verified that the `change` key of a gerrit event has `"wip": true` for changes marked WIP. For example:... [20:12:23] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Wikibugs, 10Patch-For-Review: wikibugs2 should not post on IRC for Gerrit changes marked 'WIP' - https://phabricator.wikimedia.org/T239928 (10dancy) T175929 has some discussion on this topic. Looks like an attempt was made to achi... [20:14:49] Foo bar T175929 quuz [20:14:50] T175929: Prepare wikibugs gerrit bot for gerrit 2.14 / 2.15 / 3.0 - https://phabricator.wikimedia.org/T175929 [20:15:01] dancy: .. and stashbot makes links with preview titles [20:15:08] but only in public channels [20:25:49] 10Project-Admins, 10Analytics, 10Analytics-Cluster: Rename Analytics-Cluster to Analytics-Clusters (plural) and make it a subproject of Analytics - https://phabricator.wikimedia.org/T257059 (10Nuria) >This will also remove tasks tagged with yesyes [20:29:14] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10dancy) [20:36:22] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10QChris) [20:36:26] 10Gerrit, 10Patch-For-Review: Gerrit 3.2 upgrade - https://phabricator.wikimedia.org/T254158 (10QChris) [20:36:46] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10QChris) [20:40:05] qchris found the bug for the above [20:40:08] simple fix [20:40:11] Me too. [20:40:35] Meh. I let you upload the fix then. [20:43:28] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10dancy) [20:47:07] qchris https://gerrit-review.googlesource.com/c/gerrit/+/274356 [20:47:37] Perfect! [20:47:44] Thanks. [20:47:50] I'll include that in the next build. [20:49:09] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10Krinkle) As a workaround (for end users readng this) you can click on "Gitiles" (top left of diff) instead of "Show blame". Then click `[blame]`. The commit references there do open fine, an... [20:50:14] this is fun [20:50:21] trying to pull some BlueSpice* updates [20:50:22] error: Server does not allow request for unadvertised object c049ec14db7bb894283d5dd6528ea227965a3eac [20:50:22] error: Server does not allow request for unadvertised object 2928096fe58db90eae52da9166e8fa072fa482f2 [20:50:22] error: Server does not allow request for unadvertised object 5004917cc9fa8a2353de99789221aea3b3d930a3 [20:50:22] error: Server does not allow request for unadvertised object 15e2bf4148a85d95c6461a72b662c3cc6e494dd0 [20:50:22] error: Server does not allow request for unadvertised object 7f622c2d8468e658b7fac51ce94932e9a7144bcd [20:50:24] error: Server does not allow request for unadvertised object adf972f93b5ed2fc953a469b208730251e84937e [20:50:28] etc [20:54:32] Reedy: Is this something related to the Gerrit upgrade, or something else? [20:55:17] Reedy: is that on mediawiki/extensions.git ? [20:55:21] hashar: yeah [20:55:27] git submodule sync [20:56:08] the urls are off I guess [20:56:28] or they force pushed stuff here and there and git get lost maybe [20:56:35] 10Gerrit: Clicking on entries in Gerrit's "show blame" view results in 404 - https://phabricator.wikimedia.org/T257077 (10Paladox) Fixed with https://gerrit-review.googlesource.com/c/gerrit/+/274356 [20:57:02] 10Release-Engineering-Team (Onboarding), 10Release-Engineering-Team-TODO, 10Wikibugs, 10Patch-For-Review: wikibugs2 should not post on IRC for Gerrit changes marked 'WIP' - https://phabricator.wikimedia.org/T239928 (10dancy) Test added. Ready for review. [20:57:20] qchris: I can't remember what you said about the CI report urls [20:57:42] I've just been working on that. [20:57:49] AH cool :] [20:57:49] The plugin you mentioned on the bug fails for me. [20:58:00] It just does not load. [20:58:23] 'Scripting error' and deeplinks into loooong javascript lines. [20:58:37] I've got to play some more with it to make it workable. [20:58:45] Paladox mentionned it yeah and it looked like something that matched our need [20:59:06] Yup. Agreed. [21:00:48] qchris: and kudos on fixing the soytemplates for emails/gerritbot :] [21:01:10] Hahaha :-) [21:01:19] Thanks for the Code Reviews :) [21:04:12] I mssed the changeUrls variable had the project name included [21:04:42] meanwhile, I have noticed anything suspicious since the upgrade [21:04:54] beside that PolyGerrit is pleasant to use [21:06:31] Imagine how nice it would be with an upgraded Zuul that speaks the checks API :-P [21:14:02] paladox: Does the PolyGerrit test result plugin at https://github.com/dburm/pg-test-result-plugin actually work for you? [21:16:30] qchris: I don't even know whether the Zuul v3 uses the checks API [21:17:04] I thought I read something about that in the Zuul code. But maybe I mis-read it. [21:17:52] "The Gerrit driver has experimental support for Gerrit's `checks` plugin" [21:18:05] 10Release-Engineering-Team, 10Security-Team: php-composer-security-docker failing due to git getch of non-existent REL1_35 ref - https://phabricator.wikimedia.org/T257080 (10sbassett) [21:18:22] since 3.11 [21:18:36] :-D [21:18:49] which kind of make sense since their project leader (James E.Blair) went to add Zuul to gerrit upstream [21:19:08] And gerrit upstream is loving it :-) [21:19:27] it is so tightly integrated with Gerrit :D [21:21:17] * qchris sighs [21:21:34] na that is a good news for both Gerrit and Zuul [21:21:50] I guess those communities will work together and improve each other open source software [21:22:03] Yes, I also think the Zuul<->Gerrit coupling is a good thing. [21:22:26] I always wondered why upstream gerrit did not use it. [21:22:32] Now it's happening :-) [21:22:44] I think they used Jenkins and the Gerrit trigger plugin [21:22:52] which is written in java so that is an advantage [21:23:49] or some custom google bot maybe [21:24:11] I'm so glad you chose zuul for WMF :-) [21:24:40] qchris it did when i last tried it. [21:24:51] hashar zuul supports checkers [21:25:02] at least it's checking some plugins on gerrit-review [21:25:07] Thanks paladox! [21:25:22] originally I had the Jenkins Gerrit plugin installed [21:25:32] but since I knew nothing about java it was not very helpful to me [21:25:46] then I started writting some basic template system to generate our jenkins job [21:25:57] got pointed at jjb from openstack and adopted it [21:26:22] and zuul followed cause well: A) Python <3 B) the gate system made lot of sense ;) [21:26:46] I think both jjb and zuul make a lot of sense. [21:26:52] That's a great setup. [21:26:59] well jjb is a bit painful [21:27:09] the weird templating system based on yaml is not super convenient [21:27:17] Imagine writing jobs by hand ... :-) [21:27:33] Jenkins has Jenkinsfile and groovy nowadays which is how the WMF Pipeline is implemented ( integration/pipeline.git ) [21:27:52] and I should ask Ryan Lane one day how we ended up with Gerrit [21:28:10] I might have suggested it cause I knew it from the Android project [21:28:18] but don't quote me on that ;] [21:28:22] :-D [21:37:02] 10Release-Engineering-Team, 10Security-Team: php-composer-security-docker failing due to git getch of non-existent REL1_35 ref - https://phabricator.wikimedia.org/T257080 (10Legoktm) {f3d6d21edcb13e07603e6f43ac30fd4ab3bede45} but it's going to fail until the branch is actually created... [21:48:52] paladox: I must be doing something wrong with the test result plugin. How do you run/include it? [21:58:33] Meh, will try some more tomorrow :-) [22:08:43] 10phan-taint-check-plugin, 10Patch-For-Review: Optimize phan-taint-check speed - https://phabricator.wikimedia.org/T203651 (10Daimona) Turns out this is not so necessary. I just discovered https://github.com/phan/phan/issues/4000, and I obviously was running phan without pcntl. phan's runtime on core is around... [22:08:51] 10phan-taint-check-plugin, 10Patch-For-Review: Optimize phan-taint-check speed - https://phabricator.wikimedia.org/T203651 (10Daimona) p:05High→03Medium [22:13:17] I am off :)D [22:22:20] 10Release-Engineering-Team (Development services), 10User-Demian: Evaluate the workflows and workload of managing a Node package repository - https://phabricator.wikimedia.org/T257061 (10Demian) [22:25:35] 10Release-Engineering-Team (Development services), 10Developer Productivity, 10Proposal, 10User-Demian: Draft: RFC: Evaluate alternative Node package managers for improved package security - https://phabricator.wikimedia.org/T257068 (10Demian) [22:31:35] 10Release-Engineering-Team (Development services), 10Developer Productivity, 10Proposal, 10User-Demian: Draft: RFC: Evaluate alternative Node package managers for improved package security - https://phabricator.wikimedia.org/T257068 (10Demian) [22:37:21] 10Release-Engineering-Team (CI & Testing services), 10User-Demian: Determine Node package auditing workflows - https://phabricator.wikimedia.org/T257072 (10Demian) [22:38:10] 10Release-Engineering-Team (CI & Testing services), 10User-Demian: Determine Node package auditing workflows - https://phabricator.wikimedia.org/T257072 (10Demian) [22:39:16] 10Release-Engineering-Team (Development services), 10User-Demian: Evaluate the workflows and workload of managing a Node package repository - https://phabricator.wikimedia.org/T257061 (10Demian) [22:39:43] 10Release-Engineering-Team (CI & Testing services), 10User-Demian: Evaluate the workflows and workload of managing a Node package repository - https://phabricator.wikimedia.org/T257061 (10Demian) [22:43:25] 10Release-Engineering-Team (CI & Testing services), 10User-Demian: Determine Node package auditing workflows - https://phabricator.wikimedia.org/T257072 (10Demian) >>! In T257072#6277800, @Aklapper wrote: > @Demian: Please add project tags - thanks. Done. I'm in the dark with that, so I wasn't in a hurry to d... [23:23:34] qchris you just install it in plugins [23:23:46] qchris Gerrit supports htm plugins