[00:56:33] (03CR) 10Lars SG: [C:03+2] Update our offline gifts to have offline for channel, is_major_gift = TRUE. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264780 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [00:57:41] (03CR) 10Eileen: [V:03+2] Update our offline gifts to have offline for channel, is_major_gift = TRUE. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264780 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [00:58:20] (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/1264855 [00:58:29] (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/1264855 (owner: 10Eileen) [01:00:20] !log civicrm upgraded from cf871dd3 to 591bef29 [01:00:21] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [01:10:07] (03PS1) 10Wfan: Some validation error do not have error_code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264860 (https://phabricator.wikimedia.org/T421155) [01:10:36] (03CR) 10CI reject: [V:04-1] Some validation error do not have error_code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264860 (https://phabricator.wikimedia.org/T421155) (owner: 10Wfan) [01:10:37] (03PS1) 10Wfan: Render failmail for all validation errors [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264861 (https://phabricator.wikimedia.org/T421155) [01:28:52] (03CR) 10Lars SG: [C:03+2] Set channel = Other Online For remaining online gifts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264797 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [01:31:02] 03Fundraising Sprint - Floor is Lava, 10fundraising-tech-ops, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition: Create SFTP keypair for MediaWiki donor export - https://phabricator.wikimedia.org/T421785#11769690 (10Dwisehaupt) Secrets initially populated from `private::passwords::d... [01:38:03] (03PS3) 10Eileen: Early return if no amount is set [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260236 (https://phabricator.wikimedia.org/T409994) [01:38:15] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260236 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [01:38:33] (03PS5) 10Eileen: See if this bubbles up deprecations [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260827 [01:38:39] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260827 (owner: 10Eileen) [01:47:35] (03PS4) 10Eileen: Set is_major_gift to true, channel to Other Offline for imports [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260161 (https://phabricator.wikimedia.org/T409994) [01:50:09] (03PS5) 10Eileen: Set is_major_gift to true, channel to Other Offline for imports [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260161 (https://phabricator.wikimedia.org/T409994) [01:50:26] larssandergreen: I guess we had better start sett ing them on import too ... ^^ [01:56:45] (03Merged) 10jenkins-bot: Set channel = Other Online For remaining online gifts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264797 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [02:43:11] (03Abandoned) 10Eileen: Add fields to the TransactionLog view Bug: T420057 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1253714 (https://phabricator.wikimedia.org/T420057) (owner: 10Eileen) [02:43:55] (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/1264909 [02:44:03] (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/1264909 (owner: 10Eileen) [02:45:19] (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/1264909 (owner: 10Eileen) [02:46:04] !log civicrm upgraded from 591bef29 to c3cc3562 [02:46:05] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [06:16:14] PROBLEM - check_memory on fransw2003 is CRITICAL: CRIT Memory 98% used. Largest process: trino-server-co (1564) = 97.2% https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2003&service=check_memory [06:21:14] PROBLEM - check_memory on fransw2003 is CRITICAL: CRIT Memory 98% used. Largest process: trino-server-co (1564) = 97.2% https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2003&service=check_memory [06:26:14] PROBLEM - check_memory on fransw2003 is CRITICAL: CRIT Memory 96% used. Largest process: trino-server-co (1564) = 95.0% https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2003&service=check_memory [06:31:14] RECOVERY - check_memory on fransw2003 is OK: OK Memory 89% used https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2003&service=check_memory [11:20:55] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Donor-portal, 06FR-donorrelations, and 2 others: PDF Civi receipts - feature request - https://phabricator.wikimedia.org/T368671#11771995 (10RLewis) @Lars just chiming in on this as well, we often have to send $5k+ TYL via pdfs via email and it would... [12:59:14] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 07fr-current-sprint, 07payments-orchestration: Recurring token migration - Paypal - https://phabricator.wikimedia.org/T405483#11772551 (10Damilare) We met with PayPal yesterday to discuss the importance of this migration and potential solutions t... [13:01:28] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing: "With Contact" field not filled for some DM Activities - https://phabricator.wikimedia.org/T420652#11772557 (10CHudson-WMF) Ah, thanks for confirming--good to know that it looks like these con... [13:31:49] fr-tech: PHP getting you down? Well, I might have the solution, fancy looking at some jazzy Python code? I would really appreciate some help reviewing a few patches related to our work on the Donor ID project, please, if folks have capacity. We've got 3 patches queued up that we'd like to merge today, if possible, to help with Jon Robson's recent ask to run a test later tonight. The chain starts here [13:31:49] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/tools/+/1253624/. Thanks in advance, and if folks have review they need, I'm up for some review trade too! :) [13:38:56] 03Fundraising Sprint - Floor is Lava, 10fundraising-tech-ops, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition: Create SFTP keypair for MediaWiki donor export - https://phabricator.wikimedia.org/T421785#11772792 (10jgleeson) Thanks @Dwisehaupt! I'll see if we can get these on Slack... [13:49:07] (03PS2) 10Jgleeson: Some validation error do not have error_code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264860 (https://phabricator.wikimedia.org/T421155) (owner: 10Wfan) [13:50:25] wfan: I was just reviewing your validation error patch and saw the CI gods -1ed it due to spacing... so that update should hopefully fix it ^ [13:51:46] I'll take a look at those python patches jgleeson [13:51:52] nice sales pitch [13:51:57] thanks ejegg ! [13:52:02] :) [14:06:12] (03CR) 10Jgleeson: [C:03+2] "LGTM! We could put that duplicated array in a small helper if we wanted to be really lean, but it's not blocking." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264703 (https://phabricator.wikimedia.org/T421733) (owner: 10Lars SG) [14:08:51] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264664 (https://phabricator.wikimedia.org/T421282) (owner: 10Ejegg) [14:09:24] (03Merged) 10jenkins-bot: Pending message lookup defaults to including resolved [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264664 (https://phabricator.wikimedia.org/T421282) (owner: 10Ejegg) [14:12:18] 06Fundraising-Backlog, 10fundraising-tech-ops, 07fr-current-sprint, 07FR-Fraud: Create a payment attempts table for fraud scoring service - https://phabricator.wikimedia.org/T417217#11773059 (10Damilare) Thanks for kicking off on this. The columns you've added so far, does make sense. The only change I'd r... [14:17:34] (03PS2) 10Jgleeson: Add gateway txn hints to pending message lookups [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264678 (https://phabricator.wikimedia.org/T421282) (owner: 10Ejegg) [14:18:48] (03CR) 10Jgleeson: [C:03+2] "LGTM! thanks. I added the Bug ID to the commit and guessed it was the same as the one used in the prior patch." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264678 (https://phabricator.wikimedia.org/T421282) (owner: 10Ejegg) [14:19:33] (03Merged) 10jenkins-bot: Add gateway txn hints to pending message lookups [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264678 (https://phabricator.wikimedia.org/T421282) (owner: 10Ejegg) [14:20:52] (03CR) 10Ejegg: [C:03+2] "Code looks good, file decrypts as expected" [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1253624 (https://phabricator.wikimedia.org/T416948) (owner: 10Jgleeson) [14:22:28] (03Merged) 10jenkins-bot: Add optional AES encryption for mediawiki donor export output [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1253624 (https://phabricator.wikimedia.org/T416948) (owner: 10Jgleeson) [14:22:30] (03Merged) 10jenkins-bot: Limit MW donor export CSV to email and donor_status_id only [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1256389 (https://phabricator.wikimedia.org/T416948) (owner: 10Jgleeson) [14:22:43] (03CR) 10Ejegg: [C:03+2] Limit MW donor export CSV to email and donor_status_id only [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1256389 (https://phabricator.wikimedia.org/T416948) (owner: 10Jgleeson) [14:22:48] thanks ejegg ! [14:25:41] (03CR) 10Ejegg: [C:03+2] Add optional SFTP upload for mediawiki donor export [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1264753 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [14:25:55] all looks good! [14:26:30] In prod we'll probably also need to add the host_key config for the Sftp.client to pick up [14:26:42] (not a code change) [14:26:49] just config [14:27:36] (03Merged) 10jenkins-bot: Add optional SFTP upload for mediawiki donor export [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1264753 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [14:30:12] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1265438 [14:30:42] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1265438 (owner: 10Ejegg) [14:31:08] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1265438 (owner: 10Ejegg) [14:31:25] yeah that makes sense ejegg [14:32:47] I probably need to add this to the docs somewhere in that sftp lib https://docs.paramiko.org/en/stable/api/client.html [14:33:31] it's lasted well tho! [14:33:44] 13 years and counting [14:37:07] (03Merged) 10jenkins-bot: Add frequency, currency, native currency amount and USD amount to recurring pause and cancel activities from Donor Portal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1264703 (https://phabricator.wikimedia.org/T421733) (owner: 10Lars SG) [14:39:42] heh, we have had to update that lib a couple times, haven't we? [14:39:48] fr-tech: we've got a patch which fixes donor portal on android. do we have any android users who could review it pls? https://gerrit.wikimedia.org/r/c/mediawiki/extensions/DonationInterface/+/1262122 [14:39:50] I guess just when we changed python versions [14:39:57] I've got android [14:40:04] oh right ejegg. I honestly can't remember [14:40:17] will re-route my proxy host for a sec to test that [14:41:48] oh right, we need to go to the http port not https [14:41:56] are we exposing http for donorportal? [14:43:32] not yet [14:43:49] let's do that, and let's make another set of hostnames to proxy-forward to donorportal [14:56:24] (03PS1) 10Ejegg: Add test link for Gravy / Vipps [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265446 [14:56:32] (03PS1) 10Ejegg: Add proxy forwarding for donor portal [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 [14:56:46] I just need to do the hostname config in horizon for ^^^^ [14:57:33] oh i should shut down that old instance [14:57:58] (03CR) 10Jgleeson: [V:03+2 C:03+2] Add test link for Gravy / Vipps [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265446 (owner: 10Ejegg) [14:59:19] oho, horizon's web proxies seem to be able to proxy to https now! [14:59:42] (03CR) 10Ejegg: [C:04-1] "We might not need the HTTP ports after all" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 (owner: 10Ejegg) [14:59:50] (03CR) 10Jgleeson: "Looks good apart from the dupe payments port" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 (owner: 10Ejegg) [15:00:12] thanks jgleeson [15:00:34] np [15:00:43] actually, I think we DO want the http ports, so we don't have to figure out how to get horizon to accept the self-signed certs [15:01:39] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donor-portal, 07fr-current-sprint: Create Recurring Upgrade and Recurring Downgrade activities when completed by DR through CiviCRM UI - https://phabricator.wikimedia.org/T409097#11773397 (10Lars) This came... [15:04:28] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - Update error message copy - remove 'Thank you!' - https://phabricator.wikimedia.org/T421917 (10KHill-WMF) 03NEW [15:18:00] ok, that's the hostnames configured [15:18:07] lemme fix that duplicate port [15:20:54] (03PS2) 10Ejegg: Add proxy forwarding for donor portal [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 [15:21:59] hmm, not loading anything [15:22:01] let's see [15:22:27] 8205:localhost:9015 looks good I think [15:23:16] do we need config to allow http? [15:23:26] let's see if I can just load on localhost:9015 [15:23:52] right, that's not loading [15:24:56] I wonder if I need to rebuild the container for the ports to take effect [15:25:20] just restarting doesn't seem to do it [15:27:07] ok, after down / up I can load locally [15:27:15] (03CR) 10Lars SG: "Are we setting is_major_gift even with a DM (non-MGE/F) appeal code? In other words, is all direct mail considered major gifts?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260161 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [15:27:34] but the proxy url still seems not to work [15:27:50] maybe if I restart the ssh connection? [15:29:39] oh maybe I need to open the port in horizon? [15:32:08] huh, I think all ports are allowed, at least from the proxy [15:33:54] (03CR) 10Ejegg: "Needs docker compose down email-pref-ctr && docker compose up -d email-pref-ctr" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 (owner: 10Ejegg) [15:35:18] jgleeson: want to try that locally and see if you get anything forwarded? ^ [15:35:31] sure [15:49:01] jgleeson: did you have a way to test donorportal locally via android? [15:49:27] yeah I used the chrome android and iphone viewport tools [15:50:18] ah ok [15:52:03] the routing is not working for me [15:52:11] oho, I think I found it [15:52:19] it was in the networking config in horizon [15:52:23] just not where I thought it was [15:52:48] it's in the bridge group, not the web group [15:56:43] ok, I added the 8200-8280 range to the security group [15:56:52] let's see if it takes effect immediately [15:57:06] woo, yep [15:57:22] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - Update error message copy + error code - https://phabricator.wikimedia.org/T421917#11773893 (10KHill-WMF) [15:57:51] yay, it's loading now jgleeson [15:58:04] awesome. I'll check too [15:58:10] now to look on the phone [16:00:54] 06Fundraising-Backlog, 06FR-donorrelations: Outreach to the donors from T420651 - https://phabricator.wikimedia.org/T421534#11773910 (10Ejegg) @SHust I'm having trouble doing that in SearchKit. I think I'll just do the query in SQL and deliver a CSV on the 'fileshare' folder in google drive if that works. I'll... [16:02:02] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 07fr-current-sprint, 07payments-orchestration: Recurring token migration - Paypal - https://phabricator.wikimedia.org/T405483#11773927 (10Slack-bot) 05Open→03In progress [16:02:42] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 07fr-current-sprint, 07payments-orchestration: Recurring token migration - Paypal - https://phabricator.wikimedia.org/T405483#11773935 (10Slack-bot) 05In progress→03Resolved [16:03:03] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 07fr-current-sprint, 07payments-orchestration: Recurring token migration - Paypal - https://phabricator.wikimedia.org/T405483#11773942 (10greg) 05Resolved→03In progress (oops, I clicked a button I wasn't sure what it did :) ) [16:03:46] (03PS2) 10Jgleeson: Fix donor portal mobile layout [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1262122 (https://phabricator.wikimedia.org/T420983) [16:04:02] (03CR) 10Ejegg: [C:03+2] "Looks much better in Android chrome and FF" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1262122 (https://phabricator.wikimedia.org/T420983) (owner: 10Jgleeson) [16:04:34] gonna lurk on that making space call while I get lunch [16:05:16] 03Fundraising Sprint - Floor is Lava, 10fundraising-tech-ops, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition: Create SFTP keypair for MediaWiki donor export - https://phabricator.wikimedia.org/T421785#11773963 (10CDanis) This should be working now: `deployment.eqiad.wmnet` user `... [16:05:43] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 10fundraising-tech-ops, 13Patch-For-Review: Allow SFTP outbound requests from civi1002/civi2002 to deployment.eqiad.wmnet - https://phabricator.wikimedia.org/T421751#11773967 (10JMeybohm) Removing the #serviceops_new tag, since this is being hand... [16:06:38] (03Merged) 10jenkins-bot: Fix donor portal mobile layout [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1262122 (https://phabricator.wikimedia.org/T420983) (owner: 10Jgleeson) [16:06:40] thanks ejegg|food ! [16:06:52] I'll deploy that fix [16:12:16] 06Fundraising-Backlog, 06FR-donorrelations: Outreach to the donors from T420651 - https://phabricator.wikimedia.org/T421534#11774038 (10SHust) Great idea, thanks, @Ejegg. I'll be waiting for the file so we can ensure the email is sent tomorrow at the latest. [16:14:08] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1265475 [16:14:54] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1265475 (owner: 10Jgleeson) [16:16:49] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1265475 (owner: 10Jgleeson) [16:17:55] (03PS1) 10Jgleeson: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265477 [16:21:49] 10fundraising-tech-ops, 06DC-Ops, 10ops-eqiad, 06SRE: Q3:rack/setup/install fransw100[23] - https://phabricator.wikimedia.org/T417295#11774108 (10Jgreen) passwords have been reset to the frack mgmt password [16:22:28] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265477 (owner: 10Jgleeson) [16:23:08] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265477 (owner: 10Jgleeson) [16:24:18] !log donorwiki updated from d79a98b5 to c2d03117 [16:24:19] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [16:27:42] Thank jgleeson and let me know which approach feels right to you guys, either add to validation or handling it in general error [16:38:26] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM email location type is not set to default when adding new email on contact summary - https://phabricator.wikimedia.org/T421935 (10Lars) 03NEW [16:38:36] !log tools upgraded from f605b570 to 161049ac [16:38:37] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [16:42:10] 06Fundraising-Backlog, 06FR-donorrelations: Frequency change dropdown in Civi - https://phabricator.wikimedia.org/T421937 (10AMJohnson) 03NEW [16:49:17] (03PS4) 10Wfan: Trigger failmail if a validation error not categorized [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1262356 (https://phabricator.wikimedia.org/T421155) [16:51:37] (03CR) 10Wfan: "two approachs:" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1262356 (https://phabricator.wikimedia.org/T421155) (owner: 10Wfan) [16:55:38] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog: Add appropriate consent/opt-in language to Payments form SMS capture - https://phabricator.wikimedia.org/T420271#11774364 (10Cstone) a:03Cstone [17:03:45] (03CR) 10Ejegg: "This is now working - I had to adjust port access rules in the 'bridge' group in the horizon project." [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 (owner: 10Ejegg) [17:26:49] 06Fundraising-Backlog, 06FR-donorrelations: Outreach to the donors from T420651 - https://phabricator.wikimedia.org/T421534#11774530 (10Ejegg) OK, @SHust, here's the list with the folks who have new recurring donations filtered out: https://docs.google.com/spreadsheets/d/1iePz8_uGeSuXxasziEyC2H_ULN3DtLX5cZLfPl... [17:38:40] (03PS1) 10Cstone: WIP: Add explain for sms optin [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1265510 (https://phabricator.wikimedia.org/T420271) [17:39:51] !log Standalone (IPN listener) SmashPig upgraded from abf8682a to 9af71a7c [17:39:53] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [17:41:20] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Repair contribution tracking records where utm_medium overwritten by 'Audit' - https://phabricator.wikimedia.org/T421035#11774617 (10Ejegg) I'm going to try to do this by getting the original tracking m... [17:46:06] (03PS2) 10Cstone: Add explain for sms optin [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1265510 (https://phabricator.wikimedia.org/T420271) [18:12:21] (03PS1) 10Jgleeson: Add OpenSSH private key format support to SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265521 (https://phabricator.wikimedia.org/T421772) [18:12:55] larssandergreen: that should add support for the updated key format ^ [18:20:05] ejegg: historically, I haven't used the proxy-forward script as I just used the direct SSH command from https://docs.google.com/spreadsheets/d/1Ng-L0qBWma1U_SHW_u-brtZQSv_PLntyPP9SuknW7Y0/edit?gid=0#gid=0 [18:20:35] however updating that separately seems silly so I'll switch to the script [18:20:53] I just ran `./proxy-forward.sh` with your patch and I'm getting [18:21:03] jack@debian:~/fundraising-dev$ ./proxy-forward.sh [18:21:03] ssh -f -N -o ServerAliveInterval=30 -o ServerAliveCountMax=3 -o ExitOnForwardFailure=yes -R 8001:localhost:9009 -R 8101:localhost:9006 -R 8201:localhost: payments-trixie.fr-tech-dev.eqiad1.wikimedia.cloud [18:21:03] Bad remote forwarding specification '8201:localhost:' [18:31:53] (03CR) 10Lars SG: [C:03+2] Add OpenSSH private key format support to SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265521 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [18:33:42] (03Merged) 10jenkins-bot: Add OpenSSH private key format support to SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265521 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [18:36:59] (03CR) 10Jgleeson: [V:03+2 C:03+2] "thanks! working for me" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1265447 (owner: 10Ejegg) [18:37:01] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - Update error message copy + error code - https://phabricator.wikimedia.org/T421917#11774916 (10Damilare) a:03Damilare [18:38:20] (03PS1) 10Jgleeson: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265526 [18:39:03] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265526 (owner: 10Jgleeson) [18:39:32] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265526 (owner: 10Jgleeson) [18:40:36] (03PS1) 10Wfan: Map for dlocal pix automatic [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265527 (https://phabricator.wikimedia.org/T419446) [18:40:44] !log tools upgraded from 161049ac to 9985e723 [18:40:45] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [18:41:06] (03PS2) 10Wfan: Map for dlocal pix automatic [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265527 (https://phabricator.wikimedia.org/T419446) [18:41:07] (03CR) 10CI reject: [V:04-1] Map for dlocal pix automatic [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265527 (https://phabricator.wikimedia.org/T419446) (owner: 10Wfan) [18:41:38] (03CR) 10CI reject: [V:04-1] Map for dlocal pix automatic [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265527 (https://phabricator.wikimedia.org/T419446) (owner: 10Wfan) [18:45:49] (03PS3) 10Wfan: Map for dlocal pix automatic [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265527 (https://phabricator.wikimedia.org/T419446) [18:50:14] (03PS2) 10Eileen: Put back backend_processor_txn_id [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1260822 (https://phabricator.wikimedia.org/T421282) [18:50:16] (03CR) 10Ejegg: [C:03+2] Put back backend_processor_txn_id [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1260822 (https://phabricator.wikimedia.org/T421282) (owner: 10Eileen) [18:50:29] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM email location type is not set to default when adding new email on contact summary - https://phabricator.wikimedia.org/T421935#11774947 (10Lars) [[ https://github.com/civicrm/civicrm-core/pull/35283 | Upstream PR ]] [18:51:08] (03Merged) 10jenkins-bot: Put back backend_processor_txn_id [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1260822 (https://phabricator.wikimedia.org/T421282) (owner: 10Eileen) [19:07:10] (03CR) 10Ejegg: [C:04-1] "I think we want to make code changes in the error parsing so that PS1 passes, not change the test" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1264722 (https://phabricator.wikimedia.org/T421155) (owner: 10Ejegg) [19:09:50] (03PS1) 10Eileen: Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265538 [19:10:49] 03Fundraising Sprint - Floor is Lava, 10fundraising-tech-ops, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition: Create SFTP keypair for MediaWiki donor export - https://phabricator.wikimedia.org/T421785#11775026 (10Dwisehaupt) 05Open→03Resolved Thanks @CDanis. User added to... [19:11:02] (03PS1) 10Eileen: Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1265540 [19:11:25] (03CR) 10Eileen: [C:03+2] Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265538 (owner: 10Eileen) [19:11:39] (03CR) 10Eileen: [C:03+2] Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1265540 (owner: 10Eileen) [19:13:01] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Donor-portal, 13Patch-For-Review: Donor Portal - Wikiminute video issues on mobile devices - https://phabricator.wikimedia.org/T420782#11775045 (10XenoRyet) p:05Medium→03High [19:27:16] 06Fundraising-Backlog, 07payments-orchestration: general gravy validation error as bad_request - https://phabricator.wikimedia.org/T421958 (10AnnWF) 03NEW [19:33:48] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - Update error message copy + error code - https://phabricator.wikimedia.org/T421917#11775096 (10MBeat33) If there are multiple error codes, DR team would appreciate any documentation on them if we can use them to guide donors. [19:38:04] (03CR) 10CI reject: [V:04-1] Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265538 (owner: 10Eileen) [19:40:15] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: Repair contribution tracking records where utm_medium overwritten by 'Audit' - https://phabricator.wikimedia.org/T421035#11775117 (10Ejegg) a:03Ejegg [19:42:08] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Recurring convert email failing due to donors email being on hold - https://phabricator.wikimedia.org/T419003#11775119 (10Ejegg) We can move the email sending to coworker and get it to stop blocking the recurring modify queue [19:47:55] (03PS1) 10Jgleeson: Add unit tests for OpenSSH private key parsing in SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265550 (https://phabricator.wikimedia.org/T421772) [19:48:07] ejegg: fail here is your smashpig patch - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1265538 [19:48:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM email location type is not set to default when adding new email on contact summary - https://phabricator.wikimedia.org/T421935#11775136 (10Lars) a:03Lars [19:49:33] (03PS2) 10Eileen: Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265538 [19:49:47] ah - but don't worry ejegg looks like an easy fix - I just added [19:55:21] (03CR) 10Eileen: [C:03+2] Upgrading wikimedia/smash-pig (v1.2.3.10 => v1.2.3.12) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265538 (owner: 10Eileen) [20:09:40] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - investigation required: recurring upgrade email activity tracking parameter issue - https://phabricator.wikimedia.org/T421960 (10KHill-WMF) 03NEW [20:14:43] (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/1265556 [20:15:07] (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/1265556 (owner: 10Eileen) [20:16:45] 06Fundraising-Backlog, 10FR-Donor-portal: Donor Portal - handling of PayPal donations while edits are not possible - https://phabricator.wikimedia.org/T421962 (10KHill-WMF) 03NEW [20:16:48] !log civicrm upgraded from c3cc3562 to b468301c [20:16:49] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [20:28:50] (03PS1) 10Eileen: Declare type for donations [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265558 [20:34:34] (03CR) 10Lars SG: [C:03+2] Declare type for donations [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265558 (owner: 10Eileen) [20:35:08] (03Merged) 10jenkins-bot: Declare type for donations [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1265558 (owner: 10Eileen) [20:40:03] (03PS1) 10Eileen: Upgrade smash-pig to v1.2.3.13 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265565 [20:40:53] (03PS1) 10Eileen: Upgrade smash-pig to v1.2.3.13 [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1265566 [20:47:23] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog, 10fundraising-tech-ops: Allow SFTP outbound requests from civi1002/civi2002 to deployment.eqiad.wmnet - https://phabricator.wikimedia.org/T421751#11775344 (10Dwisehaupt) Outbound requests are successful from the transport side. Leaving to @jgleeson... [21:04:21] ok eileen I have 1571 messages to replay after deduping [21:10:18] right, that's 5 fixed [21:10:27] and spot-checking, they look restored [21:10:51] still have their contribution_id [21:11:24] ok, going to dump the rest of the 1500+ [21:12:28] need to do kid pickup, but i'll look at the results later tonight [21:17:42] (03CR) 10Eileen: [C:03+2] Upgrade smash-pig to v1.2.3.13 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1265565 (owner: 10Eileen) [21:41:04] (03PS1) 10Jgleeson: Fix ssh-ed25519 public key parsing in SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265585 (https://phabricator.wikimedia.org/T421772) [21:49:21] dwisehaupt: that gets us the public key parsing ^ [21:49:28] & larssandergreen [21:49:45] do one of you wanna +2 that and I'll deploy and retest before dropping off [21:50:03] I added a unit test to confirm it works [21:51:55] looking [21:53:40] (03CR) 10Dwisehaupt: [C:03+2] Fix ssh-ed25519 public key parsing in SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265585 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [21:53:59] cool. i think that looks good. all set for you to merge and do the things. [21:56:39] ty [21:57:31] (03CR) 10Jgleeson: [C:03+2] "self-merging unit tests." [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265550 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [21:59:18] (03Merged) 10jenkins-bot: Add unit tests for OpenSSH private key parsing in SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265550 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [21:59:36] (03Merged) 10jenkins-bot: Fix ssh-ed25519 public key parsing in SFTP client [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1265585 (https://phabricator.wikimedia.org/T421772) (owner: 10Jgleeson) [22:00:59] (03PS1) 10Jgleeson: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265592 [22:01:12] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265592 (owner: 10Jgleeson) [22:01:42] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1265592 (owner: 10Jgleeson) [22:03:06] !log tools upgraded from 9985e723 to 986f7f83 [22:03:07] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [22:18:45] (03CR) 10Eileen: [C:03+2] Upgrade smash-pig to v1.2.3.13 [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1265566 (owner: 10Eileen) [22:38:16] (03CR) 10Cstone: [C:03+2] Initial Stripe Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1252306 (https://phabricator.wikimedia.org/T419628) (owner: 10Eileen) [22:40:46] (03PS6) 10Eileen: Set is_major_gift to true for imports [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260161 (https://phabricator.wikimedia.org/T409994) [22:40:57] larssandergreen: does ^^ look right [22:46:03] (03CR) 10Cstone: [C:03+2] Start saving our backend_processor_reversal_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1259287 (https://phabricator.wikimedia.org/T420527) (owner: 10Eileen) [23:08:40] (03Merged) 10jenkins-bot: Initial Stripe Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1252306 (https://phabricator.wikimedia.org/T419628) (owner: 10Eileen) [23:08:43] (03Merged) 10jenkins-bot: Start saving our backend_processor_reversal_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1259287 (https://phabricator.wikimedia.org/T420527) (owner: 10Eileen) [23:11:07] (03CR) 10CI reject: [V:04-1] Set is_major_gift to true for imports [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1260161 (https://phabricator.wikimedia.org/T409994) (owner: 10Eileen) [23:37:23] 03Fundraising Sprint - Floor is Lava, 14Fundraising Sprint: Vixen, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 4 others: Automate some Direct Mail white mail coding - https://phabricator.wikimedia.org/T386031#11775822 (10Lars) Note to self: I will also need to replicate / abstract out and r... [23:41:36] 06Fundraising-Backlog, 06FR-donorrelations: Outreach to the donors from T420651 - https://phabricator.wikimedia.org/T421534#11775833 (10SHust) @Ejegg or @AKanji-WMF, I have [[ https://docs.google.com/spreadsheets/d/1Yx3f8VKSX--zd8gcleDIhbsEaIRRP8aKlkF5jCPTEpA/edit?gid=0#gid=0 | shared a spreadsheet ]] with upd... [23:46:21] 03Fundraising Sprint - Floor is Lava, 14Fundraising Sprint: Vixen, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 4 others: Automate some Direct Mail white mail coding - https://phabricator.wikimedia.org/T386031#11775836 (10Lars) @RLewis I just want to confirm above when you say "a large numbe... [23:53:55] (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/1265626 [23:54:21] (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/1265626 (owner: 10Eileen) [23:55:27] (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/1265626 (owner: 10Eileen)