[00:32:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: payments/index.php not found or unable to stat - https://phabricator.wikimedia.org/T423539 (10AnnWF) 03NEW [01:03:03] (03PS11) 10MGChecker: Replace uses of ReplaceContext::getMain outside test code [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1271801 (https://phabricator.wikimedia.org/T423456) [01:08:38] (03PS5) 10Ejegg: Permissions check for refund form [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1268662 [01:08:38] (03PS6) 10Ejegg: SearchTask version of refund form [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1269005 (https://phabricator.wikimedia.org/T421277) [01:08:47] (03PS12) 10MGChecker: Replace uses of ReplaceContext::getMain outside test code [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1271801 (https://phabricator.wikimedia.org/T423456) [01:10:26] (03CR) 10CI reject: [V:04-1] Replace uses of ReplaceContext::getMain outside test code [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1271801 (https://phabricator.wikimedia.org/T423456) (owner: 10MGChecker) [01:57:17] 06Fundraising-Backlog, 03GNU England Shaker dresser, 13Patch-For-Review, 07payments-orchestration: Blik Payment Method- Poland - https://phabricator.wikimedia.org/T405606#11827909 (10Ejegg) I just realized we don't have Blik activated at Adyen, so I've asked them to enable it for sandbox & production. I al... [01:59:19] (03PS1) 10Ejegg: Blik via Gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272051 (https://phabricator.wikimedia.org/T405606) [02:01:25] (03CR) 10Lars SG: Set all batch import is_major_gift = true (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271973 (https://phabricator.wikimedia.org/T422221) (owner: 10Lars SG) [02:02:05] (03PS1) 10Ejegg: Add config for Blik via Gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272052 (https://phabricator.wikimedia.org/T405606) [02:30:36] (03PS1) 10Ejegg: Map XA => Pix in DLocal IPNs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272083 (https://phabricator.wikimedia.org/T419446) [02:30:58] wfan: ^^^ is a really easy fix [02:32:01] (03CR) 10Wfan: [C:03+2] Map XA => Pix in DLocal IPNs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272083 (https://phabricator.wikimedia.org/T419446) (owner: 10Ejegg) [02:33:29] (03Merged) 10jenkins-bot: Map XA => Pix in DLocal IPNs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272083 (https://phabricator.wikimedia.org/T419446) (owner: 10Ejegg) [02:37:33] 03Fundraising Sprint - Floor is Lava, 06Fundraising-Backlog: Update the recurring failure/cancellation logic - https://phabricator.wikimedia.org/T413905#11827932 (10Ejegg) We do add an activity for each failed recurring payment attempt, e.g. this one: https://civicrm.wikimedia.org/civicrm/activity?atype=172&a... [02:37:51] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1272092 [02:37:53] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1272092 (owner: 10Ejegg) [02:38:24] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1272092 (owner: 10Ejegg) [02:39:24] !log standalone (IPN listener) SmashPig upgraded from 61fee241 to f1b3f3d9 [02:39:25] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [03:13:00] (03PS1) 10Ejegg: WIP: Tool-assisted JS refactor [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272116 (https://phabricator.wikimedia.org/T383087) [03:15:22] (03PS7) 10Cstone: Send through $errorParameters['description'] [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1268304 [03:15:22] (03CR) 10CI reject: [V:04-1] WIP: Tool-assisted JS refactor [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272116 (https://phabricator.wikimedia.org/T383087) (owner: 10Ejegg) [03:15:53] (03CR) 10CI reject: [V:04-1] Send through $errorParameters['description'] [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1268304 (owner: 10Cstone) [03:16:56] (03PS8) 10Cstone: Send through $errorParameters['description'] [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1268304 [03:18:26] (03PS9) 10Cstone: Send through $errorParameters['description'] [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1268304 [06:17:12] PROBLEM - check_load on fransw2002 is CRITICAL: LOAD CRITICAL - total load average: 112.90, 72.73, 36.08 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [06:22:12] RECOVERY - check_load on fransw2002 is OK: LOAD OK - total load average: 7.17, 29.08, 27.02 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [11:26:07] (03PS1) 10EMcFarland: CentralNoticeHooks: Add onCentralNoticeBeforeHook [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1272666 [11:28:04] (03CR) 10CI reject: [V:04-1] CentralNoticeHooks: Add onCentralNoticeBeforeHook [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1272666 (owner: 10EMcFarland) [12:14:55] (03CR) 10Jgleeson: Send through $errorParameters['description'] (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1268304 (owner: 10Cstone) [12:55:19] (03CR) 10Jgleeson: [C:03+2] Add config for Blik via Gravy [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272052 (https://phabricator.wikimedia.org/T405606) (owner: 10Ejegg) [12:55:42] (03CR) 10Jgleeson: [C:03+2] Blik via Gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272051 (https://phabricator.wikimedia.org/T405606) (owner: 10Ejegg) [12:56:16] (03Merged) 10jenkins-bot: Blik via Gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1272051 (https://phabricator.wikimedia.org/T405606) (owner: 10Ejegg) [13:02:16] (03CR) 10Jgleeson: [C:03+2] "Working well for me. I added a small tweak to the other links to open in a new tab yesterday, so we could do that here, too, to avoid losi" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271916 (https://phabricator.wikimedia.org/T423462) (owner: 10Lars SG) [13:07:25] (03PS7) 10Aude: Check edit permission for insert close button and banner translation UI [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1268090 (https://phabricator.wikimedia.org/T420810) [13:08:00] (03PS1) 10Jgleeson: Open Internal Unsubscribe link in new tab [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272709 (https://phabricator.wikimedia.org/T423462) [13:10:27] (03CR) 10Jgleeson: [C:03+2] "Working well for me when testing with the above" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271099 (https://phabricator.wikimedia.org/T420624) (owner: 10Lars SG) [13:25:15] (03CR) 10Jgleeson: "This could be interesting. We could point different tools/friends at the same file with the same prompt and compare output quality." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272116 (https://phabricator.wikimedia.org/T383087) (owner: 10Ejegg) [13:34:18] (03Merged) 10jenkins-bot: Show all relevant opt out options on unsubscribe form [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271099 (https://phabricator.wikimedia.org/T420624) (owner: 10Lars SG) [13:37:01] (03Merged) 10jenkins-bot: Add link from Civi contact summary to unsub form with pre-filled email address [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271916 (https://phabricator.wikimedia.org/T423462) (owner: 10Lars SG) [13:44:57] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Provide API for key CiviCRM data by email address to load in Zendesk - https://phabricator.wikimedia.org/T421245#11829652 (10Lars) See also [[ https://docs.google.com/document/d/1NTCbulNznYUuNPZ5sCmEFBWVCFAQGJc6d6A0zWtOaaQ/edit?tab=... [13:57:06] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Provide API for key CiviCRM data by email address to load in Zendesk - https://phabricator.wikimedia.org/T421245#11829713 (10Lars) Later, DR would like to tag messages based on data from Civi. It looks like we can do this with [[ ht... [13:58:10] 06Fundraising-Backlog: Make SFTP transfer atomic - https://phabricator.wikimedia.org/T423600 (10jgleeson) 03NEW [13:58:56] 06Fundraising-Backlog: Make SFTP transfer atomic - https://phabricator.wikimedia.org/T423600#11829731 (10jgleeson) [13:58:57] 03Fundraising Sprint - Floor is Lava, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition, 13Patch-For-Review: Add SFTP upload support to MediaWiki donor export job - https://phabricator.wikimedia.org/T421772#11829730 (10jgleeson) [13:59:13] (03PS1) 10Jgleeson: Upload mediawiki donor export atomically via SFTP rename [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1272723 (https://phabricator.wikimedia.org/T423600) [13:59:39] thanks for all the review jgleeson [13:59:45] np! [14:03:23] (03CR) 10Lars SG: "Not sure this would be desirable for this particular link as it would be something DR would do and I don't think they would want to keep t" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272709 (https://phabricator.wikimedia.org/T423462) (owner: 10Jgleeson) [14:08:45] (03PS1) 10Lars SG: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272727 [14:12:30] (03PS1) 10Lars SG: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1272730 [14:13:49] (03CR) 10Lars SG: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1272730 (owner: 10Lars SG) [14:13:52] (03CR) 10Lars SG: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272727 (owner: 10Lars SG) [14:14:18] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272727 (owner: 10Lars SG) [14:14:46] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1272730 (owner: 10Lars SG) [14:19:51] !log tools upgraded from 9bff5f07 to f14a814e [14:19:52] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [14:20:51] !log civicrm upgraded from 801847a7 to 90c0ccd9 [14:20:52] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [14:23:11] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Can't preview monthly convert email in CiviCRM - https://phabricator.wikimedia.org/T423182#11829886 (10Lars) @ppenloglou this has been fixed. A string was trying to pretend to be an integer and things were breaking. [14:26:25] 06Fundraising-Backlog: SEPA recurring donation created via Adyen and not Gr4vy - https://phabricator.wikimedia.org/T423604 (10RKumar_WMF) 03NEW [14:39:06] Jeff_Green: I just deployed a change that will hopefully get us distinguisable stats for the two differen recurring charge jobs [14:39:20] should start seeing job="newer" and job="older" labels [14:39:26] as well as outputting to different files [14:39:38] not that we have many charges left for today... [14:39:58] ejegg: cool [14:43:53] ejegg what are the file names? I see recurring_smashpig_.prom, that's the only one with recurring in the filename [14:43:53] [14:54:15] Jeff_Green: oh shoot, that seems like an error :( [14:54:25] let me see what's happening there [14:55:14] fr-tech is there any reason we wouldn't want to route recurring SEPA donors to gravy? Looks like rtbt/sepadirectdebit/recurring gets you an Adyen form currently [14:57:15] huh, rtbt has recurring: false at the top level but both listed submethods have recurring: true [14:57:42] well, I thought submethod-level should override method-level, but I guess we might have missed applying that to recurring [14:59:24] (03PS1) 10Ejegg: Mark gravy/rtbt as supporting recurring [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272748 [15:16:54] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: API4 Request Authorization failed: UserJob::get on cv flush - https://phabricator.wikimedia.org/T422425#11830285 (10Lars) 05Open→03Invalid User error [15:29:30] FIRING: GnmiTargetDown: fasw1-f5b-codfw is unreachable through gNMI - https://wikitech.wikimedia.org/wiki/Network_telemetry#Troubleshooting - https://grafana.wikimedia.org/d/eab73c60-a402-4f9b-a4a7-ea489b374458/gnmic - https://alerts.wikimedia.org/?q=alertname%3DGnmiTargetDown [15:34:18] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM session id log noise - https://phabricator.wikimedia.org/T423618 (10Lars) 03NEW [15:34:27] RESOLVED: GnmiTargetDown: fasw1-f5b-codfw is unreachable through gNMI - https://wikitech.wikimedia.org/wiki/Network_telemetry#Troubleshooting - https://grafana.wikimedia.org/d/eab73c60-a402-4f9b-a4a7-ea489b374458/gnmic - https://alerts.wikimedia.org/?q=alertname%3DGnmiTargetDown [15:51:09] (03CR) 10Lars SG: [C:03+2] Permissions check for refund form [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1268662 (owner: 10Ejegg) [16:05:36] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 03GNU England Shaker dresser: Add is_eligible_for_donor_portal bool to Acoustic export - https://phabricator.wikimedia.org/T422670#11830541 (10Lars) ` ALTER TABLE silverpop_export... [16:10:03] (03CR) 10Ejegg: "recheck" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272052 (https://phabricator.wikimedia.org/T405606) (owner: 10Ejegg) [16:14:12] (03PS1) 10Lars SG: Fix is_eligible_for_donor_portal to give "Yes" or "No" as required by Acoustic [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1272772 (https://phabricator.wikimedia.org/T422670) [16:16:11] (03Merged) 10jenkins-bot: Permissions check for refund form [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1268662 (owner: 10Ejegg) [16:16:44] ejegg: could you take a quick look at ^^^. Realized I forgot about Acoustic's "Yes"/"No" string fake bool [16:16:53] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions at Authenticating in Gr4vy and not visible at Adyen? - https://phabricator.wikimedia.org/T423328#11830644 (10AMJohnson) Gotcha. Thank you for sharing this context, @Ejegg! So it sounds like donations being Gr4vy but not... [16:19:27] 06Fundraising-Backlog, 10fundraising-tech-ops, 10Wikimedia-Fundraising-CiviCRM, 03GNU England Shaker dresser, 13Patch-For-Review: Add is_eligible_for_donor_portal bool to Acoustic export - https://phabricator.wikimedia.org/T422670#11830651 (10Jgreen) 05Open→03Resolved [16:22:29] 06Fundraising-Backlog, 10fundraising-tech-ops, 10Wikimedia-Fundraising-CiviCRM, 03GNU England Shaker dresser, 13Patch-For-Review: Add is_eligible_for_donor_portal bool to Acoustic export - https://phabricator.wikimedia.org/T422670#11830675 (10Lars) Thanks @Jgreen [16:26:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions at Authenticating in Gr4vy and not visible at Adyen? - https://phabricator.wikimedia.org/T423328#11830708 (10Ejegg) Yes @AMJohnson that sounds like good advice. [16:29:03] oh lol. T419003 is not about the recurring upgrade email, it's the monthly convert TY email. [16:29:04] T419003: Recurring convert email failing due to donors email being on hold - https://phabricator.wikimedia.org/T419003 [16:29:45] (03CR) 10Ejegg: [C:03+2] "D'oh! Sorry I missed this in the earlier review." [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1272772 (https://phabricator.wikimedia.org/T422670) (owner: 10Lars SG) [16:31:37] (03Merged) 10jenkins-bot: Fix is_eligible_for_donor_portal to give "Yes" or "No" as required by Acoustic [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1272772 (https://phabricator.wikimedia.org/T422670) (owner: 10Lars SG) [16:34:10] (03PS1) 10Lars SG: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272778 [16:34:17] let's see, maybe the env var needs quotes? [16:38:00] (03CR) 10Lars SG: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272778 (owner: 10Lars SG) [16:38:38] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1272778 (owner: 10Lars SG) [16:39:35] !log tools upgraded from f14a814e to 26ab0125 [16:39:37] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [16:50:14] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions at Authenticating in Gr4vy and not visible at Adyen? - https://phabricator.wikimedia.org/T423328#11830772 (10AMJohnson) Thank you, @Ejegg! [17:16:01] (03PS3) 10Ejegg: Update images for new composer version [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271868 [17:16:23] larssandergreen: ^^^ updates all the images in the mac version to bookworm [17:18:22] huh, quotes in the environment var aren't helping [17:23:07] 06Fundraising-Backlog, 06Reader Experience Team: Be able to avoid showing donate links in certain countries - https://phabricator.wikimedia.org/T423629 (10Pcoombe) 03NEW [17:36:06] (03CR) 10Ejegg: [V:03+2] build: Updating npm dependencies [extensions/DonationInterface] (REL1_45) - 10https://gerrit.wikimedia.org/r/1252866 (owner: 10Libraryupgrader) [17:57:57] (03PS2) 10Ejegg: Mark gravy/rtbt as supporting recurring [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1272748 (https://phabricator.wikimedia.org/T423604) [18:08:02] 06Fundraising-Backlog, 06Reader Experience Team, 10WikimediaMessages: Be able to avoid showing donate links in certain countries - https://phabricator.wikimedia.org/T423629#11831051 (10Jdlrobson-WMF) [18:09:13] 06Fundraising-Backlog, 06Reader Experience Team, 10Wikimedia-Site-requests, 10WikimediaMessages, 03Readers Essential Work: Be able to avoid showing donate links in certain countries - https://phabricator.wikimedia.org/T423629#11831054 (10Jdlrobson-WMF) @SToyofuku-WMF I assume we can configure wgWikimedia... [18:09:21] 06Fundraising-Backlog, 06Reader Experience Team, 10Wikimedia-Site-requests, 10WikimediaMessages, 03Readers Essential Work: Be able to avoid showing donate links in certain countries - https://phabricator.wikimedia.org/T423629#11831056 (10Jdlrobson-WMF) [18:45:17] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Paypal, 10Recurring-Donations, 07payments-orchestration: Check on handling of payment-method.deleted IPNs from Gravy - https://phabricator.wikimedia.org/T423638 (10Ejegg) 03NEW [18:46:51] 06Fundraising-Backlog, 10fundraising-tech-ops, 10Wikimedia-Fundraising-CiviCRM, 03GNU England Shaker dresser: Add is_eligible_for_donor_portal bool to Acoustic export - https://phabricator.wikimedia.org/T422670#11831222 (10Lars) 05Resolved→03Open [19:07:41] looks like using the beta Docker VMM causes our container to fail with a QEMU SIGSEGV error even with more than enough cpu/ram resources [19:08:08] would have to go back to the apple virtualization framework option [19:13:33] sounds intriguing damilare [19:14:47] yea I moved to that version because of the experiment to use bind mounts for civi as it was faster than the apple virtualization framework... [19:15:09] I saw other complaints on the forums but seems increasing the resources solved them [19:15:18] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Recurring-Donations: Fix stats for smashpig recurring charge job - https://phabricator.wikimedia.org/T423196#11831364 (10Ejegg) FR-tech I'm a bit stumped. I think I'm passing the environment variables into the process-control job the same way we pass... [19:20:42] fr-tech I did a little rubber-ducking on the last comment ^^^. If anyone wants to take a look and see if I'm missing something obvious it would be nice to get those stats fixed [19:21:10] gonna run a quick errand [19:45:53] (03CR) 10Damilare Adedoyin: [C:03+2] Update images for new composer version [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271868 (owner: 10Ejegg) [19:46:45] (03CR) 10Damilare Adedoyin: [C:03+2] "Thanks, did a full reinstall of the services on a mac and all were successful. LGTM!" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271868 (owner: 10Ejegg) [19:46:49] (03CR) 10Damilare Adedoyin: [V:03+2 C:03+2] Update images for new composer version [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271868 (owner: 10Ejegg) [19:47:10] (03PS3) 10Damilare Adedoyin: WIP: Add Volume mounts optional for mac users [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271073 [20:08:54] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM session id log noise - https://phabricator.wikimedia.org/T423618#11831531 (10Lars) It looks like we just need to update cv again. @Eileenmcnaughton you did that recently, do we have a process worked out for that... [20:16:47] thanks damilare ! [20:17:03] np! [20:28:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Recurring-Donations: Fix stats for smashpig recurring charge job - https://phabricator.wikimedia.org/T423196#11831580 (10jgleeson) > `lang=php > ... > $statsSuffix = getenv('SMASHPIG_JOB_LABEL') ?? ''; > if ($statsSuffix !== '') { > $statsSuff... [20:34:32] (03PS4) 10Damilare Adedoyin: Make CiviCRM Volume mounts optional for mac users [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1271073 [20:36:54] thanks jgleeson ! I'll update those jobs with -E to preserve the env variables [20:37:06] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Recurring-Donations: Fix stats for smashpig recurring charge job - https://phabricator.wikimedia.org/T423196#11831656 (10jgleeson) Here's a quick test to confirm the sudo env var stripping ` jack@debian:~/Projects/wiki/safe-assistant$ FOO=bar sud... [20:37:24] (and replace that ?? '' with a check for false to get rid of the stray '_') [20:37:47] np [20:40:01] ohhh wait, it needs to be puppetized [20:40:41] Jeff_Green: does it seem safe to pass -E to the sudo -u command in the wmf-cv wrapper? [20:41:17] Jack has just figured out that all the environment variables we're trying to pass in from process-control are not getting through to the underlying cv command [20:41:49] which among other things would seem to be routing all mail through the default server instead of to the per-job server as we thought we were doing [20:42:00] but also explains why we don [20:42:17] * Jeff_Green looking at wrapper... [20:42:18] 't get the expected smashpig stats output labels / file names [20:45:36] (03PS1) 10Ejegg: Fix check for SMASHPIG_JOB_LABEL in output filename [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272890 [20:45:57] ok jgleeson there's the lil fix to avoid a stray '_' ^^ [20:47:33] ejegg: are there a modest number of variables and is the list predictable? b/c it looks like we can pass specific variables through sudo by command [20:48:23] Defaults!/srv/cv/bin/cv env_keep += "VAR1 VAR2 VAR3..." [20:48:25] Jeff_Green: currently we are only passing SMASHPIG_JOB_LABEL in one job and CIVICRM_SMTP_HOST in another [20:48:38] so just having those would be a good start [20:49:03] ok, lemme see about puppetizing this... [20:52:32] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272890 (owner: 10Ejegg) [20:53:14] thanks jgleeson [20:54:46] Jeff_Green: when run from process-control we have an additional layer of env protection, as process-control only passes through the variables defined in its config [20:55:16] but I guess since we can all run wmf-cv directly you'll still want to control that list [20:56:49] ok, there may be better ways to do it, I was just looking at options [20:57:29] this seems most predictable b/c I think it wouldn't wipe the target user's normal environment, just the variables you pass through [21:00:16] (03PS1) 10Ejegg: WIP send recurring convert email via coworker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272893 (https://phabricator.wikimedia.org/T419003) [21:00:22] cstone: here's a WIP patch to use the standard workflow for the convert email ^^^ [21:01:07] It's currently failing since that $params list is not quite what the TY-based code ends up passing to render() [21:01:22] but I think I can get it into shape [21:03:09] (03CR) 10Ejegg: "Oh, with this change I can maybe delete the RecurUpgradeEmail API actions" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271917 (https://phabricator.wikimedia.org/T419003) (owner: 10Ejegg) [21:11:04] np! [21:16:50] kidpickup time, back later [21:21:44] (03CR) 10CI reject: [V:04-1] WIP send recurring convert email via coworker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272893 (https://phabricator.wikimedia.org/T419003) (owner: 10Ejegg) [21:25:14] (03CR) 10Eileen: [C:03+2] Set all batch import is_major_gift = true [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271973 (https://phabricator.wikimedia.org/T422221) (owner: 10Lars SG) [21:25:54] ejegg|afk jgleeson wmf-cv env is done, lemme know how it goes [21:59:55] I missed that - what did we change? [22:00:39] oh I see [22:10:58] thanks Jeff_Green ! [22:20:23] (03PS2) 10Ejegg: Send recurring convert email via coworker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272893 (https://phabricator.wikimedia.org/T419003) [22:20:48] cstone: I got it working! ^^^ [22:21:15] using vanilla WorkflowMessage::send [22:21:20] gotta help with dinner [22:21:28] nice ejegg|food [22:41:19] (03CR) 10CI reject: [V:04-1] Send recurring convert email via coworker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1272893 (https://phabricator.wikimedia.org/T419003) (owner: 10Ejegg) [23:08:41] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations, 10FR-MGFE: Add link in Zendesk to contact in CiviCRM - https://phabricator.wikimedia.org/T415378#11832030 (10Lars) @AStein-WMF Do you want to tackle the Zendesk App side of this or should I/Claude take a crack at it? We now have t... [23:18:28] (03PS4) 10Lars SG: Add token for recurring donation failure link [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271005 (https://phabricator.wikimedia.org/T419046) [23:18:46] (03CR) 10CI reject: [V:04-1] Add token for recurring donation failure link [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271005 (https://phabricator.wikimedia.org/T419046) (owner: 10Lars SG) [23:20:13] (03PS5) 10Lars SG: Add token for recurring donation failure link [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271005 (https://phabricator.wikimedia.org/T419046) [23:20:29] (03CR) 10CI reject: [V:04-1] Add token for recurring donation failure link [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271005 (https://phabricator.wikimedia.org/T419046) (owner: 10Lars SG) [23:20:33] (03PS6) 10Lars SG: Add token for recurring donation failure link [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1271005 (https://phabricator.wikimedia.org/T419046) [23:32:45] 06Fundraising-Backlog, 10fundraising-tech-ops, 10FR-Tech-Analytics: Grant Maya Kampurath access to the dbt repo - https://phabricator.wikimedia.org/T423466#11832072 (10Mayakp.wiki) hi @Dwisehaupt , read only access is good. I dont anticipate needing to make changes to the FR repo. thanks! gerrit user name i... [23:41:43] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations, 10FR-MGFE, 03GNU England Shaker dresser: Add link in Zendesk to contact in CiviCRM - https://phabricator.wikimedia.org/T415378#11832084 (10Lars) [23:42:13] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations, 10FR-MGFE, 03GNU England Shaker dresser: Add link in Zendesk to contact in CiviCRM - https://phabricator.wikimedia.org/T415378#11832086 (10Lars) a:03Lars [23:43:31] (03PS3) 10Wfan: Backfill contact with special location type [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1269791 (https://phabricator.wikimedia.org/T420992) [23:49:17] (03PS4) 10Wfan: Backfill contact with special location type [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1269791 (https://phabricator.wikimedia.org/T420992)