[00:15:24] (03CR) 10CI reject: [V: 04-1] WIP: Create Fundraiseup Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959346 (https://phabricator.wikimedia.org/T346620) (owner: 10Damilare Adedoyin) [00:35:29] (03PS4) 10Ejegg: Don't use concatenated fields for new c_t table [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959763 (https://phabricator.wikimedia.org/T334904) [00:51:01] (03CR) 10CI reject: [V: 04-1] Don't use concatenated fields for new c_t table [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959763 (https://phabricator.wikimedia.org/T334904) (owner: 10Ejegg) [01:37:00] (03PS5) 10Ejegg: Don't use concatenated fields for new c_t table [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959763 (https://phabricator.wikimedia.org/T334904) [10:59:14] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: As a donor I want to be able to make an easy and convenient donation via Apple Pay in the Wikipedia app - https://phabricator.wikimedia.org/T288285 (10scblr) [11:01:28] (03PS4) 10Damilare Adedoyin: Create Fundraiseup Audit parser class in Smashpig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/958514 (https://phabricator.wikimedia.org/T346621) [11:01:38] (03PS4) 10Damilare Adedoyin: WIP: Create Fundraiseup Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959346 (https://phabricator.wikimedia.org/T346620) [11:02:35] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: As a donor I want to be able to make an easy and convenient donation via Apple Pay in the Wikipedia app - https://phabricator.wikimedia.org/T288285 (10scblr) [11:17:15] (03CR) 10CI reject: [V: 04-1] WIP: Create Fundraiseup Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959346 (https://phabricator.wikimedia.org/T346620) (owner: 10Damilare Adedoyin) [13:01:55] 10fundraising-tech-ops: Fundraising environment support for Debian 12 (Bookworm) - https://phabricator.wikimedia.org/T344290 (10Jgreen) [14:49:40] (03PS3) 10Ejegg: Improve error handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959382 (owner: 10Eileen) [14:49:48] (03CR) 10Ejegg: [C: 03+2] Improve error handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959382 (owner: 10Eileen) [15:03:52] (03PS9) 10Ejegg: Smash pig cleanup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959372 (owner: 10Eileen) [15:04:25] (03CR) 10Ejegg: [C: 03+2] "Thanks for the cleanup! Nice to use the built-in Civi test cleanup when possible." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959372 (owner: 10Eileen) [15:04:55] (03PS4) 10Ejegg: Update Smashpig processor to set Gift Data fields on repeat contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959381 (https://phabricator.wikimedia.org/T345920) (owner: 10Eileen) [15:05:22] (03Merged) 10jenkins-bot: Improve error handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959382 (owner: 10Eileen) [15:05:35] (03CR) 10Ejegg: [C: 03+2] "Looks good, works in smoke test with docker containers." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959381 (https://phabricator.wikimedia.org/T345920) (owner: 10Eileen) [15:19:28] (03Merged) 10jenkins-bot: Smash pig cleanup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959372 (owner: 10Eileen) [15:21:17] (03Merged) 10jenkins-bot: Update Smashpig processor to set Gift Data fields on repeat contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959381 (https://phabricator.wikimedia.org/T345920) (owner: 10Eileen) [15:32:50] 10fundraising-tech-ops: Fundraising environment support for Debian 12 (Bookworm) - https://phabricator.wikimedia.org/T344290 (10Jgreen) [15:37:51] (03PS3) 10Ejegg: Add missing name to matched contact [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959334 (https://phabricator.wikimedia.org/T326986) [15:54:21] (03CR) 10CI reject: [V: 04-1] Add missing name to matched contact [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959334 (https://phabricator.wikimedia.org/T326986) (owner: 10Ejegg) [17:04:18] (03PS1) 10C. Scott Ananian: Passing arguments to OutputPage::prepareErrorPage() has been deprecated [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/960107 (https://phabricator.wikimedia.org/T343994) [17:07:57] (03CR) 10CI reject: [V: 04-1] Passing arguments to OutputPage::prepareErrorPage() has been deprecated [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/960107 (https://phabricator.wikimedia.org/T343994) (owner: 10C. Scott Ananian) [17:18:35] (03PS2) 10C. Scott Ananian: Passing arguments to OutputPage::prepareErrorPage() has been deprecated [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/960107 (https://phabricator.wikimedia.org/T343994) [18:55:37] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-failmail: Civi/Silverpop Failmail Alert - https://phabricator.wikimedia.org/T347170 (10Ejegg) [19:26:22] (03CR) 10Bartosz Dziewoński: Passing arguments to OutputPage::prepareErrorPage() has been deprecated (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/960107 (https://phabricator.wikimedia.org/T343994) (owner: 10C. Scott Ananian) [19:32:17] (03PS4) 10Ejegg: Move archived campaigns to a new subaction of Special:CentralNotice [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/952489 (https://phabricator.wikimedia.org/T328424) (owner: 10Pcoombe) [19:33:33] (03CR) 10Ejegg: [C: 03+2] "Thanks! The code looks good, and it works fine in local testing. At first it wasn't totally obvious how to get back from the archived camp" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/952489 (https://phabricator.wikimedia.org/T328424) (owner: 10Pcoombe) [19:36:36] (03CR) 10Wfan: [C: 03+2] "Looks nice, thanks Dami!" [extensions/FundraiserLandingPage] - 10https://gerrit.wikimedia.org/r/959347 (https://phabricator.wikimedia.org/T346975) (owner: 10Damilare Adedoyin) [19:37:04] (03Merged) 10jenkins-bot: Move archived campaigns to a new subaction of Special:CentralNotice [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/952489 (https://phabricator.wikimedia.org/T328424) (owner: 10Pcoombe) [19:39:22] (03Merged) 10jenkins-bot: Added Fundraiseup code to DonateWiki Head. [extensions/FundraiserLandingPage] - 10https://gerrit.wikimedia.org/r/959347 (https://phabricator.wikimedia.org/T346975) (owner: 10Damilare Adedoyin) [20:02:42] (03PS4) 10Ejegg: Add missing name to matched contact [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/959334 (https://phabricator.wikimedia.org/T326986) [20:03:09] ok, ^^^ should actually work for RML -> apple pay [20:03:24] but I think we can expand that to always match on email for RML donors [20:03:53] that is, consider email alone to be enough when the stored contact has no name saved [20:04:04] I'd do that in a follow-on patch though [20:05:03] hmm, what's up with the donation queue? [20:06:50] couple of updates in the processlist [20:07:56] alright, one died due to deadlock [20:12:49] Jeff_Green: any idea what's making some DB updates run so slow? [20:13:07] normal stuff from the donations queue consumer [20:13:14] is taking > 30 seconds in some cases [20:13:30] ejegg: hmm. I haven't looked in a while, did anything change re. the query? [20:13:31] and I'm only seeing a single process in the full processlist [20:13:39] no changes today [20:14:11] the acoustic export mysteriously hung this morning too [20:15:12] let's look at slow query log [20:15:22] looks like it started abut 25 min ago [20:15:29] (the problem with the DQ consumer) [20:18:19] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-failmail: Civi/Silverpop Failmail Alert - https://phabricator.wikimedia.org/T347170 (10KHaggard) 05Open→03Resolved a:03KHaggard Just ran the Acoustic jobs and everything is coming through normally. Thanks @Ejegg ! [20:18:46] I see at least one slow user query [20:19:36] ah interesting, I can only ever see stuff from User: 1 with the latest DQ message details when I look at the processlist lately [20:20:05] there's "User : 2399 */INSERT IGNORE INTO civicrm_tmp_e_logsummary..." [20:21:12] I can't tell yet whether I'm seeing multiple similar queries or one that's sticking around... [20:21:50] well, time is 2000s so I guess there's our answer [20:22:55] ejegg: do you want the full query? [20:24:08] sure! [20:24:11] thanks Jeff_Green [20:24:20] want to kill it too please? [20:24:28] ok [20:24:42] and I wonder why I can't see it as the civi db user [20:25:03] it's a different user [20:25:10] User: civi_read [20:25:14] ahhh [20:25:39] ok killed, I'll put the query on frpm1002 [20:25:43] thank you [20:27:04] ok it's /tmp/20230922-slow_query_on_civi_db.txt [20:34:19] thanks [20:34:26] np! [20:35:38] Looks like Laurice Medley managed to make that query [20:45:59] and it seems to be the contact logging summary report [20:46:37] with no apparent where clause except log_action != 'Initialization' [20:46:54] I'll see if I can disable that report [20:46:56] on prod [20:47:10] cool [20:49:55] oh yeah, there it is in the 'All Reports' page [20:50:09] with a convenient 'View Results' link. One shot kill [20:50:17] deleting that one [20:51:24] and... gone [21:17:23] (03PS1) 10Ejegg: Move last two pagers into specials/pagers [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/960138 [21:21:38] special pagers makes me think of the t900 - http://www.pagersonline.com/pagers/t900.html [21:29:29] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Donate form API calls - https://phabricator.wikimedia.org/T345847 (10Tsevener) [21:31:58] ooh, querty keyboard? that's a fancy one [21:32:53] welp, more failmail due to ideal refunds that can't find the associated donation [21:33:07] time I checked to see that we do get IPNs for all those captures [21:34:55] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Donate form UI work - https://phabricator.wikimedia.org/T345846 (10Tsevener) @scblr I fixed a few of your lighter-weight issues (3 and 5, small change to improve the 6 situation) in the PR, but I will need to come back la... [21:35:18] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Donate form API calls - https://phabricator.wikimedia.org/T345847 (10Tsevener) a:05Tsevener→03None [21:40:14] hmmmmm nope, just auths. but in the console both auth and capture have the same psp reference for any random iDEAL one I check [21:40:23] so what are we recording here? [21:41:56] ohhh those last two were for card refunds [21:41:58] hmm [22:03:29] (03PS1) 10Ejegg: Better handling for NotificationOfChargeback [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960144 (https://phabricator.wikimedia.org/T295997) [22:03:35] (03PS4) 10Ejegg: Add manualCapture flag to Adyen auth calls [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/959247 (https://phabricator.wikimedia.org/T345736) [22:05:20] (03PS1) 10Ejegg: Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) [22:05:25] (03CR) 10Wfan: [C: 03+2] Move last two pagers into specials/pagers [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/960138 (owner: 10Ejegg) [22:05:42] (03PS1) 10Ejegg: fix logging tag for silly line [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960146 [22:05:45] (03CR) 10CI reject: [V: 04-1] Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) (owner: 10Ejegg) [22:06:11] (03CR) 10CI reject: [V: 04-1] fix logging tag for silly line [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960146 (owner: 10Ejegg) [22:12:26] (03PS2) 10Ejegg: Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) [22:12:29] (03PS2) 10Ejegg: fix logging tag for silly line [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960146 [22:13:02] (03CR) 10CI reject: [V: 04-1] Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) (owner: 10Ejegg) [22:13:13] (03CR) 10CI reject: [V: 04-1] fix logging tag for silly line [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960146 (owner: 10Ejegg) [22:14:49] (03PS1) 10Ejegg: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/960149 [22:14:53] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/960149 (owner: 10Ejegg) [22:16:06] 10Fundraising Tech - Chaos Crew, 10MediaWiki-extensions-CentralNotice, 10MW-1.41-notes (1.41.0-wmf.28; 2023-09-26): Move archived CentralNotice campaigns to a separate page - https://phabricator.wikimedia.org/T328424 (10Ejegg) a:03Pcoombe [22:16:17] (03Merged) 10jenkins-bot: Move last two pagers into specials/pagers [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/960138 (owner: 10Ejegg) [22:18:25] (03PS3) 10Ejegg: Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) [22:18:27] (03PS3) 10Ejegg: fix logging tag for silly line [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960146 [22:22:26] 10Fundraising Sprint Rasberry Pi Beret 👩🏽‍🎨, 10Fundraising-Backlog, 10FR-Smashpig, 10MediaWiki-extensions-DonationInterface: Record Adyen iDEAL donations on capture IPN, not on successful auth - https://phabricator.wikimedia.org/T345954 (10Ejegg) 05Open→03Invalid As jgleeson says, we're not getting cap... [22:27:10] (03Merged) 10jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/960149 (owner: 10Ejegg) [22:35:23] (03CR) 10Wfan: [C: 03+2] Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) (owner: 10Ejegg) [22:35:53] (03Merged) 10jenkins-bot: Fix parent transaction ID for Adyen chargebacks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960145 (https://phabricator.wikimedia.org/T345097) (owner: 10Ejegg) [22:38:04] (03CR) 10Wfan: [C: 03+1] "looks fair, maybe we want to update the function’s comment block also" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/960144 (https://phabricator.wikimedia.org/T295997) (owner: 10Ejegg)