[03:37:34] (03PS1) 10Eileen: Buy more repair time [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300948 [03:38:18] (03CR) 10Cstone: [V:03+2 C:03+2] Buy more repair time [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300948 (owner: 10Eileen) [03:39:59] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1300949 [03:40:17] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1300949 (owner: 10Eileen) [03:42:40] !log civicrm upgraded from 0b8db587 to 69a60dcb [03:42:41] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [05:34:27] (03PS1) 10Eileen: What was I thinking? [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300953 [05:53:28] FIRING: OOM: OOM killer active on fransw1002.frack.eqiad.wmnet:9100 - https://alerts.wikimedia.org/?q=alertname%3DOOM [05:58:28] RESOLVED: OOM: OOM killer active on fransw1002.frack.eqiad.wmnet:9100 - https://alerts.wikimedia.org/?q=alertname%3DOOM [06:17:06] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_46) - 10https://gerrit.wikimedia.org/r/1301017 (owner: 10L10n-bot) [12:30:41] (03CR) 10Jgleeson: [C:03+2] "LGTM. Usually, The first line of the commit is something descriptive, and the self-flagellation can be in the later lines :)" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300953 (owner: 10Eileen) [12:33:11] (03CR) 10Jgleeson: [C:03+2] "LGTM" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300907 (https://phabricator.wikimedia.org/T428831) (owner: 10Lars SG) [12:39:38] (03CR) 10Jgleeson: [C:03+1] "Thanks for highlighting this! This looks ok, but we should align the version bump with our other extensions, like DonationInterface. I'll " [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/1294398 (https://phabricator.wikimedia.org/T425578) (owner: 10Umherirrender) [12:44:22] looks like the trustly fix for the failing recurring would require donor reapproval [12:44:35] based on the last response from Gravy [12:48:22] 06Wikimedia-Fundraising: Autocomplete entry for placeholder attribution - https://phabricator.wikimedia.org/T428943#12014187 (10ppenloglou) 05Open→03Resolved Hey @RLopez-WMF, I was able to add this to our FR.E.D Extension. Grab the 0.0.3.vsix file from this [[ https://gitlab.wikimedia.org/ppenloglou/fr-e... [12:48:37] I'm not familiar with that damilare. could you link the thread? [12:48:54] https://wikimedia.slack.com/archives/C070F1DVBRN/p1781034620114699 [12:48:56] (03Merged) 10jenkins-bot: What was I thinking? [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300953 (owner: 10Eileen) [12:49:07] np was just thinking out loud jgleeson [12:49:46] np damilare [12:49:52] I wonder if they tested out the suggestion also [12:50:03] before sharing [12:50:35] speaking of thinking out loud - we had new recurring subscriptions set up against Stripe last night - I wasn't aware we were doing new recurring via Stripe [12:51:21] it must be the split traffic rule [12:51:39] I thought we turned that off [12:52:04] nah looks like its still on for US traffic [12:52:27] oh interesting [12:52:30] 5% to strinpe [12:52:34] Stripe* [12:54:12] (03Merged) 10jenkins-bot: Switch accesskeys css to a class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300907 (https://phabricator.wikimedia.org/T428831) (owner: 10Lars SG) [12:54:54] the Gr4vy US is confusing me [12:54:58] UI* [12:55:05] I see split routing set here https://wikimedia.gr4vy.app/merchants/default/flow/card-transaction/route-transaction/edbad883-85b9-4235-b70f-4ca91100c8a5 [12:55:24] but then beneath that I see basic routing selected [12:56:08] that's new actually [12:56:23] I guess it implies that the first one gets the 5% [12:56:27] which is Stripe [12:56:35] let's confirm that lol [12:57:16] before the addition, it used to be the connection with the 5% gets it for the set country and parameters. The remaining 95 is shared with the connections below it [12:58:10] hmm these numbers are interesting [12:58:52] total recurrings by gateway since we started recording the gateway against recurring charges: [12:59:10] cue flood throttle [12:59:17] +--------------------------------------+---------+------+ [12:59:17] | payment_service_id | gateway | cnt | [12:59:17] +--------------------------------------+---------+------+ [12:59:17] | 725322ad-198d-484d-b66c-e6ff44712108 | Adyen | 1490 | [12:59:17] | b4e49424-f463-4d0d-b05a-b427a4d21c95 | PayPal | 69 | [12:59:18] | c984b431-e034-45a1-9481-b6a090a9735e | Sepa | 24 | [12:59:18] | 9a259e13-f634-4145-aabe-5a2af1d2a695 | dLocal | 16 | [12:59:19] | b5f8051a-2f6c-414e-93c3-e10acf69b6ba | iDeal | 2 | [12:59:19] | 2a12608b-e9cc-483c-afc6-a9c214b101ed | Stripe | 2 | [12:59:20] +--------------------------------------+---------+------+ [12:59:22] 🥁 [12:59:30] only 2 for Stripe [12:59:47] I guess that means out of the 5%, only 2 of them decided to do recurring [13:00:10] hmm [13:00:39] are these new recurrings? [13:00:43] yep [13:01:05] between Adyen and Sepa that's < 0.5% [13:01:13] Adyen and Stripe* [13:01:43] I wonder if the total transactions match up though [13:02:50] ok looks like "Basic routing" is what we had in the past: https://docs.gr4vy.com/guides/dashboard/flow/card-transactions#outcome-route-to-connection [13:03:08] yeah that stripe number seems oddly low [13:03:09] lemme find the Slack thread where Piero explained it [13:04:07] hmm is there an easy way to see recurring trxns in the Gr4vy console? [13:04:20] only subsequent charges [13:04:37] using is_subsequent_payment= [13:04:43] is_subsequent_payment=true [13:05:14] thanks [13:05:20] so I see lots of those [13:05:26] lemme get the ID of the two new ones [13:06:51] re: flow rule slack thread: https://wikimedia.slack.com/archives/C070F1DVBRN/p1774885835001109?thread_ts=1774375605.498529&cid=C070F1DVBRN [13:08:10] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_46) - 10https://gerrit.wikimedia.org/r/1301017 (owner: 10L10n-bot) [13:08:26] https://wikimedia.gr4vy.app/merchants/default/transactions/2d1b940d-cfd8-416c-9e8d-a9f5fa701e94/overview [13:08:45] and https://wikimedia.gr4vy.app/merchants/default/transactions/7899be5e-ce75-4bf2-ac73-fdd03c041433/overview [13:09:18] seems surprisingly that there is only 2 [13:10:35] -ly [13:13:57] oh this is only for gravy trxns too [13:14:04] that might narrow it [13:17:21] oh right the Adyen direct connection too would be there [13:21:44] I've found a few cases where a new gr4vy recurring lead to us not recording the payment method id [13:21:46] https://wikimedia.gr4vy.app/merchants/default/transactions/5ca6d487-c249-4abe-a7e4-fe6764c3f9ce/overview [13:21:58] ~35 [13:30:17] hmm [13:32:54] ah I wonder if the audits are creating recurrings [13:33:01] and we need to update that [13:40:09] ah found it [13:40:18] we have two ways recurrings get created [13:40:28] immediate and delayed (via monthly convert) [13:40:40] the monthly converts go via a different queue consumer [13:40:55] I need update that to also populate the payment_service_id [13:41:17] DonationQueueConsumer::importContributionRecur & RecurringQueueConsumer::importSubscriptionSignup [14:01:10] (03PS1) 10Jgleeson: Store payment_service_id for monthly convert recurring signups [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301369 (https://phabricator.wikimedia.org/T422416) [14:01:51] that's the missing patch [14:01:59] gotta go grab the kids. back soon! [14:05:20] looking [14:33:32] 03Fundraising Sprint: Jack's Junk Drawer, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Don't store content of thank you email and annual summary receipts - https://phabricator.wikimedia.org/T421558#12014528 (10Lars) After some investigation and discussion, we've determined the best path forward is... [14:38:42] (03CR) 10Damilare Adedoyin: [C:03+2] Store payment_service_id for monthly convert recurring signups [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301369 (https://phabricator.wikimedia.org/T422416) (owner: 10Jgleeson) [14:38:42] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Add documentation of all transactional emails that we send to Wikitech - https://phabricator.wikimedia.org/T428937#12014553 (10Lars) Expanding on https://wikitech.wikimedia.org/wiki/Fundraising/Internal-facing/CiviCRM#Automated_emai... [14:38:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Change default theme on our local CiviCRM - https://phabricator.wikimedia.org/T428804#12014556 (10Lars) a:03Lars [14:40:34] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 13Patch-For-Review: Available dashlets accordion header in CiviCRM is white instead of dark in light mode since 6.16 - https://phabricator.wikimedia.org/T428953#12014560 (10Lars) a:03Lars [14:44:20] oh man damilare, sounds like those trustly tokens are unrecoverable, huh? [14:45:21] unfortunately so, though the 5 on the list are now marked as ready [14:46:10] but any similar error in future would be unrecoverable [14:55:36] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: failed to map same cid with email - https://phabricator.wikimedia.org/T428929#12014601 (10Lars) @AnnWF I assume this was some kind of testing? Is she OK with having her name details up here? We might want to remove your... [14:58:55] (03Merged) 10jenkins-bot: Store payment_service_id for monthly convert recurring signups [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301369 (https://phabricator.wikimedia.org/T422416) (owner: 10Jgleeson) [15:13:53] Thanks for the review dami! [15:13:56] I'll push that out [15:14:38] we had ~35 recurring donations come in via monthly convert that we missed saving the payment service id on due to that gap [15:18:10] ok, I need to study up on pandas and scikit-learn. Am having a heck of a time trying to output a confusion matrix with those percentage-valued labels [15:18:53] I guess I can also try just forcing all the sort-of-fraudy outcomes to 1 and the probably-not-fraudy outcomes to zero [15:29:00] 06Fundraising-Backlog, 06FR-donorrelations: Inability to cancel Venmo/Braintree recurring donations in Civi - https://phabricator.wikimedia.org/T429031 (10AMJohnson) 03NEW [15:30:52] 06Fundraising-Backlog, 10Phabricator-Bot-Requests, 06Release-Engineering-Team (Doing 😎): Phabricator Bot Request: Fundraising Tech Bug Reporter - https://phabricator.wikimedia.org/T426459#12014704 (10AKanji-WMF) Thank you! [15:46:31] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Inability to cancel Venmo/Braintree recurring donations in Civi - https://phabricator.wikimedia.org/T429031#12014745 (10Damilare) [15:46:38] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Inability to cancel Venmo/Braintree recurring donations in Civi - https://phabricator.wikimedia.org/T429031#12014746 (10Damilare) a:03Damilare [15:47:09] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1301394 [15:54:42] oh yh ejegg I think, the matrix only classifies to T/F states. In order to be able to calculate the true positive/false negative values [15:55:00] so yh you probably need to decide on a threshold [16:11:08] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 05Evelyn Prioritization/Mini Sprint: Enable UPI to Dlocal via Gr4vy - https://phabricator.wikimedia.org/T408459#12014785 (10Lars) [16:18:42] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 05Evelyn Prioritization/Mini Sprint: Enable UPI to Dlocal via Gr4vy - https://phabricator.wikimedia.org/T408459#12014795 (10EMartin) 05Open→03Resolved a:03EMartin We are unable to process in India so this task is now moot. [16:28:59] (03PS1) 10Wfan: Make recurring cancel job avaliable for all payment methods [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1301398 [16:34:45] (03PS1) 10Lars SG: Add full primary address to all example contacts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301399 (https://phabricator.wikimedia.org/T428834) [16:35:25] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 13Patch-For-Review: Add a primary address for example Barb in Civi - https://phabricator.wikimedia.org/T428834#12014848 (10Lars) a:03Lars [16:59:57] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1301394 (owner: 10Jgleeson) [17:54:29] 06Fundraising-Backlog: Delete Recurring Payment tokens on Venmo when cancelled on Gravy - https://phabricator.wikimedia.org/T429044 (10Damilare) 03NEW [17:57:53] ok I'm giving up on CI [17:58:39] https://usercontent.irccloud-cdn.com/file/J3SC0KYX/image.png [17:58:52] I guess the Friday rule holds [17:59:09] 06Fundraising-Backlog: Delete Recurring Payment tokens on Venmo when cancelled on CiviCRM - https://phabricator.wikimedia.org/T429044#12015118 (10Damilare) [18:00:13] (03CR) 10Jgleeson: [C:03+2] "Discussed this on our internal call today and decided we will upgrade the other tools using this library to follow. Thanks again for this!" [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/1294398 (https://phabricator.wikimedia.org/T425578) (owner: 10Umherirrender) [18:05:31] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1301394 (owner: 10Jgleeson) [18:05:53] what [18:07:17] ok [18:07:21] it merged [18:07:24] I'll push it out [18:08:54] !log civicrm upgraded from 69a60dcb to 646d893e [18:08:54] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [18:10:11] have a good weekend all [18:27:51] (03CR) 10Ejegg: [C:03+2] Add full primary address to all example contacts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301399 (https://phabricator.wikimedia.org/T428834) (owner: 10Lars SG) [18:30:38] ok, I have a model with 52% precision! [18:31:08] let's pickle that and plug it in to the webservice [18:42:08] (03Merged) 10jenkins-bot: build: Update predis/predis to 2.4 [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/1294398 (https://phabricator.wikimedia.org/T425578) (owner: 10Umherirrender) [18:42:16] FIRING: ContextSwitchingSpike: Host context switching high (instance frdb1008.frack.eqiad.wmnet:9100) - https://alerts.wikimedia.org/?q=alertname%3DContextSwitchingSpike [18:43:11] FIRING: NodeDown: Node frdb1008 is down. - https://frmon.wikimedia.org/d/000000377/host-overview?orgId=1&var-host=frdb1008 - https://alerts.wikimedia.org/?q=alertname%3DNodeDown [18:43:24] ^^^ those are from a reboot. host is alredy back up. [18:48:16] RESOLVED: NodeDown: Node frdb1008 is down. - https://frmon.wikimedia.org/d/000000377/host-overview?orgId=1&var-host=frdb1008 - https://alerts.wikimedia.org/?q=alertname%3DNodeDown [18:49:16] (03Merged) 10jenkins-bot: Add full primary address to all example contacts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301399 (https://phabricator.wikimedia.org/T428834) (owner: 10Lars SG) [18:57:16] RESOLVED: ContextSwitchingSpike: Host context switching high (instance frdb1008.frack.eqiad.wmnet:9100) - https://alerts.wikimedia.org/?q=alertname%3DContextSwitchingSpike [19:21:43] hmm, mysqls server has gone away [19:21:47] oh, same reboot [19:24:01] i didn't see any active connections when i restarted it. [19:24:10] at least reported to mariadb. [19:24:37] huh - there was a pending queue consumer failmail right then [19:25:20] oh but sometimes that 'gone away' error for pending queue consumer is because it holds the connection open for a long time while waiting for new messages [19:25:26] so it might not have been active right then [19:53:42] ah. could be. [19:53:50] huh, still processing a couple unsubscribes per day via the old unsubscribe extension [19:53:58] guess we can't delete it yet [19:57:23] (03PS1) 10Ejegg: Update FundraisingEmailUnsubscribe [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1301422 [22:25:27] (03PS2) 10Umherirrender: Use abstract fns in trait, not phan suppression [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1208396 (owner: 10Ejegg) [22:27:01] (03CR) 10CI reject: [V:04-1] Use abstract fns in trait, not phan suppression [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1208396 (owner: 10Ejegg) [22:30:04] (03PS3) 10Umherirrender: Use abstract fns in trait, not phan suppression [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1208396 (owner: 10Ejegg) [22:34:34] (03CR) 10CI reject: [V:04-1] Use abstract fns in trait, not phan suppression [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1208396 (owner: 10Ejegg) [22:46:27] (03PS4) 10Umherirrender: Use abstract fns in trait, not phan suppression [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1208396 (owner: 10Ejegg) [23:24:36] (03PS2) 10Lars SG: Add MG Stage Change activity when user edits [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300277 (https://phabricator.wikimedia.org/T386961) [23:24:36] (03PS1) 10Lars SG: Add hasValue() for pre hook to check if a value is included without confusing null [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1301454 (https://phabricator.wikimedia.org/T386961) [23:25:18] (03CR) 10Lars SG: "Weird, but after reverting the afform, it now works as expected on staging. I made a few other small improvements." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300277 (https://phabricator.wikimedia.org/T386961) (owner: 10Lars SG)