[00:56:26] (03PS1) 10Eileen: Do separate update for no-email emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138130 (https://phabricator.wikimedia.org/T270624) [01:15:48] (03PS2) 10Eileen: Do separate update for no-email emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138130 (https://phabricator.wikimedia.org/T270624) [01:15:48] (03PS1) 10Eileen: Extract days & earliest contribution days [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138131 [01:15:48] (03PS1) 10Eileen: Extract getMesssageLimit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 [01:15:49] (03PS1) 10Eileen: Extract update contributions without email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138133 [01:15:49] (03PS1) 10Eileen: Proposal - remove in-app disabling of thank you emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138134 [01:18:18] (03CR) 10Eileen: [C:03+2] More TY test cleanup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138091 (owner: 10Ejegg) [01:21:31] (03PS8) 10Wfan: Replace use api3 to api4 for epc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135845 (https://phabricator.wikimedia.org/T386871) [01:22:18] (03CR) 10Eileen: "@wfan@wikimedia.org - this looks like it might have been otherwise merged & can be abandoned?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135845 (https://phabricator.wikimedia.org/T386871) (owner: 10Wfan) [01:26:57] (03CR) 10Eileen: "I take that back - patchset 6 + are empty - but patchset 5 does not seem merged? did this get blanked?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135845 (https://phabricator.wikimedia.org/T386871) (owner: 10Wfan) [01:28:26] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 10FR-email-preferences, 13Patch-For-Review: update Email Preferences queue consumer to use api4 - https://phabricator.wikimedia.org/T386871#10759233 (10Eileenmcnaughton) I took a look at https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm... [01:29:45] (03CR) 10Eileen: [C:03+2] Handle new link messages with email or id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135147 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [01:30:46] (03CR) 10Eileen: [C:03+2] Add DonorPortal to new checksum link consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137287 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [01:33:31] (03CR) 10Eileen: [C:03+2] API call for donor portal summary (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137288 (https://phabricator.wikimedia.org/T388744) (owner: 10Ejegg) [01:34:34] (03CR) 10Eileen: [C:03+2] Also send recurring failure email on autorescue failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135155 (https://phabricator.wikimedia.org/T386733) (owner: 10Cstone) [01:35:25] (03PS3) 10Jgleeson: WIP: Add testMergeRetainsStateInAddress to confirm bug in T379684 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1098513 (https://phabricator.wikimedia.org/T379684) [01:36:22] (03CR) 10Eileen: [C:03+2] "I think the intent with this test was to prove a bug that did not show up in the test - However, it's still additional test cover - if it " [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1098513 (https://phabricator.wikimedia.org/T379684) (owner: 10Jgleeson) [01:36:48] (03PS4) 10Eileen: Add testMergeRetainsStateInAddress to confirm bug in T379684 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1098513 (https://phabricator.wikimedia.org/T379684) (owner: 10Jgleeson) [01:49:20] (03CR) 10CI reject: [V:04-1] Extract getMesssageLimit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 (owner: 10Eileen) [01:54:16] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 (owner: 10Eileen) [02:09:58] (03Merged) 10jenkins-bot: Handle new link messages with email or id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135147 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [02:10:13] (03Merged) 10jenkins-bot: Add DonorPortal to new checksum link consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137287 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [02:35:12] (03Merged) 10jenkins-bot: API call for donor portal summary [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137288 (https://phabricator.wikimedia.org/T388744) (owner: 10Ejegg) [02:35:15] (03Merged) 10jenkins-bot: Also send recurring failure email on autorescue failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135155 (https://phabricator.wikimedia.org/T386733) (owner: 10Cstone) [02:50:10] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1098513 (https://phabricator.wikimedia.org/T379684) (owner: 10Jgleeson) [02:52:10] (03PS5) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [02:52:10] (03PS3) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [02:52:10] (03PS7) 10Eileen: WIP - mgd files to support imports [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [02:52:11] (03PS1) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [02:52:12] (03PS1) 10Eileen: Switch contact creates to apiv4 in Benevity Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138141 (https://phabricator.wikimedia.org/T359219) [03:11:28] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_39) - 10https://gerrit.wikimedia.org/r/1137609 (owner: 10L10n-bot) [03:12:44] (03PS2) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [03:12:44] (03PS2) 10Eileen: Switch contact creates to apiv4 in Benevity Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138141 (https://phabricator.wikimedia.org/T359219) [03:16:20] (03Merged) 10jenkins-bot: Add testMergeRetainsStateInAddress to confirm bug in T379684 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1098513 (https://phabricator.wikimedia.org/T379684) (owner: 10Jgleeson) [03:16:56] (03PS3) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [03:17:19] (03CR) 10CI reject: [V:04-1] Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 (owner: 10Eileen) [03:39:12] (03CR) 10CI reject: [V:04-1] Switch contact creates to apiv4 in Benevity Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138141 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [03:50:59] (03PS3) 10Eileen: Switch contact creates to apiv4 in Benevity Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138141 (https://phabricator.wikimedia.org/T359219) [04:51:53] 06Fundraising-Backlog: Cz Republic Online Banking disabled - https://phabricator.wikimedia.org/T392393#10759335 (10RKumar_WMF) Both our CZ testers are out on vacation so I am unable to test it. We will have to reply on organic transaction coming in. [07:07:44] (03CR) 10Jdlrobson: Fixes warning: mediawiki.Uri is deprecated, use URL instead (032 comments) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1125253 (https://phabricator.wikimedia.org/T380079) (owner: 10Jdlrobson) [13:01:03] (03CR) 10Jgleeson: "Thanks for all the review on this wfan! Based on your last comment, it looks like it needs some fixes. Happy to try and resolve those issu" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1136010 (https://phabricator.wikimedia.org/T381012) (owner: 10Jgleeson) [13:07:48] 06Fundraising-Backlog, 07payments-orchestration: Log 'pending' details JSON blob for all authorized Gravy transactions - https://phabricator.wikimedia.org/T392156#10760651 (10jgleeson) is this related to some of the potential missing data encountered when running https://gerrit.wikimedia.org/r/c/wikimedia/fund... [13:33:02] 06Fundraising-Backlog, 10FR-email-preferences: Comms preference centre - email duplication check - https://phabricator.wikimedia.org/T385911#10760746 (10AKanji-WMF) Moving back to triage to discuss with @AnnWF [13:36:02] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failmail: Can't map fiscal number to Gravy Tax ID type. (US:XXXX) - https://phabricator.wikimedia.org/T392485 (10jgleeson) 03NEW [13:41:01] (03CR) 10Jgleeson: "We should be checking for these here https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1136010/4/Maintenance/BuildDonation" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1136010 (https://phabricator.wikimedia.org/T381012) (owner: 10Jgleeson) [13:43:50] (03PS1) 10Jgleeson: Revert "Convert docker cp to docker compose cp to remove intermediate directories." [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 [14:55:35] 10fundraising-tech-ops, 10FR-Tech-Analytics, 06Privacy Engineering, 06secscrum, 10Security Preview: Security review for python package dbt-metabase - https://phabricator.wikimedia.org/T385337#10761126 (10Dwisehaupt) @Mstyles Thanks, no particular date, but we would like to get it in place fairly soon. [15:18:01] PROBLEM - Host payments2005 is DOWN: PING CRITICAL - Packet loss = 100% [15:19:57] PROBLEM - Host payments2004 is DOWN: PING CRITICAL - Packet loss = 100% [15:19:57] PROBLEM - Host payments2006 is DOWN: PING CRITICAL - Packet loss = 100% [15:23:03] RECOVERY - Host payments2005 is UP: PING OK - Packet loss = 0%, RTA = 30.39 ms [15:24:59] RECOVERY - Host payments2004 is UP: PING OK - Packet loss = 0%, RTA = 30.41 ms [15:24:59] RECOVERY - Host payments2006 is UP: PING OK - Packet loss = 0%, RTA = 30.40 ms [15:28:42] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM Message Templates: "Draft" version being sent instead of "Active" - https://phabricator.wikimedia.org/T392300#10761381 (10Ejegg) I can reproduce this in development, and am trying to figure out where this behavior started. Unfortunately I've gotte... [15:31:12] 10fundraising-tech-ops, 10FR-Tech-Analytics, 06Privacy Engineering, 06secscrum, 10Security Preview: Security review for python package dbt-metabase - https://phabricator.wikimedia.org/T385337#10761402 (10Mstyles) @Dwisehaupt thank you for the update. [15:55:55] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM Message Templates: "Draft" version being sent instead of "Active" - https://phabricator.wikimedia.org/T392300#10761579 (10Ejegg) Back to 5f31fb9d193708e8d0dc8ca7f602531fee0475ff from September 2023 and still seeing the bug :S [16:08:18] 06Fundraising-Backlog, 10Recurring-Donations, 07payments-orchestration: Tax ID mapping fails for Gravy donors whose country changes - https://phabricator.wikimedia.org/T392376#10761637 (10jgleeson) [16:08:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failmail: Can't map fiscal number to Gravy Tax ID type. (US:XXXX) - https://phabricator.wikimedia.org/T392485#10761640 (10jgleeson) →14Duplicate dup:03T392376 [17:07:50] 06Fundraising-Backlog: Gravy Paypal recurrings failing on smashpig recurring charge due to no payment method - https://phabricator.wikimedia.org/T392369#10762011 (10Cstone) This was the patch that fixed it https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1137806 Checked today and the recurrings... [17:08:22] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Paypal recurrings failing on smashpig recurring charge due to no payment method - https://phabricator.wikimedia.org/T392369#10762017 (10Cstone) a:03Cstone [17:08:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Paypal recurrings failing on smashpig recurring charge due to no payment method - https://phabricator.wikimedia.org/T392369#10762019 (10Cstone) p:05Triage→03Unbreak! [17:09:18] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Paypal recurrings failing on smashpig recurring charge due to no payment method - https://phabricator.wikimedia.org/T392369#10762024 (10Cstone) p:05Unbreak!→03High [17:11:48] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices: Civi recur failure message emails not consistently deploying - https://phabricator.wikimedia.org/T386733#10762036 (10AKanji-WMF) do we need to do {T356073} before we do this @Cstone ? [17:14:36] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices: Civi recur failure message emails not consistently deploying - https://phabricator.wikimedia.org/T386733#10762053 (10Cstone) @AKanji-WMF no that is a separate thing, the one mentioned in this phab happens in the recurring queue consu... [17:16:42] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap: Payment method migration: PayPal - Import tokens to Civi - https://phabricator.wikimedia.org/T392025#10762057 (10AKanji-WMF) [17:17:12] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap: Payment method migration: PayPal - Import tokens to Civi - https://phabricator.wikimedia.org/T392025#10762061 (10AKanji-WMF) [17:17:12] 06Fundraising-Backlog, 07Fundraising-FRUP: Payment method migration: PayPal - https://phabricator.wikimedia.org/T392021#10762062 (10AKanji-WMF) [17:17:13] 06Fundraising-Backlog, 07Fundraising-FRUP: Payment method migration: Apple Pay and Google Pay - https://phabricator.wikimedia.org/T391996#10762063 (10AKanji-WMF) [17:17:14] 06Fundraising-Backlog, 07Fundraising-FRUP: Payment method migration: ACH - https://phabricator.wikimedia.org/T391992#10762064 (10AKanji-WMF) [17:17:16] 06Fundraising-Backlog, 07Fundraising-FRUP: Payment method migration: Credit cards - Import tokens to Civi - https://phabricator.wikimedia.org/T391991#10762065 (10AKanji-WMF) [17:17:17] 06Fundraising-Backlog, 07Fundraising-FRUP: Payment method migration: Credit cards - https://phabricator.wikimedia.org/T391990#10762066 (10AKanji-WMF) [17:17:18] 06Fundraising-Backlog, 07Fundraising-FRUP: Discuss Token migration and requirements with Gravy - https://phabricator.wikimedia.org/T391982#10762067 (10AKanji-WMF) [17:17:22] 06Fundraising-Backlog, 07Fundraising-FRUP, 07Epic: Fundraiseup migration - https://phabricator.wikimedia.org/T391961#10762068 (10AKanji-WMF) [17:17:53] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap: Payment method migration: PayPal - https://phabricator.wikimedia.org/T392021#10762074 (10AKanji-WMF) [17:18:08] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap: Payment method migration: Credit cards - Import tokens to Civi - https://phabricator.wikimedia.org/T391991#10762082 (10AKanji-WMF) [17:18:10] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap: Payment method migration: Credit cards - https://phabricator.wikimedia.org/T391990#10762084 (10AKanji-WMF) [17:18:13] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap: Discuss Token migration and requirements with Gravy - https://phabricator.wikimedia.org/T391982#10762086 (10AKanji-WMF) [17:18:28] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap, 07Epic: Fundraiseup migration - https://phabricator.wikimedia.org/T391961#10762091 (10AKanji-WMF) [17:21:55] 06Fundraising-Backlog: comms pref centre; instrumenting usage - https://phabricator.wikimedia.org/T392228#10762111 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [17:23:45] 06Fundraising-Backlog, 07payments-orchestration: Spike: Detect any potential issues after we migrate to gravy fully - https://phabricator.wikimedia.org/T392394#10762128 (10AKanji-WMF) [17:37:13] pcoombe: we're seeing gravy failing for all card and bank transactions - could you please redirect those back to Adyen from donatewiki? [17:45:54] (03PS1) 10Ejegg: Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 [17:46:20] cstone / jgleeson that should get us failmails when this happens ^^^ [17:46:42] logging pending? [17:47:08] ack , how did I get that there? [17:47:19] ugh, I must have made the edit in a different copy of the file [17:47:26] sorry, lemme fix that [17:47:30] sure [17:48:54] (03PS2) 10Ejegg: Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 [17:49:10] ok jgleeson, that's the one I actually wanted to push up ^^^ [17:49:33] checking [17:50:02] (03CR) 10Jgleeson: [C:03+2] Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 (owner: 10Ejegg) [17:50:20] thanks! [17:51:14] (03CR) 10CI reject: [V:04-1] Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 (owner: 10Ejegg) [17:51:27] oops [17:51:39] test not mocking that response? [17:52:03] ah no, just lint [17:52:09] ya [17:52:54] (03PS3) 10Ejegg: Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 [17:54:52] i guess i could lint changed files in a commit hook [17:55:08] save some cycles in CI-land [17:57:21] it looks like there are some changes to the gr4vy SDK between our version and the latest https://github.com/gr4vy/gr4vy-php/compare/v0.23.2...v0.25.0 [17:58:52] actually that's not to the checkout session tho [18:00:42] fr-tech, gr4vy have confirm the issue with their SDK and are working on a fix [18:00:48] confirmed [18:01:54] we should still get them to switch donate wiki though, what channel would that even be or just peter hah [18:02:14] yeah has that not happened [18:02:27] i had found that JS a couple weeks ago [18:04:37] hmmm [18:04:45] I just pined emma and peter on the slack channel [18:04:59] it looks like its already done? [18:05:16] i dmed peter too [18:05:47] donate is sending me to adyen now [18:11:41] it's sending me there now [18:12:27] hmm now let's see when we started getting that error [18:15:53] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10762435 (10MDemosWMF) @Eileenmcnaughton I just caught that the previous test I did doesn't show the creation of the relationship on either CIDs 65076114... [18:16:02] we are at least logging it, right? [18:16:07] ah yeah [18:16:50] yep [18:16:57] quick scan shows early 2nd [18:17:00] 22nd* [18:17:15] just scanning earlier [18:17:18] lines up with when we got the last good donation [18:17:52] Apr 22 10:54:05 [18:20:58] argh [18:21:04] we've had 8000 errors [18:21:14] oof [18:21:25] approx [18:21:58] `logdog -f misc model_attributes_type` shows all hit from last last few days [18:22:17] hits* [18:23:22] `logdog --first -f misc model_attributes_type` is faster [18:24:37] welp, I guess if we merge https://gerrit.wikimedia.org/r/1138423 we'll at least know about it next time that endpoint breaks! [18:24:43] I fixed the lint thing [18:24:55] then got sidetracked trying to add a lint pre-commit hook [18:27:49] (03CR) 10Jgleeson: [C:03+2] Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 (owner: 10Ejegg) [18:28:01] I didn't realise it kicked off my +2 [18:28:58] also, fr-tech, gravy have patched the issue by restoring our instance back to the version that supported the broken SDK API param [18:29:13] ah, ok [18:29:31] so they also need to release a new SDK version? [18:30:17] I guess we must be the only ones using their PHP SDK if they are getting no errors from other clients, and we were on the same server version as the rest [18:31:02] 06Fundraising-Backlog: Faircom List Clean Up - https://phabricator.wikimedia.org/T390205#10762456 (10MDemosWMF) @Eileenmcnaughton Confirming that the trigger for data on deleted contacts is active? [18:31:35] ejegg: they just bumped up the SDK. Gonna test it now [18:33:26] oh cool [18:36:16] 06Fundraising-Backlog: Cz Republic Online Banking disabled - https://phabricator.wikimedia.org/T392393#10762459 (10EMartin) I see Online BT back up in CZ. - JSGXLXS4PCVZVL25 - completed transaction in the Adyen console. [18:36:20] 06Fundraising-Backlog: Cz Republic Online Banking disabled - https://phabricator.wikimedia.org/T392393#10762460 (10EMartin) 05Open→03Resolved a:03EMartin [18:36:21] pcoombe: has switched gravy back on [18:36:30] on donatewiki [18:36:38] evelyn had a successful test payment [18:36:49] thanks for keeping the IRC chan up to date jgleeson :) [18:37:04] are you on slack? [18:37:11] i should check more often [18:37:23] but i don't have a perma-window like I do with IRC [18:37:50] https://wikimedia.slack.com/archives/C5MUATA75/p1745431465750929 [18:38:22] :) [18:41:52] (03PS1) 10Jgleeson: Update gr4vy/gr4vy-php to v0.26.0 [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1138444 [18:44:49] 06Fundraising-Backlog: Direct Mail NCOA Updates - https://phabricator.wikimedia.org/T392252#10762484 (10MDemosWMF) Adding process draft doc for more background: https://docs.google.com/document/d/1iL7ptmwTOAq7ZH23bPRkAFGu3hQjXeHSCafqAWHhJk8/edit?tab=t.0#heading=h.zhhsjeaj8h8y [18:47:18] fr-tech is someone ok +2ing that gravy sdk bump so I can cut a new version of smashpig and pull it into payments for testing pls? [18:56:42] brb relocating for call [18:59:57] PROBLEM - Host frdb1004 is DOWN: PING CRITICAL - Packet loss = 100% [19:01:04] ^^ that's me kernel updating/rebooting. [19:03:13] RECOVERY - Host frdb1004 is UP: PING OK - Packet loss = 0%, RTA = 0.52 ms [19:06:56] (03CR) 10Ejegg: [C:03+2] Update gr4vy/gr4vy-php to v0.26.0 [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1138444 (owner: 10Jgleeson) [19:07:29] (03Merged) 10jenkins-bot: Update gr4vy/gr4vy-php to v0.26.0 [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1138444 (owner: 10Jgleeson) [19:10:13] 06Fundraising-Backlog: Rearrange ayden audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528 (10Cstone) 03NEW [19:10:25] 06Fundraising-Backlog: Rearrange adyen audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528#10762570 (10Cstone) [19:10:41] 06Fundraising-Backlog: Rearrange adyen audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528#10762571 (10Cstone) [19:10:42] 06Fundraising-Backlog: Setting up consolidated settlement reporting for all PSPs via Gr4vy for new recon process - https://phabricator.wikimedia.org/T391496#10762572 (10Cstone) [19:12:58] 06Fundraising-Backlog: Faircom List Clean Up - https://phabricator.wikimedia.org/T390205#10762581 (10Eileenmcnaughton) Yep it is [19:16:25] 06Fundraising-Backlog: Rearrange adyen audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528#10762587 (10Ejegg) Ooh, let's also make it go by the header not the column order [19:16:54] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap: Rename monthly convert to recurring convert - https://phabricator.wikimedia.org/T392274#10762589 (10AKanji-WMF) [19:17:24] 06Fundraising-Backlog, 07payments-orchestration: Investigate moving native apple and google pay to gravy - https://phabricator.wikimedia.org/T392259#10762593 (10AKanji-WMF) [19:18:31] (03Merged) 10jenkins-bot: Failmail on Gravy session creation failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138423 (owner: 10Ejegg) [19:18:41] 06Fundraising-Backlog, 07payments-orchestration: Add text description to Payments India PAN form field - https://phabricator.wikimedia.org/T392103#10762597 (10AKanji-WMF) [19:19:02] 06Fundraising-Backlog, 07payments-orchestration: Add Frontend validation to Gravy Phone number field - https://phabricator.wikimedia.org/T391970#10762600 (10AKanji-WMF) [19:22:06] 06Fundraising-Backlog, 07payments-orchestration: Add Frontend validation to Gravy Phone number field - https://phabricator.wikimedia.org/T391970#10762606 (10AnnWF) https://wikimedia.slack.com/archives/C070F1DVBRN/p1744695794255839 fyi [19:25:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Smashpig, 07payments-orchestration: Gravy API::getTransaction should handle bad response more gracefully - https://phabricator.wikimedia.org/T392383#10762615 (10AKanji-WMF) [19:27:45] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Recurring-Donations, 07payments-orchestration: Tax ID mapping fails for Gravy donors whose country changes - https://phabricator.wikimedia.org/T392376#10762618 (10AKanji-WMF) p:05Triage→03Medium [19:29:38] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: Log 'pending' details JSON blob for all authorized Gravy transactions - https://phabricator.wikimedia.org/T392156#10762625 (10AKanji-WMF) [19:31:54] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10762632 (10Eileenmcnaughton) It's because the relationships only get created in the Contribution date is in the last 3 months - if this is too restricti... [19:31:56] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10762633 (10Eileenmcnaughton) It's because the relationships only get created in the Contribution date is in the last 3 months - if this is too restricti... [19:32:26] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM Message Templates: "Draft" version being sent instead of "Active" - https://phabricator.wikimedia.org/T392300#10762634 (10Eileenmcnaughton) @Ejegg originally no one actually used the draft feature so it probably worked by virtue of there being no... [19:32:56] 06Fundraising-Backlog, 07payments-orchestration: Add Frontend validation to Gravy Phone number field - https://phabricator.wikimedia.org/T391970#10762635 (10AnnWF) a:03AnnWF [19:34:55] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1138105 (owner: 10Eileen) [19:35:32] 06Fundraising-Backlog, 07payments-orchestration: Switch Credit Cards from Adyen to Gravy - https://phabricator.wikimedia.org/T392147#10762638 (10AKanji-WMF) [19:50:04] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Donor-portal, 10FR-email-preferences: Create donor portal main page displaying info from CiviCRM - https://phabricator.wikimedia.org/T388745#10762703 (10AKanji-WMF) @ehughes noting we are ready for your html for this page + the login page. [19:52:29] 06Fundraising-Backlog: Faircom List Clean Up - https://phabricator.wikimedia.org/T390205#10762718 (10MDemosWMF) @Eileenmcnaughton Great! We should be good to close this task and just monitor that alert. [19:53:23] PROBLEM - Host frbast2002 is DOWN: PING CRITICAL - Packet loss = 100% [19:58:11] RECOVERY - Host frbast2002 is UP: PING OK - Packet loss = 0%, RTA = 30.40 ms [20:12:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10762799 (10MBeat33) Hi @AnnWF the four transactions mentioned in the April 17th comment are not in Civi yet. The first three are at... [20:22:34] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10762822 (10MBeat33) One other straggler, for which Dominic at Gravy has a ticket open. Zendesk 1659060 Merch ref 229591889.1 is stil... [20:32:29] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Rearrange adyen audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528#10762849 (10AKanji-WMF) [20:32:31] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Switch Credit Cards from Adyen to Gravy - https://phabricator.wikimedia.org/T392147#10762850 (10AKanji-WMF) [20:32:33] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Add Frontend validation to Gravy Phone number field - https://phabricator.wikimedia.org/T391970#10762851 (10AKanji-WMF) [20:32:35] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Setting up consolidated settlement reporting for all PSPs via Gr4vy for new recon process - https://phabricator.wikimedia.org/T391496#10762852 (10AKanji-WMF) [20:32:37] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 07Unplanned-Sprint-Work: Merge master into wmf_deploy (Mar 31, 2025) - https://phabricator.wikimedia.org/T390634#10762853 (10AKanji-WMF) [20:32:40] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10762854 (10AKanji-WMF) [20:32:42] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Donor-portal, 10FR-email-preferences: Create donor portal main page displaying info from CiviCRM - https://phabricator.wikimedia.org/T388745#10762855 (10AKanji-WMF) [20:32:44] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donor-portal, 10FR-email-preferences: Create CiviCRM Api call to retrieve information for donor portal main page - https://phabricator.wikimedia.org/T388744#10762856 (10AKanji-WMF) [20:32:50] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences, 13Patch-For-Review: Allow donors to request a donor portal login link by email - https://phabricator.wikimedia.org/T388739#10762857 (10AKanji-WMF) [20:32:55] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10MW-1.44-notes (1.44.0-wmf.22; 2025-03-25), 07Unplanned-Sprint-Work: Remove address fields for ACH in checkout flow - https://phabricator.wikimedia.org/T387758#10762858 (10AKanji-WMF) [20:32:57] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: How will we expose payment processor errors/issues with gravy - https://phabricator.wikimedia.org/T387411#10762859 (10AKanji-WMF) [20:33:01] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10FR-email-preferences, 13Patch-For-Review: update Email Preferences queue consumer to use api4 - https://phabricator.wikimedia.org/T386871#10762860 (10AKanji-WMF) [20:33:07] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10762861 (10AKanji-WMF) [20:33:11] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07Epic, 07payments-orchestration: Gravy dLocal country traffic routing - https://phabricator.wikimedia.org/T385017#10762862 (10AKanji-WMF) [20:33:15] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07Epic, 07payments-orchestration: Gravy Dlocal - https://phabricator.wikimedia.org/T384967#10762863 (10AKanji-WMF) [20:33:19] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe: Fix dedupe merge wrapper script to track modified date - https://phabricator.wikimedia.org/T383194#10762864 (10AKanji-WMF) [20:33:23] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Investigation: Paypal upgrade through Gr4vy - https://phabricator.wikimedia.org/T382371#10762865 (10AKanji-WMF) [20:33:27] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Apple Pay enablement across all browsers - https://phabricator.wikimedia.org/T382279#10762866 (10AKanji-WMF) [20:33:31] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07fr-acoustic: Export phone numbers to Acoustic - https://phabricator.wikimedia.org/T380946#10762867 (10AKanji-WMF) [20:33:35] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07fr-acoustic, 13Patch-For-Review: Develop process for tracking & handling SMS consent records - https://phabricator.wikimedia.org/T379702#10762868 (10AKanji-WMF) [20:33:43] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Set up recurring charge for venmo - https://phabricator.wikimedia.org/T376122#10762869 (10AKanji-WMF) [20:33:47] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Create WMF standalone config - https://phabricator.wikimedia.org/T372005#10762870 (10AKanji-WMF) [20:33:55] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: 2nd Recurring Failure Email in Civi - https://phabricator.wikimedia.org/T365488#10762871 (10AKanji-WMF) [20:34:03] 14Fundraising Sprint: didAnyoneTryThis(), 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 13Patch-For-Review: Create new Benevity import - https://phabricator.wikimedia.org/T359219#10762872 (10AKanji-WMF) [20:34:19] 14Fundraising Sprint: Abula, 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 2 others: Move wmf_civicrm module into wmf-civicrm extension - https://phabricator.wikimedia.org/T270677#10762874 (10AKanji-WMF) [20:34:28] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10donate.wikimedia.org: Provide a donate skin for donate.wikimedia.org - https://phabricator.wikimedia.org/T257862#10762875 (10AKanji-WMF) [20:34:32] 14Fundraising Sprint: Abula, 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Duplicates after import - https://phabricator.wikimedia.org/T382516#10762876 (10AKanji-WMF) [20:38:44] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10762898 (10MDemosWMF) @Eileenmcnaughton Ah right! When I run the test I'll do that with a more recent date. Will report back! [20:38:51] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10MW-1.44-notes (1.44.0-wmf.25; 2025-04-15): Bad UI from payments form when recurring=1 for non recurring methods - https://phabricator.wikimedia.org/T391412#10762899 (10AKanji-WMF) [20:38:53] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap: Decommission dLocal India recurring process - https://phabricator.wikimedia.org/T391393#10762900 (10AKanji-WMF) [20:39:16] 14Fundraising Sprint: Abula, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Finish moving wmf thank you functionality to the existing extension - https://phabricator.wikimedia.org/T270674#10762904 (10AKanji-WMF) [20:40:10] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences: Create donor portal main page displaying info from CiviCRM - https://phabricator.wikimedia.org/T388745#10762905 (10AKanji-WMF) [20:40:12] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07fr-acoustic: Export phone numbers to Acoustic - https://phabricator.wikimedia.org/T380946#10762906 (10AKanji-WMF) [20:40:17] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07fr-acoustic, 13Patch-For-Review: Develop process for tracking & handling SMS consent records - https://phabricator.wikimedia.org/T379702#10762907 (10AKanji-WMF) [20:40:21] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Create WMF standalone config - https://phabricator.wikimedia.org/T372005#10762908 (10AKanji-WMF) [20:40:24] 14Fundraising Sprint: didAnyoneTryThis(), 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Create new Benevity import - https://phabricator.wikimedia.org/T359219#10762909 (10AKanji-WMF) [20:40:44] eileen: so it looks like draft messages have been getting sent instead of active message for like 2 years [20:40:53] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fix bugs in smash-pig/Maintenance/BuildDonationMessageFromPaymentLogs - https://phabricator.wikimedia.org/T392536 (10jgleeson) 03NEW [20:41:00] or at least as far back as I've tested so far [20:41:14] (re https://phabricator.wikimedia.org/T392300) [20:41:39] anyway, I should finish reviewing your patch chain that fixes the TY mail exceptions [20:41:45] 10fundraising-tech-ops, 10BDC-Implementation, 06Data-Engineering, 06Data-Engineering-Radar, and 2 others: [Trino] Develop procedure and scripting for Trino cluster maintenance. - https://phabricator.wikimedia.org/T386391#10762924 (10Jgreen) 05Open→03Resolved With API permissions deployed, trino-top... [20:42:03] and then i'd like to understand the civi translations code that's supposed to respect that draft status [20:42:18] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07payments-orchestration: Spike: Detect any potential issues after we migrate to gravy fully - https://phabricator.wikimedia.org/T392394#10762928 (10AKanji-WMF) [20:42:20] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07payments-orchestration: Investigate moving native apple and google pay to gravy - https://phabricator.wikimedia.org/T392259#10762929 (10AKanji-WMF) [20:42:22] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07payments-orchestration: Add text description to Payments India PAN form field - https://phabricator.wikimedia.org/T392103#10762930 (10AKanji-WMF) [20:42:26] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 10MediaWiki-extensions-DonationInterface: Validate payments-wiki config - https://phabricator.wikimedia.org/T390777#10762931 (10AKanji-WMF) [20:42:28] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-email-preferences, 07fr-acoustic: Add SMS specific opt-out to communications preferences page - https://phabricator.wikimedia.org/T390757#10762932 (10AKanji-WMF) [20:43:33] 06Fundraising-Backlog: Faircom List Clean Up - https://phabricator.wikimedia.org/T390205#10762949 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF {meme, src="fox_dance"} [20:44:07] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap: DI add a countdown to ask donor accept direct bt from their phone - https://phabricator.wikimedia.org/T334310#10762954 (10AKanji-WMF) [20:44:20] (03CR) 10Ejegg: [C:04-1] "We should still set the enablement to false when the consecutive failure threshold is reached, though, even when we DO send the failmail" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138107 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [20:45:27] (03PS2) 10Eileen: Set up consolidatation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138108 (https://phabricator.wikimedia.org/T270624) [20:45:29] (03CR) 10Ejegg: [C:03+2] Set up consolidatation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138108 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [20:48:05] oops, can't rebase the rest around that one i C-1ed [20:48:21] (03CR) 10Eileen: "In terms of this patch it's no change - ie it's already broken cos we never check the variable - but do you really think we should be disa" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138107 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [20:48:48] ahh eileen, i see what you mean [20:48:57] we are checking for enablement via the other var [20:49:03] yeah so it's broken [20:49:29] so I think we do prefer to disable the job rather than marking every single attempted contirbution as no_thank_you=failed [20:49:50] hmm - so that is what would happen huh [20:49:56] 06Fundraising-Backlog: Ingenico contract termination 31 March 2024 means we need to process all non-India and LATAM transactions through Adyen - https://phabricator.wikimedia.org/T344646#10763016 (10AKanji-WMF) this is done, right @EMartin [20:50:06] have we been through this scenario in the last decade? [20:51:27] (03PS1) 10C. Scott Ananian: DNM: CI test after update to wikimedia/remex-html [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138475 [20:53:32] heh, I dunno [20:54:43] I guess we have 2 reasons they don't send [20:54:44] 1) contribution-relevant [20:54:44] 2) server issue [20:54:44] & the latter is what you are worried about [20:54:50] yeah [20:55:31] - I wonder if the exception type would be different [20:57:42] the thing is - if it ever happened you would have 5 incorrectly set to 'failed' - so you still have to fix some of the ones that got set to failed since the server glitched & the process for 5 (if you remember) is similar to the process for 10000 - find the ones with that value after x date & set to NULL [20:58:53] ok, i guess you have a point [21:00:30] 06Fundraising-Backlog: Ingenico contract termination 31 March 2024 means we need to process all non-India and LATAM transactions through Adyen - https://phabricator.wikimedia.org/T344646#10763045 (10EMartin) Yes! Very done! [21:01:30] So if we're getting rid of the disablement, let's get rid of that var altogether [21:02:09] since the non-automated way of disabling jobs is just via process-control [21:02:42] oh i see, you have it still disabling on failure to failmail [21:02:43] right [21:03:43] sure, I guess we can go with that for now [21:04:10] (03PS2) 10Eileen: Remove drupal variable, minor cleanup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138107 (https://phabricator.wikimedia.org/T392387) [21:04:19] ejegg: top of the chain https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1138134 [21:05:13] (03CR) 10Ejegg: [C:03+2] "I've been convinced per IRC discussion that it's not so different to clean up 5 mistakenly marked no_thank_yous as it is to clean up 10,00" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138107 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [21:06:05] thanks eileen [21:06:38] (03PS2) 10Eileen: Extract startTime, endTime, timeLimit to simplify main flow [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138111 (https://phabricator.wikimedia.org/T270624) [21:06:42] (03CR) 10Ejegg: [C:03+2] Extract startTime, endTime, timeLimit to simplify main flow [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138111 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:09:26] (03PS2) 10Eileen: Fix batchSendTest to test something [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138112 (https://phabricator.wikimedia.org/T270624) [21:10:49] (03CR) 10Ejegg: [C:03+2] Fix batchSendTest to test something (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138112 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:11:33] (03PS2) 10Eileen: Extract, move & test activity Create [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138113 (https://phabricator.wikimedia.org/T270624) [21:12:23] (03CR) 10Ejegg: [C:03+2] "Yay tests! And yay for getting this into the right place!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138113 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:14:11] (03PS6) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [21:14:11] (03PS4) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [21:14:11] (03PS2) 10Eileen: CiviCRM upstream updates to 6.1.2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137097 [21:14:22] hmm, this guy got two change-ids by mistake https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1138116/1 [21:14:33] gonna just edit that commit message [21:15:41] ah wait let's see which is the right one to keep [21:15:44] (03CR) 10CI reject: [V:04-1] Set up consolidatation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138108 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:15:46] (03CR) 10CI reject: [V:04-1] Remove drupal variable, minor cleanup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138107 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [21:15:47] (03CR) 10CI reject: [V:04-1] Extract startTime, endTime, timeLimit to simplify main flow [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138111 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:15:47] (03CR) 10CI reject: [V:04-1] Fix batchSendTest to test something [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138112 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:15:48] (03CR) 10CI reject: [V:04-1] Extract, move & test activity Create [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138113 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:15:51] that or https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1137852 [21:16:05] and... let's get those V-1s dealt with first [21:16:27] nooo, 30 minute timeout! [21:16:49] some of those omni tests are taking 10 sec [21:17:19] (03PS1) 10Eileen: port date patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138477 (https://phabricator.wikimedia.org/T359219) [21:18:49] hmm, between "adding general wmf dev-specific settings" and "adding general wmf settings" we lose like 5 minutes [21:19:01] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138108 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:19:58] yeah, setup is slowwww - we start the CI run at 16:45:40 and then don't get to PHPUnit till 17:03:42 [21:21:52] there's another 3 min gap i think enabling the drupal modules [21:21:59] those timestamps are a bit odd [21:22:37] 16:54:50 advanced_help was enabled successfully. [196.95 sec, 97.14 MB] [21:22:54] guessing it's trying to phone home or something? [21:24:40] (03PS7) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [21:24:40] (03PS5) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [21:24:40] (03PS8) 10Eileen: mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [21:24:41] (03PS4) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [21:24:42] (03PS4) 10Eileen: Switch contact creates to apiv4 in Benevity Test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138141 (https://phabricator.wikimedia.org/T359219) [21:25:05] ejegg: yeah looking for updates maybe? [21:25:27] if it is on the drupal side then obv it doesn't warrant too much digging - but not necessarily on that side [21:25:47] hmm, but how about the dev-specific settings? [21:25:58] all that happens between those two lines in install.sh is this: [21:25:59] drush --in=json cvapi Setting.create < "$DEV_SETTINGS_FILE" [21:26:31] i guess we could try running that with cv [21:26:52] (03CR) 10Eileen: [V:03+2] Set up consolidatation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138108 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:27:30] I just forced the first one through so I can retry the next one [21:29:18] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10763093 (10AnnWF) [21:29:57] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Apple Pay enablement across all browsers - https://phabricator.wikimedia.org/T382279#10763096 (10AnnWF) a:05AnnWF→03None [21:30:34] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Apple Pay enablement across all browsers - https://phabricator.wikimedia.org/T382279#10763098 (10AnnWF) a:03AnnWF [21:35:17] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10763112 (10AnnWF) [21:35:35] (03CR) 10Eileen: [V:03+2] "putting back V+2 as jenkins is super snarled up right now" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138111 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [21:39:28] on top of all the chaos with that patch chain.... I've just got word from Melanie that she is happy with the Fidelity patch chain after her testing on staging (given the testing she has done review might just be a light once over?) [21:41:15] oh - the wmff settings is slow cos it enables components which would cause a Managed.reconcile etc [21:42:55] (03PS5) 10Eileen: Clean up thank you exception handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137852 (https://phabricator.wikimedia.org/T392387) [21:42:55] (03PS3) 10Eileen: Include previous exception as part of construct [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138117 [21:42:56] (03PS3) 10Eileen: Remove unused variable [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138119 (https://phabricator.wikimedia.org/T270624) [21:42:56] (03PS3) 10Eileen: Remove missing data cruft [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138118 [21:42:57] (03PS2) 10Eileen: Do not check the message age - we only just loaded it [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138124 [21:42:58] (03PS3) 10Eileen: Do separate update for no-email emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138130 (https://phabricator.wikimedia.org/T270624) [21:43:02] (03PS2) 10Eileen: Extract days & earliest contribution days [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138131 [21:43:06] (03PS2) 10Eileen: Extract getMesssageLimit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 [21:43:10] (03PS2) 10Eileen: Extract update contributions without email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138133 [21:43:15] (03PS2) 10Eileen: Proposal - remove in-app disabling of thank you emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138134 [21:43:33] (03Abandoned) 10Eileen: Clean up thank you exception handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138116 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [21:46:34] & also minor civi update https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1137097 - to 6.1.2 - nothing too major in it - but I wanted to port another patch & figured it was better over the top of the upstream update [21:46:54] ok, will take a look at that after the current stack [21:57:40] 06Fundraising-Backlog: Ingenico contract termination 31 March 2024 means we need to process all non-India and LATAM transactions through Adyen - https://phabricator.wikimedia.org/T344646#10763149 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [22:06:13] well on the bright side tests are coping again [22:06:46] yeah, i wish I understood why that stalled so long before [22:09:39] yeah :-( [22:10:38] (03CR) 10Ejegg: [C:04-1] "Just needs one more change away from the PHPMailer exception's errorMessage() fn. Also one question about setNoThankYou" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137852 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [22:12:42] ejegg: re that getMessage vs getErrorMessage - do you phpmailer returns the same for both? [22:13:10] (03CR) 10CI reject: [V:04-1] Remove missing data cruft [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138118 (owner: 10Eileen) [22:14:10] (03CR) 10CI reject: [V:04-1] Do separate update for no-email emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138130 (https://phabricator.wikimedia.org/T270624) (owner: 10Eileen) [22:14:20] I could do [22:14:21] `if (method_exists('getErrorMessage', $e) && strpos($e->errorMessage(), 'Invalid address:'))` [22:14:56] (03CR) 10CI reject: [V:04-1] Extract days & earliest contribution days [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138131 (owner: 10Eileen) [22:15:47] (03CR) 10CI reject: [V:04-1] Extract update contributions without email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138133 (owner: 10Eileen) [22:16:03] (03CR) 10CI reject: [V:04-1] Extract getMesssageLimit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 (owner: 10Eileen) [22:17:09] oh - `errorMessage()` is just 'helpfully formatted' `getMessage() [22:18:43] PROBLEM - Host frdb1003 is DOWN: PING CRITICAL - Packet loss = 100% [22:19:08] (03PS6) 10Eileen: Clean up thank you exception handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137852 (https://phabricator.wikimedia.org/T392387) [22:19:08] (03PS4) 10Eileen: Include previous exception as part of construct [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138117 [22:19:08] (03PS4) 10Eileen: Remove unused variable [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138119 (https://phabricator.wikimedia.org/T270624) [22:19:09] (03PS4) 10Eileen: Remove missing data cruft [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138118 [22:19:10] (03PS3) 10Eileen: Do not check the message age - we only just loaded it [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138124 [22:19:11] OK - updated! [22:19:13] (03PS4) 10Eileen: Do separate update for no-email emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138130 (https://phabricator.wikimedia.org/T270624) [22:19:17] (03PS3) 10Eileen: Extract days & earliest contribution days [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138131 [22:19:21] (03PS3) 10Eileen: Extract getMesssageLimit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138132 [22:19:25] (03PS3) 10Eileen: Extract update contributions without email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138133 [22:19:29] (03PS3) 10Eileen: Proposal - remove in-app disabling of thank you emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138134 [22:21:01] (03CR) 10Eileen: "Fixed!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137852 (https://phabricator.wikimedia.org/T392387) (owner: 10Eileen) [22:22:10] RECOVERY - Host frdb1003 is UP: PING OK - Packet loss = 0%, RTA = 0.53 ms [22:50:35] (03CR) 10CI reject: [V:04-1] Remove missing data cruft [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138118 (owner: 10Eileen) [22:51:31] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138118 (owner: 10Eileen) [22:52:17] (03CR) 10CI reject: [V:04-1] Proposal - remove in-app disabling of thank you emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138134 (owner: 10Eileen) [23:47:14] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138134 (owner: 10Eileen)