[00:02:39] 10Gerrit, 10Tools, 03Wikimedia-Hackathon-2024: Gerrit reviewer bot should add reviewers as CC instead of actual reviewers - https://phabricator.wikimedia.org/T363290#9743154 (10matmarex) If you really wanted to know for sure, I suppose you'd have to just ask everyone. There are 155 people listed (hmm, more t... [05:59:23] 10Scap: Proposal for refactoring of backport dependency handling - https://phabricator.wikimedia.org/T362987#9743607 (10jeena) >>! In T362987#9740026, @jnuche wrote: >> I'm not clear on what the proposed code changes are. I think what is mentioned in the "relevant dependencies" and "backport vote" sections are a... [07:48:40] 10Phabricator (Upstream), 07Accessibility, 07Upstream: No focus indicator for URI links on Phabricator pages - https://phabricator.wikimedia.org/T363181#9743783 (10Aklapper) Looks like this has been brought up in upstream https://we.phorge.it/Q132 too [08:47:57] 10Phabricator: Mark Maniphestbot Phabricator account as bot in database - https://phabricator.wikimedia.org/T331529#9743879 (10Aklapper) 05Open→03Resolved a:03Aklapper Done now by tweaking the DB (see command above). [09:01:55] 10Phabricator (Upstream), 07Upstream: Closed task with "Unknown status" - https://phabricator.wikimedia.org/T125803#9743913 (10Aklapper) p:05Triage→03Low [09:08:04] 10Phabricator, 10VPS-project-Codesearch: Consider adding a way to query Codesearch from Phabricator - https://phabricator.wikimedia.org/T183608#9743932 (10Aklapper) 05Stalled→03Declined > maybe we should include results from there into Phabricator's quick search menu? I'd prefer not to maintain a down... [09:09:33] 10Phabricator: When an account is disabled, remove them as watchers - https://phabricator.wikimedia.org/T256094#9743937 (10Aklapper) 05Open→03Declined Declining per last comment - random example: If I disabled an account temporarily (e.g. machine with SSH key stolen or such) and re-enabled a few days lat... [09:11:32] 10Phabricator, 06Project-Admins: Kartographer project should be a subproject instead of a milestone - https://phabricator.wikimedia.org/T161691#9743952 (10Aklapper) [09:13:36] 10Phabricator, 13Patch-For-Review: Unhandled Exception ("PhutilAggregateException") when using Phabricator in languages that translated "AM" - https://phabricator.wikimedia.org/T363215#9743960 (10Aklapper) p:05Low→03Medium [10:36:24] Project beta-scap-sync-world build #152116: 04FAILURE in 1 min 4 sec: https://integration.wikimedia.org/ci/job/beta-scap-sync-world/152116/ [10:46:35] Yippee, build fixed! [10:46:35] Project beta-scap-sync-world build #152117: 09FIXED in 1 min 14 sec: https://integration.wikimedia.org/ci/job/beta-scap-sync-world/152117/ [10:58:36] 10Phabricator: Visually indicate when a Phabricator user is new (Welcome culture) - https://phabricator.wikimedia.org/T64324#9744328 (10Aklapper) To make my original idea scale, caching would definitely by required, setting up an `AphlictDropdownDataQuery` etc. Phabricator Badges may make more sense here, howe... [11:57:08] 06Release-Engineering-Team, 06Quality-and-Test-Engineering-Team, 10Temporary accounts, 06Trust and Safety Product Team: Temp accounts deployment and the release train - https://phabricator.wikimedia.org/T355882#9744454 (10Tchanders) Sounds good to me. Thanks @kostajh [12:19:33] (03PS1) 10Subramanya Sastry: Remove code to strip p-tags around indicators: No longer necessary [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/1024381 [12:22:52] (03CR) 10Subramanya Sastry: "I've already applied this on parsing-qa-02 and been running a new visual diff run." [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/1024381 (owner: 10Subramanya Sastry) [12:24:43] 06Release-Engineering-Team, 10Temporary accounts, 06Trust and Safety Product Team, 13Patch-For-Review: Make PHPUnit, Selenium, and API testing tests pass with temp account feature flag enabled - https://phabricator.wikimedia.org/T355879#9744495 (10kostajh) [12:26:12] 06Release-Engineering-Team, 10Temporary accounts, 06Trust and Safety Product Team, 13Patch-For-Review: Make PHPUnit, Selenium, and API testing tests pass with temp account feature flag enabled - https://phabricator.wikimedia.org/T355879#9744496 (10kostajh) [12:26:28] 06Release-Engineering-Team, 10Temporary accounts, 06Trust and Safety Product Team, 13Patch-For-Review: Make PHPUnit, Selenium, and API testing tests pass with temp account feature flag enabled - https://phabricator.wikimedia.org/T355879#9744497 (10kostajh) [12:27:50] (03CR) 10Isabelle Hurbain-Palatin: [C:03+2] Remove code to strip p-tags around indicators: No longer necessary [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/1024381 (owner: 10Subramanya Sastry) [12:28:37] (03Merged) 10jenkins-bot: Remove code to strip p-tags around indicators: No longer necessary [integration/visualdiff] - 10https://gerrit.wikimedia.org/r/1024381 (owner: 10Subramanya Sastry) [13:00:18] 10Continuous-Integration-Config, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata, 10wmde-wikidata-tech, and 4 others: Make PHP 8.2 voting on development branch of Wikibase - https://phabricator.wikimedia.org/T360560#9744649 (10Lucas_Werkmeister_WMDE) Per T324202#9744647, I think we can go ahead with... [13:01:06] (03CR) 10Lucas Werkmeister (WMDE): [C:03+1] Zuul: Re-apply PHP 8.2 CI to Wikibase-based code [integration/config] - 10https://gerrit.wikimedia.org/r/1023401 (https://phabricator.wikimedia.org/T360560) (owner: 10Jforrester) [13:18:13] podman run --rm -it -v .:/src docker-registry.wikimedia.org/releng/gerrit:1.3.0 [13:18:33] sounds like I could wait for quite a while [13:31:56] This is something I am requesting to be oversighted: https://phabricator.wikimedia.org/F32412523 [13:43:33] (03PS3) 10Hashar: dockerfiles: update bazelisk/gerrit to Bullseye, drop java 8 [integration/config] - 10https://gerrit.wikimedia.org/r/1023881 (https://phabricator.wikimedia.org/T363339) [13:44:08] (03CR) 10Hashar: [C:03+2] "I have added `python-is-python3` since I recall Bazel rules looking up for `python`" [integration/config] - 10https://gerrit.wikimedia.org/r/1023881 (https://phabricator.wikimedia.org/T363339) (owner: 10Hashar) [13:45:43] (03Merged) 10jenkins-bot: dockerfiles: update bazelisk/gerrit to Bullseye, drop java 8 [integration/config] - 10https://gerrit.wikimedia.org/r/1023881 (https://phabricator.wikimedia.org/T363339) (owner: 10Hashar) [14:51:30] (03PS8) 10Hashar: wmf-build: always use upstream for git submodules [software/gerrit] (wmf/stable-3.8) - 10https://gerrit.wikimedia.org/r/1019678 [14:51:36] (03CR) 10Hashar: [C:03+2] wmf-build: always use upstream for git submodules [software/gerrit] (wmf/stable-3.8) - 10https://gerrit.wikimedia.org/r/1019678 (owner: 10Hashar) [14:53:42] (03PS1) 10Hashar: dockerfiles: gerrit: fix example script [integration/config] - 10https://gerrit.wikimedia.org/r/1024415 [14:53:42] (03PS1) 10Hashar: jjb: switch Gerrit jobs to Bullseye [integration/config] - 10https://gerrit.wikimedia.org/r/1024416 (https://phabricator.wikimedia.org/T363339) [14:53:56] I guess [14:53:58] I really [14:53:59] hate [14:54:01] upgrading OSes [14:54:08] (03CR) 10Hashar: [C:03+2] jjb: switch Gerrit jobs to Bullseye [integration/config] - 10https://gerrit.wikimedia.org/r/1024416 (https://phabricator.wikimedia.org/T363339) (owner: 10Hashar) [14:54:13] (03CR) 10Hashar: [C:03+2] dockerfiles: gerrit: fix example script [integration/config] - 10https://gerrit.wikimedia.org/r/1024415 (owner: 10Hashar) [14:55:54] (03Merged) 10jenkins-bot: dockerfiles: gerrit: fix example script [integration/config] - 10https://gerrit.wikimedia.org/r/1024415 (owner: 10Hashar) [14:55:55] (03Merged) 10jenkins-bot: jjb: switch Gerrit jobs to Bullseye [integration/config] - 10https://gerrit.wikimedia.org/r/1024416 (https://phabricator.wikimedia.org/T363339) (owner: 10Hashar) [14:58:59] (03Merged) 10jenkins-bot: wmf-build: always use upstream for git submodules [software/gerrit] (wmf/stable-3.8) - 10https://gerrit.wikimedia.org/r/1019678 (owner: 10Hashar) [15:26:44] 10Continuous-Integration-Infrastructure, 10Gerrit, 07Jenkins, 03Wikimedia-Hackathon-2024: jenkins-bot should clear the attention flag for the patch uploader after a new patchset passes tests, if it had set the flag after a previous patchset failed tests - https://phabricator.wikimedia.org/T363289#9745139 (1... [15:35:30] hey thcipriani my apologies for not making today's train log, we had a little emergency here (two stranded tourists with the address of my apartment building for a hotel, and a phone number not answering)... hoping the booking agency can get it sorted for them now [16:04:08] apergos: now worries, good luck to stranded tourists! [16:04:49] they haven't knocked on my door again, I'm hoping that's a good sign. (also I left them at a local place for gyros as they requested, so they at least could eat something good while waiting) [16:06:50] gyros surely ease uncertainty for me [16:07:08] hovering about even in terms of open tasks on the production error workboard since Feb, I guess that means we're keeping pace as more are filed. [16:08:34] that's pretty decent really [16:13:42] hope they didnt get scammed like with fake airbnbs [16:16:17] they went through booking.com the place was advertised as an actual hotel. (I checked the ad later). no way to know it was a single apartment in some random apartment building. already bordering on fraud imo (IANAL) [16:17:07] fortunately the support folks at booking.com stepped up and were going to arrange alternate accommodations if the "hotel" contact could not be reached [16:19:14] *nod* I have seen similar on booking in LA where it's like "0 reviews, just got added" and you are wondering if it could be fake listing [16:36:40] How Do I request oversight on Phabricator [16:41:49] ......and what does that mean? [16:42:13] andre: running "destroy" commands on the shell?::p) [16:43:31] uh swanky :D [16:57:24] Someone is requesting oversight / removal of personal information from phab. Is there an email address they can be directed to to request that? [17:06:07] andre: ^ I am thinking just send them to https://phabricator.wikimedia.org/maniphest/task/edit/form/23/ and let them know that ticket won't be public? [17:09:09] 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 13Patch-For-Review: Migrate all CI jobs from buster to bullseye or later and drop buster testing support - https://phabricator.wikimedia.org/T335765#9745661 (10hashar) [17:10:15] 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 13Patch-For-Review: Migrate all CI jobs from buster to bullseye or later and drop buster testing support - https://phabricator.wikimedia.org/T335765#9745666 (10hashar) [17:10:42] 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: CI image with openjdk-11-jre-headless fails to build - https://phabricator.wikimedia.org/T363339#9745681 (10hashar) Leaving it open since the `java11` and `maven` images certainly suffer from the same issue and should al... [17:36:40] done [17:36:52] thanks andre [17:37:10] yw [18:52:48] 06Release-Engineering-Team, 10Scap: scap: hide helmfile operations behind a progress bar - https://phabricator.wikimedia.org/T325530#9746031 (10CodeReviewBot) dancy updated https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/292 Simple k8s deployment progress reporting [19:05:47] 10Phabricator (phabricator-next): Unhandled Exception ("PhutilAggregateException") when using Phabricator in languages that translated "AM" - https://phabricator.wikimedia.org/T363215#9746087 (10Pppery) [19:14:31] 10Phabricator (2024-04-23), 06translatewiki.net, 07I18n: Adding sicilian language (scn) - https://phabricator.wikimedia.org/T299694#9746112 (10valerio.bozzolan) I'm from Milano but I switched to Sicilian since that's awesome. Thanks Pppery, Brennen, and all! For newcomers, you can translate from here as... [19:15:22] 10Phabricator: Duplicate entry exception adding/removing task token - https://phabricator.wikimedia.org/T363518 (10Aklapper) 03NEW p:05Triage→03Low [19:16:23] 10Phabricator: Duplicate entry exception adding/removing task token - https://phabricator.wikimedia.org/T363518#9746131 (10Aklapper) @Pppery: Do you remember by any chance if anything was "different" when you awarded/removed token(s)? (If not then please feel free to unsubscribe again!) [19:19:58] 06Release-Engineering-Team, 10Scap: Run weekly "scap clean" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519 (10dancy) 03NEW [19:22:27] 06Release-Engineering-Team, 10Scap: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746146 (10dancy) p:05Triage→03Medium [19:32:39] 10Gerrit: Create a plugin that add's recheck as a button to PolyGerrit's ui - https://phabricator.wikimedia.org/T214631#9746166 (10thcipriani) Guessing this no longer works since gerrit moved to lit, is that right @Paladox ? (also, seems we've undeployed the wikimedia gerrit plugin judging from our [[https:... [19:32:56] 10Phabricator: Duplicate entry exception adding/removing task token - https://phabricator.wikimedia.org/T363518#9746168 (10Pppery) I think I clicked on the wrong token, and then while the page was still processing that request clicked on the token I meant to award - the first (dislike) token went through, the se... [19:33:45] 10Phabricator: Duplicate entry exception adding/removing task token - https://phabricator.wikimedia.org/T363518#9746174 (10Pppery) I tried to do the same thing on this task itself - did it generate another instance of the same error? [19:35:37] 10MediaWiki-Releasing: Big holes in the MediaWiki release archive - https://phabricator.wikimedia.org/T190369#9746178 (10Krinkle) Likely unrelated, but I'll leave this here: [19:35:59] 06Release-Engineering-Team, 10Scap, 13Patch-For-Review: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746179 (10CodeReviewBot) dancy opened https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/294 stage-train: Remove clean stage [19:44:47] 06Release-Engineering-Team, 10Scap, 13Patch-For-Review: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746206 (10CodeReviewBot) thcipriani opened https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/72 Calendar: add event for weekly... [19:44:55] 10Phabricator (Upstream), 07Upstream: Duplicate entry exception adding/removing task token - https://phabricator.wikimedia.org/T363518#9746204 (10Aklapper) >>! In T363518#9746170, @Pppery wrote: > I tried to do the same thing on this task itself - did it generate another instance of the same error? Heh, yes,... [20:12:24] 06Release-Engineering-Team, 10Scap, 13Patch-For-Review: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746307 (10CodeReviewBot) dancy merged https://gitlab.wikimedia.org/repos/releng/release/-/merge_requests/72 Calendar: add event for weekly `sca... [20:13:28] 06Release-Engineering-Team, 10Scap, 13Patch-For-Review: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746310 (10dancy) [20:13:41] 06Release-Engineering-Team, 10Scap, 13Patch-For-Review: Run weekly "scap clean auto" separately from "scap stage-train" - https://phabricator.wikimedia.org/T363519#9746313 (10dancy) [20:32:56] 06Release-Engineering-Team, 10Scap, 06serviceops, 13Patch-For-Review: scap should optionally display helmfile diffs for review - https://phabricator.wikimedia.org/T362717#9746332 (10CodeReviewBot) swfrench opened https://gitlab.wikimedia.org/repos/releng/scap/-/merge_requests/295 Optionally collect, displ... [21:05:31] 10Phabricator (2024-04-23), 06translatewiki.net, 07I18n: Adding sicilian language (scn) - https://phabricator.wikimedia.org/T299694#9746426 (10Pppery) Wow - never thought this would get anywhere near as much attention as it did. [21:13:24] 10Phabricator (2024-04-23), 06translatewiki.net, 07I18n: Adding sicilian language (scn) - https://phabricator.wikimedia.org/T299694#9746438 (10Pppery) Please do see my notes at T318763 on how the deployment process works, though, so it will probably be a while before the next deploy. [22:14:52] 10Release-Engineering-Team (Now this 🫠), 10Scap, 10MediaWiki-libs-Mime, 10MediaWiki-Platform-Team (Radar), 13Patch-For-Review: Get rid of the /srv/mediawiki/php symbolic link - https://phabricator.wikimedia.org/T359643#9746582 (10CodeReviewBot) dancy opened https://gitlab.wikimedia.org/repos/releng/train... [22:29:36] 10Release-Engineering-Team (Priority Backlog 📥), 13Patch-For-Review, 05Release, 05Train Deployments: 1.43.0-wmf.2 deployment blockers - https://phabricator.wikimedia.org/T361396#9746602 (10brennen) 05Open→03Resolved Stable on all wikis since deploy. Optimistically resolving. [23:22:47] 10Release-Engineering-Team (Now this 🫠), 10Scap, 10MediaWiki-libs-Mime, 10MediaWiki-Platform-Team (Radar), 13Patch-For-Review: Get rid of the /srv/mediawiki/php symbolic link - https://phabricator.wikimedia.org/T359643#9746780 (10CodeReviewBot) jhuneidi merged https://gitlab.wikimedia.org/repos/releng/tr...