[00:08:43] 10fundraising-tech-ops, 10FR-Tech-Analytics: Determine and deploy retention for unsampled FR banner logs - https://phabricator.wikimedia.org/T412743#11466847 (10Dwisehaupt) [00:11:23] (03Merged) 10jenkins-bot: Use original payment_instrument_id for extraneous chargeback [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218850 (https://phabricator.wikimedia.org/T411547) (owner: 10Eileen) [00:11:25] (03Merged) 10jenkins-bot: Email fixes - formatting, correct 'is_endowment' [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218852 (https://phabricator.wikimedia.org/T411547) (owner: 10Eileen) [00:11:26] (03Merged) 10jenkins-bot: Add incomplete batches to output [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218856 (https://phabricator.wikimedia.org/T411547) (owner: 10Eileen) [00:19:45] 10fundraising-tech-ops, 10FR-Tech-Analytics: Determine and deploy retention for unsampled FR banner logs - https://phabricator.wikimedia.org/T412743#11466855 (10Dwisehaupt) unsampled logs removed from frban1002 reclaiming 27% of the disk space on /srv. [00:22:36] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218859 [00:22:48] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218859 (owner: 10Eileen) [00:23:41] (03Merged) 10jenkins-bot: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218859 (owner: 10Eileen) [00:24:26] 10fundraising-tech-ops, 10FR-Tech-Analytics: Determine and deploy retention for unsampled FR banner logs - https://phabricator.wikimedia.org/T412743#11466876 (10Dwisehaupt) [00:25:38] 10fundraising-tech-ops, 10FR-Tech-Analytics: Determine and deploy retention for unsampled FR banner logs - https://phabricator.wikimedia.org/T412743#11466877 (10Dwisehaupt) 05Open→03Resolved logs captured. old unsampled copy purged from frban1002. only unsampled logs retained on frbackup2002. [00:27:35] !log civicrm upgraded from 000ff848 to a0d1f1f7 [00:27:37] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [00:47:52] 10fundraising-tech-ops: Fundraising access request for Neil Shah-Quinn - https://phabricator.wikimedia.org/T412775#11466927 (10Dwisehaupt) [00:54:58] (03PS1) 10Eileen: Revised endowment vs annual fund format in email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218861 (https://phabricator.wikimedia.org/T411547) [00:59:14] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Additional adyen transaction type in audit... Refunded Reversed - https://phabricator.wikimedia.org/T412880 (10Eileenmcnaughton) 03NEW [01:03:51] the refund was reversed? [01:12:54] (03PS1) 10Eileen: Bubble up refunded reversed transactions [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218863 (https://phabricator.wikimedia.org/T412880) [01:13:09] greg-g: yeah - sigh [01:13:19] hah [01:13:32] so I already wrote code for when the donation is refunded AND charged back [01:13:43] and then I had to write code for when that charge back is reversed [01:13:46] and now .... [01:27:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Venmo button missing on desktop Google Chrome and Safari? - https://phabricator.wikimedia.org/T410306#11466949 (10KHancock99) Thank you @Ejegg and @jgleeson for digging deeper on this one. We will keep you posted. Great advice too,... [02:18:16] (03CR) 10Ejegg: Allow logging of raw data (032 comments) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218824 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:20:41] ejegg: do you want to push in what you have on ^^ cos I think you have it locally ? [02:25:58] oh sure [02:27:40] (03PS3) 10Ejegg: Allow logging of raw data [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218824 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:29:06] (03CR) 10Eileen: [C:03+2] Allow logging of raw data [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218824 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:29:24] eileen: something tells me we'll end up draining most of that $ignoredTypes array by this time next year :S [02:29:35] (03Merged) 10jenkins-bot: Allow logging of raw data [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218824 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:29:36] lol looking like it ... [02:30:36] any bets on when we get our first bankinstructionreturned ? [02:30:51] (03PS2) 10Eileen: Bubble up refunded reversed transactions [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218863 (https://phabricator.wikimedia.org/T412880) [02:30:55] (03CR) 10Ejegg: [C:03+2] Bubble up refunded reversed transactions [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218863 (https://phabricator.wikimedia.org/T412880) (owner: 10Eileen) [02:31:24] (03Merged) 10jenkins-bot: Bubble up refunded reversed transactions [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218863 (https://phabricator.wikimedia.org/T412880) (owner: 10Eileen) [02:31:47] lol - tomorrow probably [02:31:54] (03PS4) 10Eileen: Add extra hoped for fields to disputes-specific graphql [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218826 (https://phabricator.wikimedia.org/T411136) [02:32:26] does your name patch replace mine or build on it? [02:34:17] just another PS on yours [02:34:55] (03CR) 10Ejegg: [C:03+2] Add extra hoped for fields to disputes-specific graphql [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218826 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:35:10] oh the lastName is supposed to be a replacement [02:35:17] but I didn't get the normalization working [02:35:23] (03Merged) 10jenkins-bot: Add extra hoped for fields to disputes-specific graphql [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218826 (https://phabricator.wikimedia.org/T411136) (owner: 10Eileen) [02:36:21] i guess one of us should redo that normalization fn to be less confusing [02:53:03] hmm, and we can probably do bigger pages than 50 [03:06:31] eileen: I'm attempting to do that normalization rewrite [03:07:05] though now i wonder why we would do it in the maintenance script and not in the 'provider' class [03:11:54] ejegg: yeah - we discussed that at tech talk & agreed it is would be better not in the script [03:20:18] oops, I'll need to pick this up again in the morning [03:31:48] (03PS1) 10Cstone: Add activites for first and second recurring failure email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218867 (https://phabricator.wikimedia.org/T365488) [03:53:35] cstone: we didn't already have an activity type? [03:53:51] - for the first I mean [03:58:25] no its a type email with subject Recur failure [04:00:32] "Recur fail message" [04:12:14] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Duplicate transaction not settling correctly (also should probably reverse it) - https://phabricator.wikimedia.org/T412888 (10Eileenmcnaughton) 03NEW [04:12:43] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Duplicate transaction not settling correctly (also should probably reverse it) - https://phabricator.wikimedia.org/T412888#11467161 (10Eileenmcnaughton) The ones I found SELECT count(*) as c, backend_processor_txn_id FROM wmf_contributi... [04:16:07] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Duplicate transaction not settling correctly (also should probably reverse it) - https://phabricator.wikimedia.org/T412888#11467162 (10Eileenmcnaughton) [04:16:25] (03CR) 10Eileen: [C:03+2] Add activites for first and second recurring failure email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218867 (https://phabricator.wikimedia.org/T365488) (owner: 10Cstone) [04:17:10] (03CR) 10Cstone: [C:03+2] Revised endowment vs annual fund format in email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218861 (https://phabricator.wikimedia.org/T411547) (owner: 10Eileen) [04:32:48] 06Fundraising-Backlog: Gravy duplicate donation - https://phabricator.wikimedia.org/T412328#11467176 (10Cstone) [04:33:33] (03Merged) 10jenkins-bot: Add activites for first and second recurring failure email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218867 (https://phabricator.wikimedia.org/T365488) (owner: 10Cstone) [04:33:35] (03Merged) 10jenkins-bot: Revised endowment vs annual fund format in email [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218861 (https://phabricator.wikimedia.org/T411547) (owner: 10Eileen) [04:45:00] 06Fundraising-Backlog: Gravy duplicate donation - https://phabricator.wikimedia.org/T412328#11467197 (10Cstone) [04:46:34] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Duplicate transaction not settling correctly (also should probably reverse it) - https://phabricator.wikimedia.org/T412888#11467200 (10Eileenmcnaughton) 6 of the above were templates (cleared) & I cleared the original one manually New q... [05:12:19] (03PS1) 10Eileen: Increase log level for alert [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218871 [05:17:55] (03CR) 10Cstone: [C:03+2] Increase log level for alert [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218871 (owner: 10Eileen) [05:34:45] (03Merged) 10jenkins-bot: Increase log level for alert [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1218871 (owner: 10Eileen) [05:35:36] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218874 [05:35:51] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218874 (owner: 10Eileen) [05:36:44] (03Merged) 10jenkins-bot: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1218874 (owner: 10Eileen) [05:42:35] !log civicrm upgraded from a0d1f1f7 to 28ef5eb1 [05:42:36] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [06:54:48] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 05MW-1.46-notes (1.46.0-wmf.7; 2025-12-16): Duplicate donations made via Apple Pay and Google Pay from the WP app - https://phabricator.wikimedia.org/T407383#11467430 (10KHancock99) Hey @Cstone, we have another donor occurrence of... [13:47:40] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: utm weirdness - different values in contribution tracking, on the donation message, and in the logs - https://phabricator.wikimedia.org/T412307#11468632 (10Damilare) a:03Damilare [14:40:28] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing: Share address and email between DAF and DAF owner when importing donations - https://phabricator.wikimedia.org/T411639#11468845 (10RLewis) @MDemosWMF - yes i think we definitely need a way to be ab... [14:41:18] 03Fundraising Sprint: Yabba Dabba Stew, 06Fundraising-Backlog, 07fr-current-sprint, 13Patch-For-Review: Add in tracking (and graphs) for how long api calls to the payment processors take - https://phabricator.wikimedia.org/T411673#11468853 (10jgleeson) a:03jgleeson [14:47:43] fr-tech I'm gonna take a look at building on ejegg's patch to get some timings on the various API calls we make and then look at how best to share them with dallas and jeff so we can graph them [14:48:22] I'll see if I can get this pass on CI to start https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1217280 [14:52:47] hi jgleeson [14:52:57] we can probably just take out the typehint [14:55:17] oh interesting, 'readonly' properties need to be typed [14:55:59] ahh I see it's not that typehint [14:56:23] ok, so you might mock the wrapper class [15:20:51] (03PS1) 10Ejegg: Add elapsed time to CurlWrapper return array [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) [15:24:43] well, ^^^ gets it one level up the call chain [15:44:34] (03PS1) 10Ejegg: Add API call timings to Adyen calls [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219175 (https://phabricator.wikimedia.org/T411673) [15:46:25] Jeff_Green: can you suggest a good format for a log line that your scraper can use to get the elapsed time and function name ? [15:48:08] our provisional patch has something like this: [15:48:08] 2025-12-12T18:13:31+00:00 ce814dc84add SmashPig-Gravy-GetTransactionDet SPCID-0050814241 | (APITimings) fn: getTransaction, elapsed: 0.34401106834412 s | | [15:48:31] so your scraper would look for all lines including APITimings [15:48:54] then get the provider name from the word after SmashPig- [15:49:05] the function name from the word after fn: [15:49:17] and the timing from the numeric string after elapsed: [15:49:18] the scraper basically uses PCRE to parse predictable line formats, and can collect a numeric value from the line i.e. to sum a bunch of values from the scrape window, or it can just count them [15:49:33] k, I think we'd want an average here [15:50:00] so i.e. average of values for a 1 minute scrape window is good? [15:50:14] yep [15:50:26] it might already do averages, I can't remember [15:50:31] it'd be an average for each combination of provider+function [15:50:31] easy to add if not [15:51:10] I have to look re. text capture to prometheus label, I think it already does that but I'm not positive [15:53:15] it handles one label now, so if we can combine provider+function into one string that would work now [15:53:49] is it important to have those as separate labels in prometheus, or is a combined label workable? [15:59:42] Jeff_Green: I think combined is fine [16:00:05] should still be easy enough to check e.g. AdyenCapture vs GravyCapture [16:00:30] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Segmentation Cross-Check for Primary and Secondary Emails - https://phabricator.wikimedia.org/T412864#11469395 (10SHust) I had a feeling this was on the Acoustic side—thanks for the explanation and... [16:00:41] and a lot of the fn calls will be pretty unique to each provider [16:01:20] jgleeson: I had a skeletal version of the non-gravy timings patch kicking around locally so I just got it passing tests and pushed it up: https://gerrit.wikimedia.org/r/1219168 [16:01:37] then added a way to use it for Adyen here: https://gerrit.wikimedia.org/r/1219175 [16:02:07] but we would want to add those ->setLogTag calls to the other processors too [16:02:13] and smoke test of course [16:05:47] ejegg: it would be good to have something unique/common at the beginning of the message, so we can set the scraper to look for that line and know to expect the label and value [16:07:07] also the scraper only deals with one label/value per log line, as long as we're just averaging that elapsed number from the end it should be good [16:08:36] I was actually going to look at that script today to try to fix a case where it's unable to deserialize a log line, so it's good timing to hack in the "average" feature [16:15:48] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Segmentation Cross-Check for Primary and Secondary Emails - https://phabricator.wikimedia.org/T412864#11469506 (10Lars) [16:25:30] 03Fundraising Sprint: Yabba Dabba Stew, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 06Product Safety and Integrity, and 2 others: [Temporary Accounts] Update CentralNotice extension to support Temporary Accounts - https://phabricator.wikimedia.org/T374437#11469575 (10Ejegg) Hi again @Nihar... [16:29:56] 14Fundraising Sprint: Vixen, 03Fundraising Sprint: Yabba Dabba Stew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 3 others: Implement recurring donation monthly-to-annual conversion - https://phabricator.wikimedia.org/T388753#11469608 (10Ejegg) a:05AnnWF→03AKanji-WMF @AKanji-WMF you re-o... [16:34:26] 10Fundraising analytics stack: Select approach for Trino to Sage integration - https://phabricator.wikimedia.org/T388711#11469634 (10AStein-WMF) closing since this will be handled by the civi folks [16:34:34] 10Fundraising analytics stack: Select approach for Trino to Sage integration - https://phabricator.wikimedia.org/T388711#11469635 (10AStein-WMF) 05Stalled→03Declined a:05AStein-WMF→03None [16:36:02] thanks ejegg I'll take a look and also think about a good way to log/track it [16:39:14] feels like we a need a convention tracking processor, sub-processor maybe, action, stage, trxn_id. Something along the lines the [gravy|cc|auth|request] [16:39:26] of* [16:39:49] 10Fundraising analytics stack: Bring Wikipedia traffic data into the data lakehouse - https://phabricator.wikimedia.org/T399893#11469645 (10AStein-WMF) closing - dupe of https://phabricator.wikimedia.org/T252049 [16:39:51] 10Fundraising analytics stack: Bring Wikipedia traffic data into the data lakehouse - https://phabricator.wikimedia.org/T399893#11469650 (10AStein-WMF) 05Open→03Declined a:05AStein-WMF→03None [16:40:36] [gravy|cc|auth|request|time] [16:40:46] that way we don't have to worry about measuring gaps [16:41:04] similar to that stuff you're already doing in that patch from last week [16:41:23] hmm [16:41:30] [gravy|cc|auth|request|attempt|2] [16:42:14] something like that might help monitor rejections / errors [16:43:30] having a format like that makes Jeff_Green and Dallas ' life easier too [16:44:01] so for direct we could have [adyen|cc|auth|request|time] [16:44:07] 14Fundraising Sprint: Vixen, 03Fundraising Sprint: Yabba Dabba Stew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 3 others: Implement recurring donation monthly-to-annual conversion - https://phabricator.wikimedia.org/T388753#11469666 (10AKanji-WMF) Thanks @Ejegg I think I was not seeing in... [16:44:10] and for gravy maybe [gravy|adyen|cc|auth|request|time] [16:44:22] jgleeson: that makes sense [16:44:36] what service/consumer will write these? [16:44:42] payments [16:44:51] hmm [16:45:15] I guess we could log this data from payments & civi but it probably makes sense to start with payments, the entrypoint [16:45:31] ok [16:46:09] wow [16:46:27] we could capture all different types of events with that structure [16:46:36] and monitor averages [16:47:30] there are actually two scrapers that could be used [16:48:16] there's the simple syslog_collector which is the one we've been talking about, but there's also logged_payments_collector which is the one that deserializes yaml/json/etc and does more complicated stuff [16:48:38] nice [16:48:52] I ~think~ this is a case for syslog_collector, but not positive [16:50:40] yeah, that makes sense to me, keeping it simple for now. just getting the timing averages of these events and graphing them is a big step in the right direction. It's tempting to add additional context, definitely in the non-happy-bath scenarios, but it could get complicated quickly [16:50:56] path! [16:51:00] bath ha [16:51:27] flashbacks of the morning routine with the kids [16:59:08] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Skipping refunds for contributions still in Donations Queue - https://phabricator.wikimedia.org/T412984 (10Damilare) 03NEW [17:00:14] 10Fundraising analytics stack: Connect Trino to banner data (Kafka topic) - https://phabricator.wikimedia.org/T377625#11469742 (10AStein-WMF) @IAckerman-WMF do we want to compare the output of the two pipelines in the description? I don't know if there are still discrepancies between the two. On the other hand w... [17:16:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy duplicate donation - https://phabricator.wikimedia.org/T412328#11469829 (10Cstone) [18:01:14] PROBLEM - check_mysql on frdb2005 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 1680 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2005&service=check_mysql [18:11:12] RECOVERY - check_mysql on frdb2005 is OK: Uptime: 2935773 Threads: 4 Questions: 593289552 Slow queries: 1698 Opens: 411724 Open tables: 1538 Queries per second avg: 202.089 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2005&service=check_mysql [18:42:40] 10fundraising-tech-ops: Fundraising access request for Neil Shah-Quinn - https://phabricator.wikimedia.org/T412775#11470161 (10Dwisehaupt) `Date: Tue, 16 Dec 2025 20:18:49 From: Lisa Seitz Gruwell To: Dallas Wisehaupt Cc: Neil Shah-Quinn, Omari Sefu Subject: Re: Access request for Neil Shah-Quinn ---------------... [18:42:51] 10fundraising-tech-ops: Fundraising access request for Neil Shah-Quinn - https://phabricator.wikimedia.org/T412775#11470164 (10Dwisehaupt) [18:57:38] (03PS1) 10Ejegg: Drop cancel IPNs if already cancelled [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219200 (https://phabricator.wikimedia.org/T412539) [19:02:50] (03PS1) 10Lars SG: Don't resnooze if snooze date is unchanged. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219203 (https://phabricator.wikimedia.org/T392726) [19:04:41] 10Fundraising analytics stack: Connect Trino to banner data (Kafka topic) - https://phabricator.wikimedia.org/T377625#11470282 (10AStein-WMF) @Ottomata is the eventlogging dataset still relevant, or is there a replacement? I see this phab and that the wikitech page is archived: https://phabricator.wikimedia.org/... [19:04:59] (03CR) 10Lars SG: Don't resnooze if snooze date is unchanged. (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219203 (https://phabricator.wikimedia.org/T392726) (owner: 10Lars SG) [19:10:10] (03PS2) 10Jgleeson: Add Gravy SDK wrapper for API timings [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1217280 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:12:13] ejegg: that gets the gravy one passing on CI ^ [19:12:23] I'll take a peek at the other chain [19:16:35] 10fundraising-tech-ops: Fundraising access request for Neil Shah-Quinn - https://phabricator.wikimedia.org/T412775#11470303 (10AStein-WMF) [19:26:33] 10fundraising-tech-ops: Fundraising access request for Neil Shah-Quinn - https://phabricator.wikimedia.org/T412775#11470330 (10AStein-WMF) I've generated the cert and sent the pw to Neil over whatsapp! [19:30:03] ah right, we don't want to always mock the wrapper because it's doing some error handling [19:30:48] (03PS3) 10Jgleeson: Add Gravy SDK wrapper for API timings [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1217280 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:31:26] (03CR) 10Ejegg: [C:03+1] "C+1 to jgleeson's fixes in PS2, happy to see it merge if someone wants to give it a C+2" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1217280 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:32:38] (03CR) 10Jgleeson: [C:03+2] "Thanks ejegg! Your stuff LGTM" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1217280 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:33:08] (03Merged) 10jenkins-bot: Add Gravy SDK wrapper for API timings [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1217280 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:34:11] (03PS1) 10Lars SG: Better error handling for bulk re opt in [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219208 (https://phabricator.wikimedia.org/T409958) [19:43:18] (03CR) 10Jgleeson: "It looks like cURL already does a bunch of timing measurements, which we can access in $curlInfo. I compared our microtime measure to $cur" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:47:42] (03CR) 10Ejegg: Add elapsed time to CurlWrapper return array (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [19:54:29] (03PS2) 10Ejegg: Add elapsed time to CurlWrapper return array [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) [19:54:29] (03PS2) 10Ejegg: Add API call timings to Adyen calls [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219175 (https://phabricator.wikimedia.org/T411673) [19:55:06] ok jgleeson I switched it over to using the curlInfo timings ^^^ [19:55:13] lemme just do some smoke testing [19:56:32] oh hmm, i'm getting Warning: Undefined array key 6291506 [20:31:47] (03CR) 10Jgleeson: Add elapsed time to CurlWrapper return array (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) (owner: 10Ejegg) [20:44:50] (03PS4) 10Eileen: Fix financial txn rows on contribution convert [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1193439 (https://phabricator.wikimedia.org/T405765) (owner: 10Ejegg) [20:44:50] (03PS1) 10Eileen: Fix changeCurrency [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219218 (https://phabricator.wikimedia.org/T405765) [21:02:34] (03PS3) 10Ejegg: Add elapsed time to CurlWrapper return array [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219168 (https://phabricator.wikimedia.org/T411673) [21:02:34] (03PS3) 10Ejegg: Add API call timings to Adyen calls [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219175 (https://phabricator.wikimedia.org/T411673) [21:03:37] ok, there's the code I thought I had committed earlier ^^^ [21:03:47] (plus fixing to use the string constant) [21:13:56] (03PS1) 10Lars SG: Prevent opting out of emails when shared. [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1219221 (https://phabricator.wikimedia.org/T409958) [21:16:17] (03CR) 10Lars SG: Prevent opting out of emails when shared. (031 comment) [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1219221 (https://phabricator.wikimedia.org/T409958) (owner: 10Lars SG) [21:17:58] eileen: ^^ is what we discussed yesterday about secondary emails and opting out [21:27:11] thanks ejegg I'll pick up on those tomorrow [21:27:14] bye for now [22:12:05] 14Fundraising Sprint: Vixen, 03Fundraising Sprint: Yabba Dabba Stew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 3 others: Implement recurring donation monthly-to-annual conversion - https://phabricator.wikimedia.org/T388753#11470904 (10CHudson-WMF) Yes, I think so! Quick question, though,... [22:12:39] (03Abandoned) 10Eileen: Wouldn't we need to do this to get venmo lastName? [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1218829 (owner: 10Eileen) [22:21:49] hmm - I pushed up the PhoneConsent job - but I don't think I ever got a 'verbal' +2 & deployed & it doesn't seem to be there [22:21:49] Sean messaged me [22:21:49] "Hi Eileen, just want to confirm here that Caitlin and I did some query testing in Acoustic yesterday to re-include users with emails, and exclude subsets of those users based on their Civi donor status + segment IDs. Our list counts all appeared to to be working as expected -- do you have any flags with us sending a broadcast on Monday the 22nd?" [22:22:16] & I *think* it's all ok as long as the job is running & it seems OK to them.... [22:23:04] anyone - first step - if someone can look at that job [22:39:52] eileen: on frpm? [22:40:37] ah i see it [22:41:30] looks good to me eileen [22:53:47] (03PS1) 10Eileen: Add function to reverse gravy UUIDs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219233 (https://phabricator.wikimedia.org/T411745) [23:00:57] (03CR) 10Eileen: [C:03+2] Better error handling for bulk re opt in [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219208 (https://phabricator.wikimedia.org/T409958) (owner: 10Lars SG) [23:01:55] thanks cstone [23:02:58] !log config revision changed from 7d6ad875 to e478c565 [23:03:00] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [23:32:20] (03PS1) 10Lars SG: Correct overwritten cancellation reasons for paypal recurrings [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219238 (https://phabricator.wikimedia.org/T412539) [23:34:59] (03CR) 10Cstone: [C:03+2] Add function to reverse gravy UUIDs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219233 (https://phabricator.wikimedia.org/T411745) (owner: 10Eileen) [23:35:29] (03Merged) 10jenkins-bot: Add function to reverse gravy UUIDs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1219233 (https://phabricator.wikimedia.org/T411745) (owner: 10Eileen) [23:41:14] (03PS1) 10Cstone: Switch failure email to new type [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219239 (https://phabricator.wikimedia.org/T365488) [23:45:24] (03PS1) 10Cstone: Upgrade smash-pig to v1.0.3 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219240 [23:45:51] (03CR) 10Cstone: [C:03+2] Upgrade smash-pig to v1.0.3 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219240 (owner: 10Cstone) [23:46:27] (03PS2) 10Lars SG: Correct overwritten cancellation reasons for paypal recurrings [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1219238 (https://phabricator.wikimedia.org/T412539)