[00:55:27] (03PS1) 10Eileen: Reverse workplace/match_gift types in Benevity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1143977 (https://phabricator.wikimedia.org/T359219) [00:56:07] It would be really good to get that ^^ super minor patch out before Melanie does further testing with Ellen (on their Monday) [00:57:10] 14Fundraising Sprint: didAnyoneTryThis(), 14Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Create new Benevity import - https://phabricator.wikimedia.org/T359219#10810132 (10Eileenmcnaughton) I've just pushed up a patch to flip them - am hoping I can get it reviewed & deployed... [01:02:28] 06Fundraising-Backlog, 06FR-donorrelations: Civi recurring donations display question - https://phabricator.wikimedia.org/T393227#10810134 (10Eileenmcnaughton) It doesn't look like the tab has been converted to Search Kit (which would make it editable) - it would be possible to add an additional SearchKit 'top... [02:26:43] 06Fundraising-Backlog, 10fundraising-tech-ops: Documentation: how to import events to CiviCRM using the CSV Import extension - https://phabricator.wikimedia.org/T393172#10810145 (10Eileenmcnaughton) @Qgil go ahead & put something in my calendar - something like this works - https://www.timeanddate.com/worldclo... [04:59:48] 03Fundraising Sprint Impossible shepherds pie, 14Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 13Patch-For-Review: Apple Pay enablement across all browsers (no longer needed) - https://phabricator.wikimedia.org/T382279#10810191 (10AnnWF) 05Open→03Declined [06:07:11] PROBLEM - check_load on fransw2002 is CRITICAL: LOAD CRITICAL - total load average: 13.93, 20.11, 11.85 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [06:12:11] RECOVERY - check_load on fransw2002 is OK: LOAD OK - total load average: 1.07, 8.06, 8.88 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [10:21:52] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1143977 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [10:40:52] (03Merged) 10jenkins-bot: Reverse workplace/match_gift types in Benevity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1143977 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [11:10:09] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1144503 [11:24:16] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1144503 (owner: 10Jgleeson) [11:25:07] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1144503 (owner: 10Jgleeson) [11:45:58] !log civicrm upgraded from dc096105 to 852c6ee6 [11:45:59] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [12:42:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Civi recur fail message: confirm it checks for Gr4vy recurrings - https://phabricator.wikimedia.org/T393497#10811326 (10jgleeson) SQL used to count donors who recieved the recurring failure email when also having a separate active r... [13:25:20] (03PS4) 10Wfan: Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) [13:25:30] (03CR) 10Damilare Adedoyin: [C:03+2] Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [13:32:05] 10fundraising-tech-ops: Migrate backup fundraising analytics server from fran1001 to fran1002 - https://phabricator.wikimedia.org/T392818#10811628 (10Jgreen) 05In progress→03Resolved [13:32:37] (03Merged) 10jenkins-bot: Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [13:33:19] 10fundraising-tech-ops, 10BDC-Implementation, 06Privacy Engineering, 10Security Preview, 07SecTeam-Processed: [Trino] Investigate install, operation and maintenance, and security monitoring. - https://phabricator.wikimedia.org/T367818#10811638 (10Jgreen) 05Open→03Resolved [13:33:42] (03PS1) 10Jgleeson: Remove address fields on for Gravy ACH(Trustly) form [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1144575 (https://phabricator.wikimedia.org/T387758) [13:34:29] 10fundraising-tech-ops, 10BDC-Implementation, 06Privacy Engineering, 10Security Preview, 07SecTeam-Processed: [Trino] Investigate install, operation and maintenance, and security monitoring. - https://phabricator.wikimedia.org/T367818#10811656 (10Jgreen) Vulnerability and update tracking are working... [13:43:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 16004 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [13:48:17] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 16304 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [13:53:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 16604 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [13:58:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 16904 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:03:17] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 17204 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:07:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Null backend_processor for gravy records in civi - https://phabricator.wikimedia.org/T393477#10811812 (10jgleeson) Yep @EMartin. I pulled the missing backend data from Gravy on Friday via our backfill scr... [14:08:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 17504 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:13:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 17804 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:18:17] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 18104 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:20:28] Hmm, I'm importing contribution record updates into CiviCRM 5000 at a time, and it's timing out with bad gateway requests every other attempt. 5000 doesn't feel high. I guess I can break them into small chunks [14:23:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 18404 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:28:05] ejegg: I didn't make much progress reviewing your donor prefs patchchain last week, sorry. The analytics folks found a hole in our gravy data, https://phabricator.wikimedia.org/T393477, so I got distracted with that, but I will hopefully jump back to it today! [14:28:13] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 18704 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:33:15] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 19004 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:35:33] (03PS1) 10Damilare Adedoyin: Add validation to make sure amounts entered through the form are numeric [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144588 (https://phabricator.wikimedia.org/T392655) [14:36:25] (03CR) 10CI reject: [V:04-1] Add validation to make sure amounts entered through the form are numeric [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144588 (https://phabricator.wikimedia.org/T392655) (owner: 10Damilare Adedoyin) [14:37:37] (03PS2) 10Damilare Adedoyin: Add validation to make sure amounts entered through the form are numeric [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144588 (https://phabricator.wikimedia.org/T392655) [14:38:15] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 14190 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:39:09] yep, sounds like a good priority jgleeson|skrun [14:39:25] lmk if i can help with that gravy ticket [14:43:13] RECOVERY - check_mysql on frdb1004 is OK: Uptime: 1093145 Threads: 6 Questions: 46279650 Slow queries: 557 Opens: 3394 Open tables: 1246 Queries per second avg: 42.336 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:54:15] jgleeson|skrun: for that ACH patch I think we also need to check it with the buyer info coming from the gravy flow along with removing the fields [14:54:50] ejegg: I looked at a couple of your patches I left some review on the first one I think one of the messages is missing (or it wasnt showing up for me) [15:00:29] ah thanks cstone [15:02:24] oops, right, we can keep using the old message in that first patch [15:04:43] and yeah, email validation would be a good idea too [15:18:17] (03PS5) 10Ejegg: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) [15:18:17] (03PS6) 10Ejegg: Donor Portal login page [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1141917 (https://phabricator.wikimedia.org/T388739) [15:55:07] (03PS6) 10Ejegg: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) [15:55:07] (03PS7) 10Ejegg: Donor Portal login page [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1141917 (https://phabricator.wikimedia.org/T388739) [15:56:29] (03CR) 10Ejegg: "Thanks for the review cstone! I've added server-side email validation and marked the input as required. It should already have browser-nat" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [16:05:42] 06Fundraising-Backlog, 10fundraising-tech-ops: Documentation: how to import events to CiviCRM using the CSV Import extension - https://phabricator.wikimedia.org/T393172#10812587 (10Qgil) Thank you so much! Invite sent. [16:16:35] hi jgleeson|dinner is the plan to use pago efectivo in Peru for all BT and cash payments? [16:17:04] I moved this ticket to blocked before my break but I noticed you and wfan pushed changes to unblock it [16:17:10] T385012 [16:17:10] T385012: Gravy dLocal Bank Transfers (one-time) - https://phabricator.wikimedia.org/T385012 [16:32:14] (03PS1) 10Damilare Adedoyin: Ensure Math.floor does not pass NaN value to timezone offset. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1144613 (https://phabricator.wikimedia.org/T389411) [16:36:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Timezone offset is invalid is back with NaN - https://phabricator.wikimedia.org/T389411#10812785 (10Damilare) Checked the logs and found this transaction as the only occurence of this error, perhaps it was a onetime issue that adyen... [16:41:14] 06Fundraising-Backlog: Temporarily turn off AVS checks while Japan campaigns are in progress - https://phabricator.wikimedia.org/T393930 (10EMartin) 03NEW [16:42:00] 06Fundraising-Backlog: Temporarily turn off AVS checks while Japan campaigns are in progress - https://phabricator.wikimedia.org/T393930#10812808 (10EMartin) [16:49:45] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Already cancelled paypals from inactive script are uncancelling - https://phabricator.wikimedia.org/T387949#10812847 (10AMJohnson) Just adding another one that I believe is similar. Donor in #1675916/CID 58906727 reached out about an unwanted recurring do... [16:51:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Already cancelled paypals from inactive script are uncancelling - https://phabricator.wikimedia.org/T387949#10812884 (10MBeat33) [17:10:40] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Temporarily turn off AVS checks while Japan campaigns are in progress - https://phabricator.wikimedia.org/T393930#10813030 (10jgleeson) a:03jgleeson [17:13:47] ah ejegg I wasn't getting any browser validation thats why I said that at least in chrome [17:14:10] oh hmm, maybe it needs to be a sumbit button? [17:14:14] let me check [17:14:23] or maybe needs the form tag around it [17:14:23] firefox is letting me just put in letters too [17:14:39] ok, let me try again [17:15:04] lul still geolocating to turkey [17:15:07] we have it on the donate form right? [17:15:26] have what on the donate form? [17:15:32] email validation? [17:15:32] the email validation [17:15:43] oh yeah, we are using some ancient ugly JS for that [17:15:51] ah yeah i see now [17:15:59] but it's also inside real
tags [17:16:05] so let me try adding that [17:18:03] 06Fundraising-Backlog: Japan credit card processing error - https://phabricator.wikimedia.org/T393939 (10EMartin) 03NEW [17:21:42] (03PS2) 10Cstone: WIP: Add codex to gravy form [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1140994 [17:22:08] trolling myself with that WIP patch haha [17:23:31] (03CR) 10CI reject: [V:04-1] WIP: Add codex to gravy form [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1140994 (owner: 10Cstone) [17:25:21] jgleeson: damilare the phab above ^ looks like JP is on the list for fiscal number but I don't think it needs one? or I dont remember one for adyen [17:25:57] cstone: do you mean the one Evelyn just created [17:26:07] yeah [17:26:12] really [17:26:30] that should be dlocal only right? [17:26:30] yeah the validator errors [17:26:44] checking the log [17:27:07] the commit message days that but its in paymentprovidervalidator class doesn't mention dlocal? [17:27:09] "createPayment call came back with validation error in fiscal_number with message: required" [17:27:24] it's gravy [17:27:33] interesting I thought I added that for dlocal only [17:27:41] I'll take a peek too [17:27:50] its in FIELD_COUNTRY_REQUIREMENTS [17:27:51] JP [17:27:57] I fguess we couldn't know the gateway on the frontend [17:28:00] -f [17:28:05] so it couldn't be [17:28:16] oh interesting is it routing to dlocal then/ [17:28:26] maybe not [17:28:28] i just think JP doesnt need to be in this list though [17:28:37] we can just remove it from that list [17:28:40] I'll do that now [17:30:19] jgleeson: did you get the list from gravy's doc? [17:31:32] can't remember tbh damilare [17:31:46] ok so we're not asking for a fiscal number and we should be it seems [17:32:34] (it's not in our lists) [17:33:40] the docs has it as required on gravy too buyer.billing_details.tax_id: https://docs.gr4vy.com/reference/transactions/new-transaction [17:35:24] nevermind we don't collect it for some of the countries listed there e.g GB/US [17:37:33] ok it looks like we've never actually tested a Japanese card payment via gravy [17:37:39] This is a new thing [17:38:25] it's not on our test links docs that we've shared previously https://docs.google.com/spreadsheets/d/18qmz3GaPp8SA8U0vk-uftZsRBaoichWPn9JkL1uq5lA/edit?gid=0#gid=0 [17:39:30] XenoRyet: anilk do you know if we've committed to offering support for Japan via gravy? cstone damilare the issue might just be that it should be going to Adyen [17:39:40] I guess we did cut over for all cards [17:39:51] and this one has been missed during testing [17:40:31] yea we don't collect tax id on our direct Adyen connection [17:43:05] so let's see if this is something we can switch off [17:43:50] I guess if the campaign is going live tonight, this is another high-priority one [17:48:39] (03PS7) 10Ejegg: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) [17:48:39] (03PS8) 10Ejegg: Donor Portal login page [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1141917 (https://phabricator.wikimedia.org/T388739) [17:49:09] ok cstone I figured it out - there needs to be a and it's on the submit event that we get the validation to trigger [17:49:13] yh we need to take JP and any other non-fiscal number required countries off the list and run a test in Japan. The time in between is too short, wonder if we can delay the campaign from going live [17:49:38] jgleeson: ^^ [17:52:59] I guess "tonight" means US tonight right? [17:53:11] so we might have time, although it's not ideal [17:53:22] I was just testing it locally [17:58:46] (03PS1) 10Jgleeson: Remove JP from countries which require a fiscal number [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144626 (https://phabricator.wikimedia.org/T393939) [17:59:15] ok that change removes JP from the list of countries requiring a fiscal number cstone damilare ^ [17:59:35] thanks I'd check it out [18:03:18] (03CR) 10Damilare Adedoyin: [C:03+2] Remove JP from countries which require a fiscal number [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144626 (https://phabricator.wikimedia.org/T393939) (owner: 10Jgleeson) [18:04:00] (03Merged) 10jenkins-bot: Remove JP from countries which require a fiscal number [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1144626 (https://phabricator.wikimedia.org/T393939) (owner: 10Jgleeson) [18:04:05] thanks damilare [18:04:12] np [18:04:39] nice damilare jgleeson [18:08:08] damilare: re:where did I get that list. I'd like to imagine it was the gravy docs but I honestly can't remember. I recall looking up some python manual page about countries that need national identifiers also [18:08:52] np, the list actually coincides with whats on the doc I shared earlier [18:09:11] I think we removed some countries during CR but might have missed JP [18:10:16] ok I'll deploy that. thanks damilare and cstone for the quick reactions on this one. I guess this also highlights that we need to add a few more countries to that testing document we gave Rakhi. [18:10:26] was it from the dlocal docs maybe jgleeson ? [18:10:32] yep [18:11:01] possibly ejegg, that commit message suggests my head was in a very dlocal space at the time [18:11:31] here's the patch https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1115372 [18:11:59] oh look at that, in the commit message [18:12:01] "This could fail if we run into processors with different [18:12:01] country field requirements, but for now, it should let us [18:12:01] progress." [18:12:08] ha [18:25:55] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Null backend_processor for gravy records in civi - https://phabricator.wikimedia.org/T393477#10813410 (10Ejegg) Looks like the SmashPig code to send donations to the queue from IPNs isn't setting the back... [18:26:03] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1144629 [18:27:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Null backend_processor for gravy records in civi - https://phabricator.wikimedia.org/T393477#10813412 (10jgleeson) @Ejegg, I was digging into 3D Secure as a possible source of the problem. [18:29:51] ejegg: I noticed that pretty much all the random samples I was looking at went through 3D secure [18:31:04] unrelated I did a 3DS transaction yesterday and it made a noise that jumpscared me [18:32:23] hmm I just had to reconcile a bunch of conflicts after merging the latest from smashpig into master. cstone damilare|brb I see this patch https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1140221/3/PaymentProviders/Gravy/Validators/PaymentProviderValidator.php removes some methods, did you put through a test payment to confirm nothing breaks with change ? :) just checking before I push it out ha [18:32:25] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Null backend_processor for gravy records in civi - https://phabricator.wikimedia.org/T393477#10813423 (10Ejegg) jgleeson Ahh, that could fit. When something goes through 3d secure, it's more likely that w... [18:33:12] merging smashpig master into deployment* [18:33:33] jgleeson and damilare|brb: do we still need to be making those status requests with every IPN? Or are they sending us all the details in the IPN now? [18:33:35] jgleeson: I merged those I didn't test but saw that the IDE did say they were unused and checked that [18:35:32] cool thanks [18:35:45] ejegg: status requests? [18:36:20] we seem to make an API request out to gravy to get the latest payment status [18:37:08] https://phabricator.wikimedia.org/diffusion/WFSP/browse/master/PaymentProviders/Gravy/Actions/TransactionAction.php$65 [18:40:12] (03PS1) 10Jgleeson: Update smashpig to v0.9.6.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1144630 (https://phabricator.wikimedia.org/T393939) [18:41:05] (03CR) 10Jgleeson: [C:03+2] Update smashpig to v0.9.6.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1144630 (https://phabricator.wikimedia.org/T393939) (owner: 10Jgleeson) [18:41:50] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1144632 [18:42:30] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1144632 (owner: 10Jgleeson) [18:51:04] checking ejegg [18:52:12] (03CR) 10Cstone: [C:03+2] "thanks for the changes, looks good!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [18:54:41] I'm not sure ejegg. I think originally we were under the impression they would be sending us "light" IPNs right? and we'd have to hit them to get the latest info. I remember us discussing that at last years offsite [18:56:34] (03Merged) 10jenkins-bot: Update smashpig to v0.9.6.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1144630 (https://phabricator.wikimedia.org/T393939) (owner: 10Jgleeson) [18:56:35] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1144632 (owner: 10Jgleeson) [18:56:36] jgleeson: ejegg I'm not sure if they've changed the ipn to the full load, we don't seem to log the received message. I'd test locally [18:56:37] (03Merged) 10jenkins-bot: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [18:57:56] looks like the docs have been updated to show that the transaction payload now comes with the status: https://docs.gr4vy.com/guides/features/webhooks/payload#webhook-payload [19:00:53] thanks! I'm still trying to trace the full voyage of txn e1b45053-7f95-48c4-* through the IPN to the donations queue consumer [19:01:48] (03PS1) 10Jgleeson: Update composer lock [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1144640 [19:03:42] (03CR) 10Jgleeson: [C:03+2] Update composer lock [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1144640 (owner: 10Jgleeson) [19:05:08] ok I see, lemme know if anything is confusing. Also this is the trxn event I just got locally: https://etherpad.wikimedia.org/p/gravy-trxn-event-payload showing the status in the payload ejegg [19:05:19] I'd create a phab for the update [19:09:44] it feels odd to tell stakeholders we are in a queue on our CI server [19:11:37] we just waited 20 minutes for a bunch of selenium tests to pass on a future audiences patch on gate-and-submit before a composer bump could be merged [19:12:10] realities of the commune! :) [19:12:34] its always when you want something to merge fast [19:13:05] for payments, we get a priority queue, gate-and-submit-fundraising [19:13:29] but for mediawiki extensions, they go through the economy queue [19:13:41] should we ask for that to be changed? [19:14:03] payments=mediawiki core for anyone reading [19:15:02] (03Merged) 10jenkins-bot: Update composer lock [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1144640 (owner: 10Jgleeson) [19:15:06] yay [19:18:28] ejegg: do you remember why it was decided to only pushing core through the gate-and-submit-fundraising queue? [19:18:36] push* [19:20:35] !log payments-wiki upgraded from fac09775 to 92a8cbb8 [19:20:36] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:21:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.45-notes (1.45.0-wmf.1; 2025-05-13), 13Patch-For-Review: Japan credit card processing error - https://phabricator.wikimedia.org/T393939#10813648 (10jgleeson) [19:24:59] jgleeson: not sure, but we can put up gerrit changes for that in this repo: https://phabricator.wikimedia.org/source/integration-config/\ [19:25:10] oops that \ was a typo [19:27:22] looks like this trigger: https://phabricator.wikimedia.org/source/integration-config/browse/master/zuul/layout.yaml$886 [19:27:30] just got to find out how to make that match the other repos [19:31:36] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Update SmashPig Gravy listener to use the new transaction payload - https://phabricator.wikimedia.org/T393958 (10Damilare) 03NEW [19:32:32] jgleeson: ejegg anilk midsprint [19:33:02] loading page... [19:38:31] 06Fundraising-Backlog, 10fundraising-tech-ops, 06Fundraising-Tech-Roadmap: FR-Tech FY2425Q4 maintenance window - https://phabricator.wikimedia.org/T337585#10813705 (10greg) [19:41:49] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Temporarily turn off AVS checks while Japan campaigns are in progress - https://phabricator.wikimedia.org/T393930#10813720 (10XenoRyet) p:05Triage→03High [19:43:51] 14Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Fidelity Mapping Import - Change in gift framework - https://phabricator.wikimedia.org/T386017#10813727 (10MDemosWMF) @Eileenmcnaughton Hey! @Elbar53 and I ran a real fidelity import file. We noticed that for anonymous records the import shows a blank... [19:56:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Update SmashPig Gravy listener to use the new transaction payload - https://phabricator.wikimedia.org/T393958#10813778 (10XenoRyet) p:05Triage→03Medium [19:57:08] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: India failed transactions - https://phabricator.wikimedia.org/T393553#10813779 (10XenoRyet) p:05Triage→03Medium [19:57:55] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Refund shows as contribution line without original contribution line - https://phabricator.wikimedia.org/T393036#10813782 (10XenoRyet) p:05Triage→03Medium [19:58:32] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Ignore fraud check for recurring trxns - https://phabricator.wikimedia.org/T393040#10813785 (10XenoRyet) p:05Triage→03Medium [19:58:58] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: clean up yaml files for gravy - https://phabricator.wikimedia.org/T392855#10813788 (10XenoRyet) p:05Triage→03Medium [20:00:16] 10fundraising-tech-ops, 10Mail: DMarc Email Address for Wikimedia.org - https://phabricator.wikimedia.org/T316899#10813789 (10Jgreen) 05Open→03Declined Requesting user is no longer with WMF [20:03:52] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.45-notes (1.45.0-wmf.1; 2025-05-13): Japan credit card processing error - https://phabricator.wikimedia.org/T393939#10813812 (10AKanji-WMF) p:05Triage→03High [20:04:29] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Undefined array key "pagoefectivo" - https://phabricator.wikimedia.org/T393696#10813815 (10AKanji-WMF) p:05Triage→03Medium [20:30:59] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10fundraising-tech-ops: Several Silverpop failmail updates this week - https://phabricator.wikimedia.org/T393830#10813874 (10AKanji-WMF) @nisrael I will slack you in the short term to discuss [20:35:58] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10fundraising-tech-ops: Several Silverpop failmail updates this week - https://phabricator.wikimedia.org/T393830#10813891 (10Dwisehaupt) I have added @LLugo-WMF @ppenloglou @RLopez-WMF to the `fr-tech-acoustic-failmail` google group. [21:08:20] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 07Regression: CentralNotice campaign settings changes not showing in logs - https://phabricator.wikimedia.org/T393482#10813990 (10Ejegg) Thanks for doing the query @jeremyb. On my dev machine I've been unable to r... [22:04:40] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10fundraising-tech-ops: Several Silverpop failmail updates this week - https://phabricator.wikimedia.org/T393830#10814235 (10ppenloglou) Appreciate that @Dwisehaupt ! [22:27:42] (03PS2) 10Eileen: Update mappings in UserJob mgd files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1141997 [22:27:46] (03PS3) 10Eileen: Fixes for 6.2 import mapping changes [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1141571 [22:27:50] (03PS2) 10Eileen: Reapply our hack Temporary fix for handling duplicate contacts on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1141570 (https://phabricator.wikimedia.org/T374063) [22:27:56] (03PS4) 10Eileen: 6.3 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1140254