[00:02:14] (03PS1) 10Eileen: Fix , extend Benevity gift sources [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076869 (https://phabricator.wikimedia.org/T369647) [00:23:01] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog, 13Patch-For-Review: Benevity import not working - https://phabricator.wikimedia.org/T369647#10190118 (10Eileenmcnaughton) SO this is what the new mapping looks like {F57574206} [00:24:00] cstone: ^^ should be an easy +2 - just adding / fixing a mapping. It's right according to my understanding of what Nora is asking for & if that isn't right it will make it easier to clarify [00:24:26] (03Abandoned) 10Reedy: Test [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1075260 (owner: 10Reedy) [00:24:44] capital letters ruining everything [00:24:50] (03CR) 10Cstone: [C:03+2] Fix , extend Benevity gift sources [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076869 (https://phabricator.wikimedia.org/T369647) (owner: 10Eileen) [00:28:36] cstone: took me a while to spot it - even after seeing there was a patch to fix them on the other 2 [00:44:57] (03Merged) 10jenkins-bot: Fix , extend Benevity gift sources [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076869 (https://phabricator.wikimedia.org/T369647) (owner: 10Eileen) [01:09:05] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Can we retroactively update a tracking code for this email ID? sp74475455 - https://phabricator.wikimedia.org/T375685#10190167 (10Cstone) There were 6 of them, they have been updated to wmf_medium=MGFE [02:01:00] I'mma get the mediawiki security update out to payments + email prefs center [02:03:26] oh i see, it's a maintenance release, not a security release [02:03:37] and I basically got all the stuff merged last week [02:03:52] well, I'll just pull it in anyway [02:04:07] and update email prefs center, that's a bit out of date [02:05:08] (03PS1) 10Ejegg: Merge branch 'REL1_39' into fundraising/REL1_39 [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1076890 [02:05:11] (03CR) 10Ejegg: [C:03+2] Merge branch 'REL1_39' into fundraising/REL1_39 [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1076890 (owner: 10Ejegg) [02:14:43] (03Merged) 10jenkins-bot: Merge branch 'REL1_39' into fundraising/REL1_39 [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1076890 (owner: 10Ejegg) [02:16:14] !log payments-wiki upgraded from 8d3b8e94 to e88750e6 [02:16:18] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [02:17:35] !log email preference center upgraded from 8ff002ef to e88750e6 [02:17:39] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [02:25:08] (03PS6) 10Ejegg: composer: Install phan locally [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/637020 (owner: 10Jforrester) [02:25:28] (03CR) 10Ejegg: [C:03+2] composer: Install phan locally [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/637020 (owner: 10Jforrester) [02:26:12] 06Fundraising-Backlog: Dlocal timeouts - https://phabricator.wikimedia.org/T376095 (10Cstone) 03NEW [02:26:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Dlocal timeouts - https://phabricator.wikimedia.org/T376095#10190244 (10Cstone) a:03Cstone [02:26:49] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1076891 [02:27:10] (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/1076891 (owner: 10Eileen) [02:27:11] (03CR) 10Ejegg: "suggestion for different error code" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1058684 (owner: 10Damilare Adedoyin) [02:27:59] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1076891 (owner: 10Eileen) [02:28:27] (03Merged) 10jenkins-bot: composer: Install phan locally [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/637020 (owner: 10Jforrester) [02:30:58] (03CR) 10Ejegg: [C:03+2] "Looks good in local testing, code changes look fine. Thanks!" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1075966 (https://phabricator.wikimedia.org/T367189) (owner: 10Matr1x) [02:34:33] (03Merged) 10jenkins-bot: Convert styles to use LESS and make them dark-mode comptatible [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1075966 (https://phabricator.wikimedia.org/T367189) (owner: 10Matr1x) [02:34:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: Add backend processor to gravy refunds - https://phabricator.wikimedia.org/T374634#10190264 (10Cstone) a:03Cstone [02:47:01] !log civicrm upgraded from cf27c789 to 28fd5e3b [02:47:05] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [02:49:52] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: 'Expected field payments_final_status bound for table payments_initial' failmails - https://phabricator.wikimedia.org/T374132#10190284 (10Cstone) a:03Cstone [02:50:13] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog: Benevity import not working - https://phabricator.wikimedia.org/T369647#10190287 (10Eileenmcnaughton) @NNichols - try now [03:06:35] 03Fundraising Sprint: Sssss, 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Duplicate contributions after import - https://phabricator.wikimedia.org/T376068#10190309 (10Eileenmcnaughton) I've done some more pondering about this and the reason this does not happen all the time is the `Queue.claimItem... [03:07:05] 03Fundraising Sprint: Sssss, 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Duplicate contributions after import - https://phabricator.wikimedia.org/T376068#10190310 (10Eileenmcnaughton) p:05High→03Medium [03:07:51] 03Fundraising Sprint: Sssss, 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Duplicate contributions after import - https://phabricator.wikimedia.org/T376068#10190311 (10Eileenmcnaughton) I've triage the task back down to medium since I think monitoring is the best course of action for now - if it ha... [03:12:26] 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Non Campaign Emails in Database - https://phabricator.wikimedia.org/T361621#10190316 (10Eileenmcnaughton) @bsisolak looking at the code & docs and your comments we do the things you say but our requests do not pass the optional campaign ta... [03:24:12] (03PS1) 10Eileen: Minor tidy up in config function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076895 (https://phabricator.wikimedia.org/T374063) [03:24:14] (03PS1) 10Eileen: [untested] Add campaign statuses to the flags for Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076896 (https://phabricator.wikimedia.org/T361621) [03:24:32] (03PS2) 10Eileen: [untested] Add campaign statuses to the flags for Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076896 (https://phabricator.wikimedia.org/T361621) [03:25:18] (03CR) 10CI reject: [V:04-1] Minor tidy up in config function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076895 (https://phabricator.wikimedia.org/T374063) (owner: 10Eileen) [03:26:07] 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Non Campaign Emails in Database - https://phabricator.wikimedia.org/T361621#10190333 (10Eileenmcnaughton) @AKanji-WMF this doesn't need to be tied to the outage - it might be as simple as this - https://gerrit.wikimedia.org/r/c/wikimedia/f... [04:52:02] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Switch Acoustic mapping to use the API to define the columns - https://phabricator.wikimedia.org/T368470#10190417 (10bsisolak) I took you files and ran a job in Postman, and it was successful. The only thing I updated was th... [04:54:51] 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Non Campaign Emails in Database - https://phabricator.wikimedia.org/T361621#10190418 (10bsisolak) You would want to loop through all three statues to get all CAMPAIGNS (aka, program/automation emails). [07:40:01] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1076933 (owner: 10L10n-bot) [08:33:28] (03CR) 10Raimond Spekking: [C:03+2] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1076933 (owner: 10L10n-bot) [09:27:30] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog: Delete payment token for Gravy Adyen trxns created during one-hour test - https://phabricator.wikimedia.org/T375734#10190755 (10Damilare) We reached out to Adyen and they confirmed that there was no token attached to those subscriptions as Gravy uses a differ... [09:40:34] (03PS5) 10Damilare Adedoyin: Handle exception to prevent js errors on DI [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1058684 [09:41:03] (03CR) 10CI reject: [V:04-1] Handle exception to prevent js errors on DI [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1058684 (owner: 10Damilare Adedoyin) [09:41:42] (03PS6) 10Damilare Adedoyin: Handle exception to prevent js errors on DI [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1058684 [09:43:43] (03CR) 10Damilare Adedoyin: "Thanks ejegg for the review, done." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1058684 (owner: 10Damilare Adedoyin) [09:47:07] (03PS1) 10Damilare Adedoyin: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1076981 [09:49:48] (03CR) 10Damilare Adedoyin: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1076981 (owner: 10Damilare Adedoyin) [09:50:13] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1076981 (owner: 10Damilare Adedoyin) [10:03:46] lol those seed emails had me thining Lisa was leaving [10:06:21] same haha^ [10:24:01] (03PS5) 10Damilare Adedoyin: Refactor Gravy provider to use guest checkout. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1073841 (https://phabricator.wikimedia.org/T374461) [10:27:24] (03PS9) 10Damilare Adedoyin: Setup Venmo and general redirect payment provider class for methods. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1075583 (https://phabricator.wikimedia.org/T375635) [10:30:39] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog, 07payments-orchestration: Set up recurring charge for venmo - https://phabricator.wikimedia.org/T376122 (10Damilare) 03NEW [13:30:06] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM unavailable intermittently - https://phabricator.wikimedia.org/T376142 (10jgleeson) 03NEW [13:32:26] hmm, Argument 1 passed to SmashPig\PaymentProviders\dlocal\DlocalPaymentResponseFactory::fromErrorResponse() must be of the type array, null given [13:32:38] that was what killed the recurring charge job [13:32:47] let's see if the actual error is anywhere [13:33:47] ah yeah, Request returned (502) GATEWAY ERROR: .... [13:33:54] so I guess we need to handle that [13:35:31] this civicrm issue is weird [13:35:44] damilare: were you getting the same error earlier and then it just went away? [13:36:25] ooh weird, what's been deployed lately? [13:37:01] maybe ConfigAndLog will have the table name? [13:37:13] I didn't see anything in there ejegg [13:37:21] dang [13:37:23] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM unavailable intermittently - https://phabricator.wikimedia.org/T376142#10191770 (10jgleeson) These patches went out last night https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1076891 [13:38:03] (03PS1) 10Ejegg: Default rawErrors to [] not null [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1077023 [13:38:04] that log dir is slightly chaotic [13:38:12] hah, slightly [13:38:40] I think that default [] should maybe make the TypeErrors in fromErrorResponse() not happen ^^ [13:39:10] no updates to be run on Civi? [13:39:19] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1077023 (owner: 10Ejegg) [13:39:19] nope... [13:39:23] thanks! [13:39:27] np [13:39:51] (03Merged) 10jenkins-bot: Default rawErrors to [] not null [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1077023 (owner: 10Ejegg) [13:40:55] I wonder if we can figure out what the requests are that cause those errors [13:41:48] ok, i see one 500 response for an advanced search result [13:41:59] jgleeson: not really, the error was "Db table not found" [13:43:30] and a bunch of 500 errors on contact views. huh [13:43:34] which is what is in the phab [13:44:35] ahh I was out for a bit, I see it came up again [13:44:53] I just grepped for '/500' in syslog and found 61 [13:45:09] seems like that could be disruptive... [13:45:49] I wonder if it has to do with replication and the read/write db split? [13:46:18] though I guess if they're making a new table for a search in one page load the next page loads should all hit the write db [13:46:39] Jeff_Green: do we have the raw sql errors logged anywhere at the db level? [13:46:49] I have some timestamps I'd like to check if possible [13:47:25] or if it's not possible to log just the SQL that causes errors, could we turn on general SQL logging on Civi for a little while? [13:48:22] ejegg: I'm not sure whether trivial errors would be logged the way we're configured now, but we can try enabling general logging [13:48:27] looking at what we have [13:49:24] aha, it's in syslog! [13:49:30] oh cool [13:49:37] no need Jeff_Green, thanks anyway [13:49:38] I was just gonna say we only have slow logs enabled [13:49:40] great [13:49:56] lemme paste some stuff [13:50:03] in phab [13:50:55] gotta grab the kids but thanks for digging! [13:51:36] ooh there's a full stack trace [14:04:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM unavailable intermittently - https://phabricator.wikimedia.org/T376142#10191882 (10Ejegg) Some SQL and a stack trace. Looks like it's the query to build the summary page missing a table definition: {P69439} [14:06:18] I wonder what triggers that then [14:11:09] guessing the issue had to do with this contact layout editor update: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1075645 [14:11:15] I'mma try just rolling that one back [14:11:36] oho, it says 'Requires hitting civicrm/menu/rebuild' in the commit message [14:11:47] I guess we can try that [14:13:10] ok, that's the cache cleared [14:14:42] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:servers migration task - https://phabricator.wikimedia.org/T375151#10191898 (10Jhancock.wm) [14:15:15] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM unavailable intermittently - https://phabricator.wikimedia.org/T376142#10191900 (10Ejegg) On a patch to update the contact layout editor I found the instruction "Requires hitting civicrm/menu/rebuild", so I've just hit that URL to rebuild all the... [14:15:53] awesome, thanks for finding that ejegg [14:18:52] (03PS1) 10Damilare Adedoyin: Add tests to ensure recurring token is sent for recurring charges [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1077033 (https://phabricator.wikimedia.org/T376122) [14:51:49] well, no additional 500 errors in syslog since the cache clear 40 minutes ago [14:56:23] damilare: so what other methods do you expect to use the RedirectPaymentProvider for? I'm looking over https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1075583 [14:57:40] ejegg basically methods that have us redirect the donor to another page for authentication/approval e.g BT and Venmo [14:58:24] with gravy they all follow the same flow, so I'm thinking we can use the same class for them [15:04:43] ok, cool [15:05:18] shoot, it's got a merge conflict with the guest checkout [15:05:41] so which do you prefer to merge first and which do you prefer to rebase damilare ? [15:05:48] yh I wasn't sure which would go first [15:06:01] but preferably guest checkout should go first [15:06:06] i guess maybe we want guest checkout for the next card test, yeah [15:06:10] ok, I'll focus on that first [15:06:16] I'd rebase over that [15:06:18] thanks [15:06:33] (should be easier for me to smoke test the guest checkout, too!) [15:24:30] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155 (10Dwisehaupt) 03NEW [15:25:49] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192254 (10Dwisehaupt) [15:27:34] (03PS6) 10Damilare Adedoyin: Refactor Gravy provider to use guest checkout. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1073841 (https://phabricator.wikimedia.org/T374461) [15:27:34] (03PS10) 10Damilare Adedoyin: Setup Venmo and general redirect payment provider class for methods. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1075583 (https://phabricator.wikimedia.org/T375635) [15:28:24] (03PS2) 10Damilare Adedoyin: Add tests to ensure recurring token is sent for recurring charges [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1077033 (https://phabricator.wikimedia.org/T376122) [15:43:20] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192447 (10CaCogdill-WMF) Thanks so much, @Dwisehaupt ! Excited to be making my (re)inaugural post in Phab. I do not have a yubikey. Should I initiate that request or does fr-tech need to kick it off? [15:43:36] !!! [15:44:36] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192467 (10jgleeson) Welcome back @CaCogdill-WMF !!!! [15:48:04] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192478 (10CaCogdill-WMF) Thank you, @jgleeson <3 so happy to be back!! Also noticing I want to make sure this request include mysql access if possible (I don't see that box ticked at the top, not... [15:48:52] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192494 (10Dwisehaupt) @CaCogdill-WMF You can do the request. Otherwise it's just me asking the exact same question. :) If you provide your shipping address and if you would like a USB A or C versio... [15:49:34] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192510 (10Dwisehaupt) [15:50:00] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192512 (10Dwisehaupt) >>! In T376155#10192478, @CaCogdill-WMF wrote: > Thank you, @jgleeson <3 so happy to be back!! > > Also noticing I want to make sure this request include mysql access if poss... [15:52:25] 10fundraising-tech-ops: Fundraising access request for ccogdill - ssh - https://phabricator.wikimedia.org/T376155#10192549 (10CaCogdill-WMF) Lovely, thanks! Just put in the yubikey request. [15:55:37] 03Fundraising Sprint: Sssss, 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Duplicate contributions after import - https://phabricator.wikimedia.org/T376068#10192585 (10MDemosWMF) Possibly connected to this is that it takes a long time for the files to process so we sometimes open the list of donati... [17:02:21] 03Fundraising Sprint: Sssss, 06Fundraising-Backlog, 07payments-orchestration: Setup DI Frontend for Venmo - https://phabricator.wikimedia.org/T376182 (10Damilare) 03NEW [17:02:30] (03PS1) 10Damilare Adedoyin: Setup Venmo frontend on Gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1077074 (https://phabricator.wikimedia.org/T376182) [17:02:47] (03PS2) 10Damilare Adedoyin: WIP: Setup Venmo frontend on Gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1077074 (https://phabricator.wikimedia.org/T376182) [17:04:54] (03CR) 10CI reject: [V:04-1] WIP: Setup Venmo frontend on Gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1077074 (https://phabricator.wikimedia.org/T376182) (owner: 10Damilare Adedoyin) [17:18:57] 06Fundraising-Analysis, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: Archive banner impression stats on a rolling basis - https://phabricator.wikimedia.org/T373529#10193082 (10AKanji-WMF) Discussed in Analytics call Oct 1/24 - lower priority, though worth a discussion with leadership re: long... [17:22:50] 06Fundraising-Analysis, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Gut-check new donor numbers - https://phabricator.wikimedia.org/T357348#10193135 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF closed based on oct 1 call; @JMando cleans and provides data to BB. [17:28:03] hmm we do need to get that guest checkout out before venmo [17:29:15] the gravy buyer id complicates things when we need to save the venmo username. [17:49:31] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 07recurring upgrade: Can the recurring charge job pick up payments one by one? - https://phabricator.wikimedia.org/T357317#10193219 (10MSuijkerbuijk_WMF) Thanks. I'm planning a new upgrading push in Q3 (February-March). Great to... [18:49:39] 06Fundraising-Backlog: 0.00 native Google Pay payment - https://phabricator.wikimedia.org/T376195 (10Cstone) 03NEW [19:38:06] 06Fundraising-Backlog: Figure out how we want payments that have a delay in settlement in civi - https://phabricator.wikimedia.org/T376203 (10Cstone) 03NEW [20:41:25] (03CR) 10Ejegg: [C:03+2] "Thanks, this looks good!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1073841 (https://phabricator.wikimedia.org/T374461) (owner: 10Damilare Adedoyin) [20:41:54] (03Merged) 10jenkins-bot: Refactor Gravy provider to use guest checkout. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1073841 (https://phabricator.wikimedia.org/T374461) (owner: 10Damilare Adedoyin) [21:34:51] (03PS6) 10Jgleeson: Add test to probe Pending Transaction Resolver code handling of duplicate emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1075293 (https://phabricator.wikimedia.org/T375397) [21:35:14] (03Abandoned) 10Jgleeson: Refactor pending transaction resolver code to improve testability. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1075290 (https://phabricator.wikimedia.org/T375397) (owner: 10Jgleeson) [21:42:13] ejegg|away: I got that test passing after refactoring in the changes you suggested. it was failing earlier locally due to time math failures on my part which I just realised after a 3 hour break. Anyway, I've fixed that and also removed the test seams patchs I added to inject in testable stubs as we no longer need them using the alternative approach you suggested. thanks again for [21:42:15] the idea! hopefully it will pass on CI after I go! :) see you all tomorrow o/ [21:54:55] (03CR) 10CI reject: [V:04-1] Add test to probe Pending Transaction Resolver code handling of duplicate emails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1075293 (https://phabricator.wikimedia.org/T375397) (owner: 10Jgleeson) [21:56:54] hmm looking at https://phabricator.wikimedia.org/T376068#10192585 - the only speed thing I can think of is that we don't run coworker when there is load [21:59:39] 03Fundraising Sprint: Sssss, 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Duplicate contributions after import - https://phabricator.wikimedia.org/T376068#10194020 (10Eileenmcnaughton) @MDemosWMF we can look at whether there are are any bottlenecks in the code we can improve (maybe add a phab) - b... [23:06:47] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:rack/install/configuration new firewalls - https://phabricator.wikimedia.org/T374176#10194218 (10Papaul) [23:14:14] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:servers migration task - https://phabricator.wikimedia.org/T375151#10194252 (10Papaul) [23:39:41] (03PS2) 10Eileen: Minor tidy up in config function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1076895 (https://phabricator.wikimedia.org/T374063) [23:39:41] (03PS1) 10Eileen: WIP fix import handling of duplicates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1077120 (https://phabricator.wikimedia.org/T374063) [23:43:35] (03CR) 10CI reject: [V:04-1] WIP fix import handling of duplicates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1077120 (https://phabricator.wikimedia.org/T374063) (owner: 10Eileen) [23:54:24] (03PS1) 10Eileen: Fix our import hook for when contact ID is present [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1077121 (https://phabricator.wikimedia.org/T374063) [23:56:42] (03PS1) 10Eileen: Create a group to add imported duplicates to [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1077122 (https://phabricator.wikimedia.org/T374063)