[00:17:25] 10fundraising-tech-ops, 10SRE: As an FR-Tech SRE, we want to be able to designate a host for decommissioning - https://phabricator.wikimedia.org/T334154 (10ssingh) [02:22:05] (03PS3) 10Ejegg: Add new custom field for initial scheme txn ID [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) [02:22:07] (03PS4) 10Ejegg: Use API4 for inserting ContributionRecur records [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905752 (https://phabricator.wikimedia.org/T331754) [02:22:09] (03PS2) 10Ejegg: Add initial scheme id when present in message [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905768 (https://phabricator.wikimedia.org/T331754) [02:22:11] (03PS2) 10Ejegg: Use API4 to get payments to charge [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905772 (https://phabricator.wikimedia.org/T331754) [02:22:13] (03PS4) 10Ejegg: Send initial scheme transaction id with recur charge [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905773 (https://phabricator.wikimedia.org/T331754) [02:31:38] (03PS2) 10Ejegg: Filter form submethod list by recurring support [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) [03:32:45] 10Fundraising Sprint Fish HEAD^, 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10fr-donorservices: Civi not keeping the mailing address attached to the active recurring donation after merging - https://phabricator.wikimedia.org/T330231 (10Eileenmcnaughton) a:03Eileenmcnaughton In staging e... [06:29:20] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/906256 (owner: 10L10n-bot) [07:01:20] PROBLEM - check_puppetrun on frbast1001 is CRITICAL: CRITICAL: Puppet has 1 failures. Last run 7 minutes ago with 1 failures. Failed resources (up to 3 shown): Package[initramfs-tools] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frbast1001&service=check_puppetrun [07:06:20] RECOVERY - check_puppetrun on frbast1001 is OK: OK: Puppet is currently enabled, last run 4 minutes ago with 0 failures https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frbast1001&service=check_puppetrun [07:06:24] PROBLEM - check_puppetrun on pay-lvs1004 is CRITICAL: CRITICAL: Puppet has 1 failures. Last run 5 minutes ago with 1 failures. Failed resources (up to 3 shown): Package[initramfs-tools] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=pay-lvs1004&service=check_puppetrun [07:11:24] PROBLEM - check_puppetrun on pay-lvs1004 is CRITICAL: CRITICAL: Puppet has 1 failures. Last run 10 minutes ago with 1 failures. Failed resources (up to 3 shown): Package[initramfs-tools] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=pay-lvs1004&service=check_puppetrun [07:16:24] RECOVERY - check_puppetrun on pay-lvs1004 is OK: OK: Puppet is currently enabled, last run 1 minute ago with 0 failures https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=pay-lvs1004&service=check_puppetrun [07:26:31] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906484 (owner: 10L10n-bot) [07:33:09] (03CR) 10Raimond Spekking: [C: 03+2] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906484 (owner: 10L10n-bot) [08:38:38] (03CR) 10Nikerabbit: [V: 03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/906256 (owner: 10L10n-bot) [13:24:50] (03PS3) 10Damilare Adedoyin: Filter form submethod list by recurring support [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [13:28:05] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Ensure fiscal number for India (PAN) is not stored - https://phabricator.wikimedia.org/T333322 (10Damilare) [13:28:07] 10Fundraising-Backlog, 10FR-dlocal: Update the PAN field label on the dLocal India recurring form. - https://phabricator.wikimedia.org/T334103 (10Damilare) [14:01:18] (03CR) 10Damilare Adedoyin: [C: 03+1] "Thanks for this ejegg, I left a small comment about the removal of the paytm logo from the list." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [15:05:59] (03PS1) 10Wfan: Enable monthly convert from DI for dlocal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) [15:22:39] (03CR) 10Damilare Adedoyin: [C: 03+2] "looks good to me, thanks for the tests also." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901316 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [15:23:13] (03Merged) 10jenkins-bot: Handle initial scheme transaction ID [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901316 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [15:25:20] thanks for the review damilare ! [15:25:45] np [15:29:37] (03CR) 10Ejegg: Filter form submethod list by recurring support (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [15:31:47] hmm, I guess if the donor selects paytm on a donatewiki or banner form, we still want to show that same logo on the paymentswiki form [15:32:23] so let's see, where would be the best place to mutate that? [15:33:14] Or do we want to update the SmashPig code to treat paytmwallet recurring as upi recurring? [15:33:27] I guess both are technically IR [15:33:43] so let me start making those changes in SmashPig [15:34:44] It might be useful to have the method the donor selected listed in CiviCRM as their payment method, even if they are sent the same way to the payment processor [15:37:28] (03CR) 10Damilare Adedoyin: "Hi wfan, thanks for starting the work on this. Dlocal MC process might have a unique flow given what is available on Dlocal. Adyen made it" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan) [15:42:57] sorry I missed this, yh thanks [15:51:26] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal, 10MW-1.41-notes (1.41.0-wmf.4; 2023-04-10): Add optional phone number field for DLocal one-time UPI - https://phabricator.wikimedia.org/T333857 (10EMartin) @AKanji-WMF @jgleeson @Ejegg 6April. We had a call with Dlocal again tod... [16:27:56] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal, 10MW-1.41-notes (1.41.0-wmf.4; 2023-04-10): Add optional phone number field for DLocal one-time UPI - https://phabricator.wikimedia.org/T333857 (10Ejegg) @EMartin We already have logic to exclude it from the recurring UPI form, and I... [16:48:09] 10Fundraising-Backlog, 10fundraising-tech-ops: Ingenico Audit - They are moving their SFTP server to MFT - https://phabricator.wikimedia.org/T334239 (10Cstone) [17:17:55] wfan: can you explain more about the DLocal MC flow reusing the card token? [17:18:23] would we save the smartfield token to Civi, or just use it to make a second API call to get a card_id once the donor clicks the 'yes' button in the MC modal? [17:18:43] 10Fundraising-Backlog, 10fundraising-tech-ops: Ingenico Audit - They are moving their SFTP server to MFT - https://phabricator.wikimedia.org/T334239 (10Dwisehaupt) So reading this closely and looking at the attachments from the email, I think this is what we may want to do: * create an 'webuser' account specif... [17:19:03] When we smartfield token saved, it will return us a card_id for future reuse. No need to store the token itself [17:19:52] 10fundraising-tech-ops, 10SRE, 10SRE-Access-Requests: As an FR-Tech SRE, we want to be able to designate a host for decommissioning - https://phabricator.wikimedia.org/T334154 (10ssingh) [17:20:16] ok, so we make a second API call to save the card, using the same token [17:20:49] sounds good, though I guess it needs a bit more new code [17:21:07] But I am not fully understand what is the thing Seb said that we can make temp token to perm token. I emailed him and he is out of office till 4/9 ask tam@dlocal.com now [17:22:20] so I think we need to implement a new API method: https://docs.dlocal.com/reference/create-a-card [17:22:45] Yeah I was using two separate token one for create card and the other for init payment then when recharge it needs to verify if the card_id still valid,(it could be delete) then make the recurring charge. [17:23:45] I think the recurring logic will be the same once it gets to Civi, no matter how we obtain the card_id [17:23:50] Then realized that we do not actually need this part since when we purchase with token and add save: true, the unit payment will generate a card_id too! And the card_id is valid for recurring [17:24:19] 10fundraising-tech-ops, 10SRE, 10SRE-Access-Requests: As an FR-Tech SRE, we want to be able to designate a host for decommissioning - https://phabricator.wikimedia.org/T334154 (10ssingh) (Person on clinic duty here): I initially removed the `SRE-Access-Requests` tag, my apologies, because I thought that this... [17:24:26] The recurring one will be fairly the same [17:24:33] right, that's how we do it for other processors, but I think it's actually better to use DLocal's separate API calls [17:24:35] Recurring From civi queue [17:24:53] that way, for donors who click 'no', we can avoid keeping their card on file at all [17:25:30] and donor services will likely get fewer complaints about keeping it when we don't need it [17:25:58] While the problems is once token been used can not create a card to generate card_id anymore. That’s why my old code is to generate two tokenid, one for create card id with separate api the other for init payment [17:26:17] ohhh, interesting [17:27:35] so we are able to use the smartfields to generate two tokens without the donor filling them in again? [17:27:51] I wonder if we could do the same thing for Adyen [17:28:01] But I think that’s might not necessary since two token is not necessary if we deny monthly recurring. So I needs to confirm with dlocal again. They can save card with token or card info. So I also thought about use the card info for the new card_id. But still probably token with save is the easiest one~ [17:28:24] Yes, we can just call the create token twice and store it to donordata [17:29:05] hmm, ok, let me look at that code [17:30:16] wfan did you send your two-smartfield-token code to gerrit? [17:30:35] I just see the standard one [17:31:56] Nope I deleted it but the idea is from dlocal.js create two token I called the first one as token_for_card_id.Then from paymentprovider if token_for_card_id exist call create card and save the card_id. [17:33:00] While then when I realized the save part from card param can do the same. I delete those part of code😂 [17:33:19] oh ok [17:34:08] 10fundraising-tech-ops, 10SRE, 10SRE-Access-Requests: As an FR-Tech SRE, we want to be able to designate a host for decommissioning - https://phabricator.wikimedia.org/T334154 (10Dwisehaupt) @ssingh Thanks for looking at this. This task was created to help capture the output of some ongoing discussions to fi... [17:35:46] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal, 10MW-1.41-notes (1.41.0-wmf.4; 2023-04-10): Add optional phone number field for DLocal one-time UPI - https://phabricator.wikimedia.org/T333857 (10EMartin) @Ejegg I have gone back to Dlocal about this. After further input, Rakhi co... [17:39:47] 10fundraising-tech-ops, 10SRE, 10SRE-Access-Requests: As an FR-Tech SRE, we want to be able to designate a host for decommissioning - https://phabricator.wikimedia.org/T334154 (10Dzahn) my 2 cents: Yes, it's possible to resolve this with a new admin group that gets sudo privs to run a particular set of cook... [17:49:15] (03PS1) 10Wfan: breadcrumb style update for customized-civicrm.css to show civi breadcrumb [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906639 [18:38:29] (03PS1) 10Ejegg: Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) [18:38:53] (03CR) 10CI reject: [V: 04-1] Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [18:39:49] damilare: ^^^ is supposed to let us send the paytmwallet submethod to SmashPig and map it to the same 'IR' api call as upi [18:39:59] but I need to smoke test and look at that test failure [18:41:47] oh i see [18:41:48] got it [18:50:08] thanks ejegg [18:56:38] heh, now I've come up against the other UI/UD edge case [18:56:44] just trying to make it nice and neat [18:57:40] I had started changing the ReferenceData class to use that isIndiaRecurring convenience function [19:01:13] (03PS2) 10Ejegg: Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) [19:01:39] (03CR) 10CI reject: [V: 04-1] Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [19:03:18] (03PS3) 10Ejegg: Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) [19:03:48] ok, that should pass tests now. Will smoketest and confirm it sends the right codes [19:10:23] Yep, seems to be sending the expected codes [19:11:20] though I don't seem to be able to make any successful DLocal sandbox API calls from this IP address on the library's public wifi [19:11:29] ok damilare that's ready to review [19:14:05] I'll update the other patch to show paytm on the recurring page too [19:14:28] thanks ejegg ! [19:18:09] (03PS4) 10Ejegg: Filter form submethod list by recurring support [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) [19:18:51] ok, there's also a new patch in config-private that adds the new inr_subscription_* config values for the updated smashpig patch [19:42:35] dang it, even on the VPN I can't seem to make Ingenico sandbox donations [19:43:02] they didn't change the sandbox API url, did they? [19:43:17] ah, or maybe I just need to restart the docker box [19:44:10] yep, that did it [19:46:08] ah yeah docker networking weirdness is a recurrent gotcha [19:46:50] * AndyRussG|ish runs away [19:47:47] hibye [19:50:05] hmph, I got the 3ds results on that hosted checkout status lookup, but no initialSchemeTransactionId [19:50:23] ah, I probably need to tokenize [19:57:37] nope, still no new txn ID [19:57:44] guess I'll send an email [20:57:43] (03CR) 10Damilare Adedoyin: [C: 03+2] "LGTM, tested and works great. Thanks!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [20:58:16] (03Merged) 10jenkins-bot: Generalize India Recurring code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/906645 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [21:01:16] (03CR) 10Damilare Adedoyin: [C: 03+2] "Thanks for this, combining this with the Smashpig patch works great too." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [21:03:06] (03Merged) 10jenkins-bot: Filter form submethod list by recurring support [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906049 (https://phabricator.wikimedia.org/T334097) (owner: 10Ejegg) [21:07:47] 10Fundraising-Backlog, 10fr-donorservices: How can we enable upselling of existing recurring donation amount? - https://phabricator.wikimedia.org/T325317 (10Ejegg) [21:08:06] thanks damilare|away ! [21:08:18] I'll deploy that stuff [21:16:51] (03PS1) 10Ejegg: Update SmashPig dependency [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906659 [21:16:54] (03CR) 10Ejegg: [C: 03+2] Update SmashPig dependency [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906659 (owner: 10Ejegg) [21:25:03] (03PS12) 10Wfan: Add validate and collect part for direct bt [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/903690 (https://phabricator.wikimedia.org/T324294) [21:28:08] (03PS13) 10Wfan: Add validate and collect part for direct bt [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/903690 (https://phabricator.wikimedia.org/T324294) [21:28:10] (03PS3) 10Wfan: Add unit test and add ud to getPaymentMethodId [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/904870 [21:30:16] (03PS4) 10Wfan: Add unit test and add ud to getPaymentMethodId [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/904870 [21:32:00] (03CR) 10Ejegg: "I think we don't need this now that you took it out of messageFields on the DI side." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904658 (https://phabricator.wikimedia.org/T324294) (owner: 10Wfan) [21:32:55] yeah right, can abandon now~ [21:33:05] (03CR) 10Ejegg: "Hmm, it looks like this file is compiled and would be overwritten the next time we pull down an update to the shoreditch extension. Is th" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906639 (owner: 10Wfan) [21:33:15] (03Abandoned) 10Wfan: Remove upi_id from civi log for security reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904658 (https://phabricator.wikimedia.org/T324294) (owner: 10Wfan) [21:33:31] (03Merged) 10jenkins-bot: Update SmashPig dependency [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906659 (owner: 10Ejegg) [21:34:12] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/906664 [21:34:14] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/906664 (owner: 10Ejegg) [21:35:01] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/906664 (owner: 10Ejegg) [21:37:13] yep it will be overwrite, but next time when we pull down an update, the update could have the fix itself, since this breadcrumb is off for a while someone from civi must already noticed~ [21:56:39] (03PS1) 10Ejegg: Update composer.lock for new SmashPig version [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/906670 [21:56:41] (03CR) 10Ejegg: [C: 03+2] Update composer.lock for new SmashPig version [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/906670 (owner: 10Ejegg) [21:58:16] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/906671 [21:58:18] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/906671 (owner: 10Ejegg) [21:58:48] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/906671 (owner: 10Ejegg) [22:04:57] !log payments-wiki upgraded from 75b068a1 to 0f15a101 [22:05:00] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [22:05:14] (03Merged) 10jenkins-bot: Update composer.lock for new SmashPig version [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/906670 (owner: 10Ejegg) [22:05:42] !log SmashPig upgraded from 7c19151f to 24d700f4 [22:05:44] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [22:18:49] (03PS1) 10Ejegg: WIP send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T333725) [22:20:17] (03CR) 10CI reject: [V: 04-1] WIP send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T333725) (owner: 10Ejegg) [22:28:43] (03PS2) 10Ejegg: WIP send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T333725) [22:45:04] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10MW-1.40-notes (1.40.0-wmf.27; 2023-03-13), 10User-notice: CentralNotice banners being shown too many times - https://phabricator.wikimedia.org/T331671 (10Quiddity) Hallo. For Tech News, please could you help d... [23:14:44] (03CR) 10Ejegg: "Just a first impression - looking at these factory methods I see a lot of repeated boilerplate, and I wonder if we're being too strict abo" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/905258 (https://phabricator.wikimedia.org/T333725) (owner: 10Damilare Adedoyin) [23:17:07] (03PS3) 10Ejegg: WIP send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) [23:32:19] (03CR) 10Ejegg: "I think we'll need to add more currencies to DonationInterfaceMonthlyConvertAmounts - none of the DLocal country currencies seem to be sup" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan) [23:35:05] oh man, I get a 'please correct the errors in your fiscal number' error on the dlocal ZA card form - only there's no fiscal number field! [23:35:30] wfan: what currency were you testing that MC patch with? [23:35:41] oh shoot, I need to head home [23:38:52] ah, it was in the link [23:40:38] (03PS1) 10Ejegg: Remove fiscal_number from ZA dlocal links [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/906677 [23:56:26] (03CR) 10Ejegg: "I'm getting a 'No tokenized donation in session' error in doRecurringConversion" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan)