[00:01:00] (03PS5) 10Eileen: port date patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138477 (https://phabricator.wikimedia.org/T359219) [00:01:22] (03PS3) 10Eileen: Port patch to speed up managed reconcile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138934 [00:06:00] (03PS10) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [00:06:00] (03PS8) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [00:06:00] (03PS11) 10Eileen: mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [00:06:00] (03PS7) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [00:06:01] (03PS3) 10Eileen: Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) [00:06:02] (03PS2) 10Eileen: Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) [00:06:06] (03PS2) 10Eileen: Make SetTimeOfDayIfStockDonation non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139204 (https://phabricator.wikimedia.org/T359219) [00:06:10] (03PS2) 10Eileen: Extract inValidateModeDoNotRequireTotalAmount [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) [00:06:14] (03PS2) 10Eileen: Benevity import feature 1 - filter not-shared-info [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) [00:06:18] (03PS2) 10Eileen: Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 [00:06:22] (03PS2) 10Eileen: Benevity feature 2 - field transformations on Gift Sourc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) [00:06:26] (03PS1) 10Eileen: Extract Fidelity wrangling to own function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139575 [00:08:01] 06Fundraising-Backlog: Phab template request: new payments type - https://phabricator.wikimedia.org/T392864 (10AKanji-WMF) 03NEW [00:21:16] !log civicrm upgraded from 3ca2db06 to a82ee0f8 [00:21:19] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [00:22:24] weird [00:22:24] now on prod [00:22:25] wmf-cv [00:22:25] [00:22:26] The "--user" option does not exist. [00:38:41] (03CR) 10CI reject: [V:04-1] mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [00:41:57] (03CR) 10CI reject: [V:04-1] Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 (owner: 10Eileen) [00:45:47] (03CR) 10Ejegg: [C:03+2] port date patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138477 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [01:03:39] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Rearrange adyen audit columns in the settlement report to match the default - https://phabricator.wikimedia.org/T392528#10774989 (10Cstone) a:03Cstone I take it back gravy need three more columns: booking_date booking__date_timezone metadata [01:10:52] (03Merged) 10jenkins-bot: port date patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138477 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [01:15:55] PROBLEM - Host frdb2005 is DOWN: PING CRITICAL - Packet loss = 100% [01:16:09] ^^ that's me again. :) [01:20:57] RECOVERY - Host frdb2005 is UP: PING OK - Packet loss = 0%, RTA = 30.49 ms [01:40:50] eileen: does that 'checksum' column have to be manually added? [01:41:17] did the upgrade not work to add it? [01:41:22] oh i see, it's in the 6.1.0 bit [01:41:25] I had already run that [01:41:40] lemme just put the version to something lower [01:42:37] ok, that added it [01:43:39] eileen: but prod is already on 6.1.2 ? [01:44:13] so it won't run in the upgrade script there unless we move it to a different fn name, right? [01:44:46] hmm - it should have cos I included it in the 6.1.2 we pushed out but it doens't look like it did [01:44:58] cos it was on 6.1.beta before [01:45:13] ah dang, I didn't merge the checksum patch before you deployed [01:45:38] sorry, i keep getting called away to help with kid stuff [01:45:43] oh dang [01:45:51] ok I didn't spot that [01:46:10] hmm, want to put it in a 6.1.3 alpha update or something? [01:46:49] ejegg: I think we need to put in wmf_civicrm now cos if we DO upgrade to a real 6.1.3 alpha.. [01:50:44] ok [01:51:10] want me to move that? [01:51:29] ejegg: actually - I'll just stick the domain version on prod back to 6.1.beta so it runs again - bit of a cheat but... [01:51:39] sure, sounds good [01:51:57] didn't seem to have any ill effects when I re-ran the upgrade locally [01:52:02] so I'll just merge that then [01:52:07] ejegg: cool [01:52:32] (03CR) 10Ejegg: [C:03+2] Port patch to speed up managed reconcile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138934 (owner: 10Eileen) [02:10:09] (03Merged) 10jenkins-bot: Port patch to speed up managed reconcile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138934 (owner: 10Eileen) [02:17:29] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139582 [02:17:32] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139582 (owner: 10Ejegg) [02:18:06] ejegg: I just reset the domain version [02:18:13] oh thanks! [02:18:21] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139582 (owner: 10Ejegg) [02:18:51] heh, 'Database is marked with invalid version format. You may want to investigate this before you proceed further.' [02:19:18] locally I tried '6.0.9' and it worked [02:19:38] oh cool [02:19:56] !log civicrm upgraded from a82ee0f8 to 6c4c4016 [02:19:59] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [02:20:03] well there's the new code up [02:20:40] ejegg: just did update civicrm_domain SET version = '6.1.beta1'; [02:21:36] ok, upgrading via the UI [02:21:45] and there's the column [02:22:16] gotta sign off but I'll look at that import patch chain tomorrow [02:23:07] (03PS2) 10Ejegg: Set base URL for donor portal in Civi [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139541 (https://phabricator.wikimedia.org/T388739) [02:23:12] (03CR) 10Ejegg: [V:03+2 C:03+2] Set base URL for donor portal in Civi [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139541 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [02:23:13] cool thanks [02:23:19] thank you [03:13:52] eileen: are you still around? [03:13:58] cstone: yep [03:14:13] what am i doing wrong here, gona upload a WIP [03:14:21] ok sure [03:15:18] i used search kit to build a query and was trying to match it in api4 [03:15:26] but its not happy with my activity wheres [03:15:44] (03PS1) 10Cstone: WIP: Add second recurring failure email send [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139584 (https://phabricator.wikimedia.org/T365488) [03:16:28] line 34 ^ there, its doing the recur.id find fine but it cant find any activity lines [03:17:17] er or it tells me like activity.created_date doesnt exist [03:17:40] so firstly - is the recur.id = 1 hardcoding a problem? [03:18:00] no that was me just testing [03:18:09] to see if it would break on all of the joins [03:18:27] mm ok [03:18:53] oh hand on - the activity.created_date is not in the select so it won't be there [03:19:05] - no you mean in the where [03:19:13] ooh okay i was wondering that but then i thought the one i was looking at as an example didnt have it [03:19:15] BUT activity is ActivityContact [03:19:21] but created_date is on Activity [03:19:28] try activity_id.created_date [03:20:00] ahh i was looking at it wrong in mysql workbench [03:20:46] ok so it needs to be in the select for the where to know about it? [03:21:21] and actually use the right table hah thanks [03:22:47] I think the select was me misunderstanding [03:22:53] it's the wrong table that's the issue [03:23:08] yeah im missing a join [03:24:11] I think you would get an implicit join if you use activity_id.created_date [03:30:17] ok i got it working I had ActivityContact in the wrong place and Activity not at all [03:34:18] (03CR) 10Cstone: "can you update the commit message to match what is actually happening now? since the check is gone and we are now casting to float" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [03:42:25] (03PS1) 10Eileen: Extract & remove ensureTrxnIdentifiersSet [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139586 (https://phabricator.wikimedia.org/T359219) [03:42:27] (03PS1) 10Eileen: Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) [04:02:17] (03CR) 10CI reject: [V:04-1] Extract & remove ensureTrxnIdentifiersSet [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139586 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [04:03:53] (03CR) 10CI reject: [V:04-1] Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [04:14:47] (03PS2) 10Eileen: Extract & remove ensureTrxnIdentifiersSet [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139586 (https://phabricator.wikimedia.org/T359219) [04:14:47] (03PS2) 10Eileen: Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) [04:33:06] (03PS1) 10Wfan: fix state selection style in gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139590 [04:53:33] (03PS2) 10Cstone: WIP: Add second recurring failure email send [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139584 (https://phabricator.wikimedia.org/T365488) [04:57:39] (03PS1) 10Wfan: Add a placeholder phone number for gravy india card payment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139592 (https://phabricator.wikimedia.org/T391970) [04:59:50] (03CR) 10CI reject: [V:04-1] Add a placeholder phone number for gravy india card payment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139592 (https://phabricator.wikimedia.org/T391970) (owner: 10Wfan) [05:14:51] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: LATAM test issues - Gr4vy - https://phabricator.wikimedia.org/T392677#10775183 (10AnnWF) Issue resolved by moving the https://wikimedia.gr4vy.app/merchants/default/flow/card-transaction rule for dlocal above the one on adyen [05:15:02] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: LATAM test issues - Gr4vy - https://phabricator.wikimedia.org/T392677#10775184 (10AnnWF) a:03AnnWF [09:16:48] (03PS11) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [09:16:48] (03PS9) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [09:16:48] (03PS12) 10Eileen: mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [09:16:49] (03PS8) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [09:16:50] (03PS4) 10Eileen: Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) [09:16:51] (03PS3) 10Eileen: Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) [09:16:55] (03PS3) 10Eileen: Make SetTimeOfDayIfStockDonation non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139204 (https://phabricator.wikimedia.org/T359219) [09:16:59] (03PS3) 10Eileen: Extract inValidateModeDoNotRequireTotalAmount [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) [09:17:03] (03PS3) 10Eileen: Benevity import feature 1 - filter not-shared-info [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) [09:17:07] (03PS3) 10Eileen: Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 [09:17:11] (03PS3) 10Eileen: Benevity feature 2 - field transformations on Gift Sourc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) [09:17:15] (03PS2) 10Eileen: Extract Fidelity wrangling to own function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139575 [09:17:19] (03PS3) 10Eileen: Extract & remove ensureTrxnIdentifiersSet [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139586 (https://phabricator.wikimedia.org/T359219) [09:17:23] (03PS3) 10Eileen: Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) [09:17:27] (03PS2) 10Eileen: Add Benevity mgd file [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139209 [09:17:31] (03PS1) 10Eileen: Bulk of the Benevity logic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139793 (https://phabricator.wikimedia.org/T359219) [09:44:20] thanks eileen [09:44:34] it was about time! [09:44:53] (03CR) 10CI reject: [V:04-1] Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [09:47:16] (03CR) 10CI reject: [V:04-1] Benevity import feature 1 - filter not-shared-info [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [09:48:30] (03CR) 10CI reject: [V:04-1] Benevity feature 2 - field transformations on Gift Sourc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [09:51:27] (03CR) 10CI reject: [V:04-1] Add Benevity mgd file [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139209 (owner: 10Eileen) [09:51:58] (03CR) 10CI reject: [V:04-1] Bulk of the Benevity logic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139793 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [10:01:05] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [10:01:35] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [10:08:02] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139209 (owner: 10Eileen) [12:22:50] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 (owner: 10Eileen) [12:28:17] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10dev-images, 10FR-Docker, 13Patch-For-Review: fundraising-tools-bullseye dev image depends on non-existent fundraising-bullseye-php73 - https://phabricator.wikimedia.org/T380134#10776220 (10Daimona) 05Open→03Resolved a:05Daimona→03j... [13:00:52] (03CR) 10Jgleeson: [C:03+1] "I feel like I have no idea what's going on here, but I'm happy to +2 it anyway after a few minor tidy-ups." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [13:07:58] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Set up recurring charge for venmo - https://phabricator.wikimedia.org/T376122#10776369 (10Damilare) Gravy is currently working on other feature requests from the Wikimedia foundation, as such this is not a priority on their end... [13:18:54] (03CR) 10Jgleeson: "The import worked as described but I can't see the records anywhere. Where should I be looking?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [13:19:36] (03CR) 10Jgleeson: "yay an easy one!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 (owner: 10Eileen) [13:26:01] (03CR) 10Jgleeson: [V:03+2 C:03+2] "I ended up fixing the startup errors and got CiviCRM working by running ./scripts/civicrm-update-full.sh. Once that was done, I tried your" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138861 (owner: 10Wfan) [13:41:07] (03CR) 10Jgleeson: [C:03+2] "Looks good. I see you updated the @see @see for that sailor." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [13:41:19] (03CR) 10Damilare Adedoyin: "Thanks wfan and cstone for working on this, I asked a question inline to clarify the need for the cast in the RequestMapper." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [13:42:58] (03CR) 10Jgleeson: [C:03+1] Update Fidelity Import (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [13:57:03] (03PS2) 10Jgleeson: WIP: Fix issues in donation message build script [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139564 (https://phabricator.wikimedia.org/T381012) [14:05:27] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences: Create donor portal main page displaying info from CiviCRM - https://phabricator.wikimedia.org/T388745#10776609 (10Ejegg) a:03Ejegg [14:05:43] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw: Possible frdb2004 hardware failure. - https://phabricator.wikimedia.org/T392579#10776610 (10Jgreen) [14:13:35] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw: Possible frdb2004 hardware failure. - https://phabricator.wikimedia.org/T392579#10776664 (10Papaul) a:03Jhancock.wm @Jhancock.wm when you have a minutes can you please check this host. Also can you also please upgrade the CPLD. Thank you [14:20:18] (03CR) 10Damilare Adedoyin: "Thanks wfan, I've added a follow on comment after confirming the issue." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [14:26:28] (03PS1) 10Ejegg: Short array syntax for CiviProxy config [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139871 [14:29:09] (03PS1) 10Damilare Adedoyin: Remove clean up on mac push scripts [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139872 [14:36:22] (03PS3) 10Jgleeson: WIP: Fix issues in donation message build script [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139564 (https://phabricator.wikimedia.org/T381012) [14:37:41] damilare: does civicrm and standalone install conrrectly now without those lines? I recall you had some issues at the offsite that were fixed due to those changes [14:39:17] yh it does, we had to destroy the volumes for wfan so I don't think those lines are necessary tbh [14:39:47] destroy the volumes and fresh install [14:40:48] wfan: I've fixed the script to build a queue message from the gravy logs. It's still in WIP, as I wanna clean it up a little bit more before deploying it but it's building messages. I had to tidy up how we turn the gravy payment method and submethod in the shorthand names we use e.g. cc, mc, and there way some weirdness in the minfraud logs breaking the parsing of the ip address, which is now fixed. [14:41:29] great damilare. I'll put my mac on charge now as I head out to pick the kids up :) [14:41:52] unless wfan, the other mac user, gets there first :D [14:43:52] sounds good [14:44:11] I also got a new mac too [15:20:17] is it fancy, damilare|brb ? [15:20:24] any new tricks [15:20:38] or port removals [15:20:39] .. [15:30:13] yea its the m4 max, they returned the HDMI, magsafe charger, and SD card slots [15:30:23] so technically nothing new [15:30:49] it feels faster though [15:49:08] nice! [15:58:09] I love USB-C for charging all my devices from phones to headlamps, but I find them to be too brittle for laptop charging, especially for folks who aren't as aware of their surroundings (ADHDers like most of my family). Magsafe or similar is an objectively better option. [15:58:17] RECOVERY - Host frdb2004 is UP: PING OK - Packet loss = 0%, RTA = 30.34 ms [15:59:00] (I bought an older laptop that I would have preferred for the kids' laptop, so that it had the older style Lenovo charger port, the rectangle one) [15:59:46] oh jeez - {"city":"\u00d1u\u00f1oa","country":"US" [16:00:03] and that translates to [16:00:10] city: "Ñuñoa" [16:00:25] which is not great for log parsers [16:00:27] ... [16:01:48] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw, 06SRE: Possible frdb2004 hardware failure. - https://phabricator.wikimedia.org/T392579#10777103 (10Jhancock.wm) @Jgreen reseated all the connections to the backplane. server came up. I checked the firmware version of the CPLD and it is current (1.0.7). lemme... [16:03:47] greg-g: re:charger traps. we have to be mindful of those here too [16:04:13] yh I also like the vintage feel of using magsafe :) [16:04:33] we had to set up a rule of no charging while using [16:05:14] after a few things on the unit where the charger block for the kids devices got wiped out due to a burst of energy [16:06:56] Cool thanks Jack! [16:07:54] wfan: turkish just changed everyones seats on our flight and then wont let us change them back, hopefully this doesn't happen to you but just a warning incase [16:08:41] * dwisehaupt imagines a bingo ball wheel for seat selection. [16:08:43] I thought you reserved your seat? I also just reserved yesterday haha [16:09:16] Let me check last time system blocked me then I called them to resolved the issue so I just booked the seat yesterday 😀 [16:10:18] Mines the same, wish you a safe fight and see u there Christine :) [16:13:29] yep even with reserved seats [17:33:25] 06Fundraising-Backlog, 07payments-orchestration: Set up recurring charge for venmo - https://phabricator.wikimedia.org/T376122#10777731 (10XenoRyet) [18:37:25] 06Fundraising-Backlog, 06FR-donorrelations: Civi Thank You email check April 29 - https://phabricator.wikimedia.org/T392947 (10MBeat33) 03NEW [18:44:32] (03PS1) 10Ejegg: Fix message limit for BatchSend [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139921 (https://phabricator.wikimedia.org/T392947) [18:54:57] (03CR) 10Wfan: Fix message limit for BatchSend (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139921 (https://phabricator.wikimedia.org/T392947) (owner: 10Ejegg) [19:03:16] (03CR) 10Cstone: [C:03+2] "worked locally for me" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139921 (https://phabricator.wikimedia.org/T392947) (owner: 10Ejegg) [19:04:35] (03CR) 10CI reject: [V:04-1] Fix message limit for BatchSend [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139921 (https://phabricator.wikimedia.org/T392947) (owner: 10Ejegg) [19:19:49] (03Merged) 10jenkins-bot: Fix message limit for BatchSend [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139921 (https://phabricator.wikimedia.org/T392947) (owner: 10Ejegg) [19:26:53] 06Fundraising-Backlog: Wrong PSP chosen for Apple Pay and Gpay in LATAM - https://phabricator.wikimedia.org/T392957 (10EMartin) 03NEW [19:29:12] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139924 [19:29:28] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139924 (owner: 10Ejegg) [19:30:18] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139924 (owner: 10Ejegg) [19:34:48] (03PS12) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [19:35:31] (03PS13) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [19:37:51] (03PS14) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [19:37:56] (03CR) 10Eileen: Update Fidelity Import (035 comments) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [19:39:11] (03CR) 10Eileen: "OK - I addressed those things" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [19:44:02] (03CR) 10Cstone: Add amount check before create payment request in gravy (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [19:48:30] !log civicrm upgraded from 6c4c4016 to 2b8b4651 [19:48:32] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:56:39] (03CR) 10Eileen: "You should be able to navigate back to the outcome via https://wmff.localhost:32353/civicrm/imports/my-listing - note that it won't import" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [19:58:02] (03CR) 10Eileen: "actually the link seems to be missing from there now - but the results are also under packaged searches https://wmff.localhost:32353/civic" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [19:59:19] 06Fundraising-Backlog, 06FR-donorrelations: Civi Thank You email check April 29 - https://phabricator.wikimedia.org/T392947#10778258 (10Cstone) This was from switching the thank you send out of the thank you module, no batch limit became limit=0 [20:01:33] 10fundraising-tech-ops: Fix check_puppetrun to correctly parse yaml on bookwormwhen there are failures - https://phabricator.wikimedia.org/T392961 (10Dwisehaupt) 03NEW [20:07:58] 06Fundraising-Backlog: Wrong PSP chosen for Apple Pay and Gpay in LATAM - https://phabricator.wikimedia.org/T392957#10778293 (10EMartin) 05Open→03Resolved a:03EMartin [20:11:22] @ejegg [20:12:39] Ooops sorry, ejegg, are you good for me to set up some time for you to talk about donor portal/donor identification stuff? [20:13:49] 06Fundraising-Backlog: Monthly convert specific email send setting contribution thank you date - https://phabricator.wikimedia.org/T392962 (10Cstone) 03NEW [20:21:27] (03PS1) 10Eileen: Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 [20:21:27] (03PS1) 10Eileen: Port lost buttons [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139929 [20:26:58] (03PS2) 10Eileen: Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 [20:26:58] (03PS2) 10Eileen: Port lost buttons [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139929 [20:41:47] (03CR) 10CI reject: [V:04-1] Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 (owner: 10Eileen) [20:45:21] (03PS3) 10Eileen: Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 [20:45:21] (03PS3) 10Eileen: Port lost buttons [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139929 [20:53:08] 10fundraising-tech-ops, 13Patch-For-Review: Fix check_puppetrun to correctly parse yaml on bookworm when there are failures - https://phabricator.wikimedia.org/T392961#10778409 (10Dwisehaupt) [21:16:20] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [21:17:55] (03CR) 10CI reject: [V:04-1] Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 (owner: 10Eileen) [21:22:26] (03CR) 10Jgleeson: [C:03+2] "lgtm!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [21:24:34] wfan: I dunno if you caught this, but looks like we don't need to worry about that log line we discussed on the call [21:25:21] Cool the donation_rule.yaml right [21:25:31] it would always appear when the getDonationRules function is called https://github.com/wikimedia/mediawiki-extensions-DonationInterface/blob/2a680bae187a7f7df6a5a5dfa2c8639ef0f6954a/gateway_common/gateway.adapter.php#L928 [21:25:36] yep [21:25:56] damilare: what if one of those return lines are hit [21:26:05] Not if rulematched [21:26:17] But we will have no rule for those currencies [21:26:31] jgleeson: that's what happens with the INR that is set [21:26:35] Maybe just rename the log… feel annoying haha [21:26:51] so if a rule matches, we won't get the log line from that code right? [21:27:43] no we shouldn't [21:28:38] I'd just debug through it to confirm [21:28:55] so does that mean for all non-INR dlocal currencies we've been getting that log line [21:28:56] to confirm if setting that default value like in dlocal would match [21:29:04] ah good idea [21:29:12] maybe the default changes things [21:29:32] possibly, this transaction got the log line and it's in USD: https://wikimedia.gr4vy.app/merchants/default/transactions/74bdf491-e971-49a6-b33d-50301f99e1e5/overview [21:33:29] 10fundraising-tech-ops: New browser cert & ssh key for greg - https://phabricator.wikimedia.org/T392965 (10greg) 03NEW [21:34:07] (03Merged) 10jenkins-bot: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) (owner: 10Eileen) [21:37:39] (03PS13) 10Eileen: mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [21:38:06] (03PS9) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [21:39:19] (03PS10) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [21:39:52] yea jgleeson wfan, with the default values we don't run into that line [21:40:05] the USD default? [21:40:24] this line here: https://github.com/wikimedia/mediawiki-extensions-DonationInterface/blob/2a680bae187a7f7df6a5a5dfa2c8639ef0f6954a/gateway_common/gateway.adapter.php#L924-L926 returns the default rule as it does not have any conditions stated [21:40:40] yep USD default [21:40:42] ah nice [21:40:49] so it was those magic config files [21:41:03] yepp [21:41:04] we should probably review whether we even need those anymore [21:41:08] when convenient [21:41:24] that would take a way a layer of hidden and easily forgettable config [21:41:27] away* [21:42:19] yea but it's convenient to make config changes without having to do a full payments deploy [21:43:25] (03PS5) 10Eileen: Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) [21:44:35] jgleeson wfan: I've pushed up the default rule, please can you help review [21:44:59] (03CR) 10Eileen: [C:03+2] "Putting the + 2 back as the new patch set is not actually changed - I just resolved a conflict because I had tweaked a comment block earli" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [21:45:14] (03PS4) 10Eileen: Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) [21:47:33] checking damilare [21:47:41] ty! [21:47:55] LGTM damilare [21:49:22] ok I'd deploy that tomorrow when I resume to watch the logs [21:49:27] thanks all [21:49:34] (thumbs up) [21:49:36] and goodnight! [21:49:42] good night [22:00:16] (03Merged) 10jenkins-bot: Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [22:04:34] (03CR) 10CI reject: [V:04-1] Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [22:16:27] 10fundraising-tech-ops: New browser cert & ssh key for greg - https://phabricator.wikimedia.org/T392965#10778598 (10Dwisehaupt) a:03Dwisehaupt SSH pub key updated and previous keys removed per request. SSL client certificate sent via email and password sent via ssh. puppet runs forced on frbast1002 and frpm100... [22:22:39] 10fundraising-tech-ops, 13Patch-For-Review: Fix check_puppetrun to correctly parse yaml on bookworm when there are failures - https://phabricator.wikimedia.org/T392961#10778603 (10Dwisehaupt) The change has been rolled out for fr-tech. The gerrit patch is up for review for production for use there if needed. [22:22:47] (03CR) 10Eileen: [V:03+2] "Putting back the +2 since jenkins does not seem to be re-running this after a big wait & there is no relevant change" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 (owner: 10Eileen) [22:23:57] (03CR) 10Eileen: [V:03+2] "Putting back the +2 as jenkins not showing much sign of kicking in & the rebase was not material" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 (owner: 10Eileen) [22:24:45] (03CR) 10Eileen: [V:03+2] "putting back the +2 as change was not material & jenkins showing no sign of kicking in" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 (owner: 10Eileen) [22:27:34] (03PS5) 10Eileen: Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) [22:27:34] (03PS4) 10Eileen: Make SetTimeOfDayIfStockDonation non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139204 (https://phabricator.wikimedia.org/T359219) [22:27:34] (03PS4) 10Eileen: Extract inValidateModeDoNotRequireTotalAmount [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) [22:27:35] (03PS4) 10Eileen: Benevity import feature 1 - filter not-shared-info [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) [22:27:35] (03PS4) 10Eileen: Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 [22:27:38] (03PS4) 10Eileen: Benevity feature 2 - field transformations on Gift Sourc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) [22:27:42] (03PS3) 10Eileen: Extract Fidelity wrangling to own function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139575 [22:27:46] (03PS4) 10Eileen: Extract & remove ensureTrxnIdentifiersSet [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139586 (https://phabricator.wikimedia.org/T359219) [22:27:50] (03PS4) 10Eileen: Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) [22:27:54] (03PS2) 10Eileen: Bulk of the Benevity logic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139793 (https://phabricator.wikimedia.org/T359219) [22:27:58] (03PS3) 10Eileen: Add Benevity mgd file [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139209 [22:28:37] well I v+2 forced the ones that were +2 & then rebased the chain to bring them back to life [22:29:16] (03PS4) 10Eileen: Add new activity types for recurring convert, eoy [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 [22:29:35] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139928 (owner: 10Eileen) [22:41:51] (03CR) 10Ejegg: [C:03+2] Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [22:42:19] yay thanks [22:46:05] will review more in a bit [22:50:37] yay - thanks [22:50:54] (03CR) 10CI reject: [V:04-1] Extract inValidateModeDoNotRequireTotalAmount [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [22:51:24] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [22:53:05] (03CR) 10CI reject: [V:04-1] Wait until end to transfer contact_id to the contribution [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [23:27:04] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139587 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [23:48:39] 10fundraising-tech-ops, 10FR-Tech-Analytics: Determine how to keep acceptable levels of dbt assets - https://phabricator.wikimedia.org/T392747#10778683 (10Dwisehaupt) Just cleaned up some more since we hit the warning threshold again. Here are the commands/stats: ` root@fran2001:/var/lib/dagster# du -sh dbt_t...