[00:30:48] 10fundraising-tech-ops: Deactivate fundraising accounts for test_user - https://phabricator.wikimedia.org/T332654 (10Dwisehaupt) [00:32:07] 10fundraising-tech-ops: Deactivate fundraising accounts for test_user - https://phabricator.wikimedia.org/T332654 (10Dwisehaupt) 05Open→03Declined a:03Dwisehaupt Test of offboarding template automation. [00:49:08] (03PS2) 10Eileen: Apply civicrm code formatting, no other change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898030 (https://phabricator.wikimedia.org/T288585) [01:22:02] 10Fundraising-Backlog: import contact with 4 parts of addressee_display, find our if add suffix or middle name - https://phabricator.wikimedia.org/T326585 (10XenoRyet) [01:22:09] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: DB Error 'No such field' when sorting Civi advanced search results by country - https://phabricator.wikimedia.org/T324288 (10XenoRyet) [01:22:22] I might make wikibugs get kicked again [01:23:59] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Delete blank contacts - https://phabricator.wikimedia.org/T322887 (10XenoRyet) [01:24:13] 10Fundraising-Backlog: Recurring Charge Processor throwing Failed to write to socket: unknown error - https://phabricator.wikimedia.org/T281439 (10XenoRyet) [01:24:21] 10Fundraising-Backlog: ApplePay token amount-mismatch - https://phabricator.wikimedia.org/T324413 (10XenoRyet) [01:24:31] 10Fundraising-Backlog, 10Documentation: Document Donor Data Delivery Tool - https://phabricator.wikimedia.org/T323144 (10XenoRyet) [01:25:00] 10Fundraising-Backlog: Transaction not found in Adyen. Error Reference 157787384.1 - https://phabricator.wikimedia.org/T328344 (10XenoRyet) [01:25:24] 10Wikimedia-Fundraising-CiviCRM: Civi DB Error: constraint violation detected from civicrm_mailing_event_delivered from recurring_queue_consume log - https://phabricator.wikimedia.org/T319445 (10XenoRyet) [01:25:34] 10Fundraising-Backlog, 10FR-Alerts: Update the frequency of smart-group cache refresh - https://phabricator.wikimedia.org/T321337 (10XenoRyet) [01:26:22] 10Fundraising-Backlog: Something is causing a lot of deadlocks in ConfigAndLog - https://phabricator.wikimedia.org/T320287 (10XenoRyet) [01:53:08] 10Fundraising-Backlog: Payments main cache backend is not using memcached - https://phabricator.wikimedia.org/T324693 (10XenoRyet) [01:53:27] 10Fundraising-Backlog, 10Documentation: Complete Adyen Checkout documentation to include donation statuses for all stages/permutations - https://phabricator.wikimedia.org/T316583 (10XenoRyet) [01:53:50] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10Patch-For-Review, 10Security, 10User-revi: &banner causes CSP warning - https://phabricator.wikimedia.org/T199055 (10XenoRyet) [01:53:53] 10Fundraising Sprint Amazing grep, 10Fundraising-Backlog: Bus error 2: The bus strikes back - https://phabricator.wikimedia.org/T326013 (10XenoRyet) [01:54:08] 10Fundraising-Backlog: Docker dev setup: Fundraising process-control cron - https://phabricator.wikimedia.org/T328506 (10XenoRyet) [01:55:05] 10Fundraising-Backlog, 10fr-donorservices: questions re: Civi Recur fail email Jan 2022 - https://phabricator.wikimedia.org/T326770 (10XenoRyet) [01:55:10] 10Fundraising-Backlog: Add gateway label to 'IP repeat count' grafana graph - https://phabricator.wikimedia.org/T327958 (10XenoRyet) [01:55:34] 10Fundraising-Backlog, 10MediaWiki-extensions-CLDR, 10Language-Team (Language-2023-January-March), 10ci-test-error (WMF-deployed Build Failure): quibble-fundraising-cldr-REL1_39-php74-docker failure on CLDR master - https://phabricator.wikimedia.org/T328240 (10XenoRyet) [01:55:49] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10Documentation: Update deduper documentation to explain the critieria better - https://phabricator.wikimedia.org/T327727 (10XenoRyet) [01:55:57] 10Fundraising-Backlog: Add a time limit to contribution tracking queue consume - https://phabricator.wikimedia.org/T329600 (10XenoRyet) [01:56:36] 10Fundraising-Backlog: Get all fr-tech engineers added to the WMF Developer Account - https://phabricator.wikimedia.org/T329724 (10XenoRyet) [01:56:43] 10Fundraising-Backlog, 10FR-Adyen: Better handle terse Adyen idempotency result - https://phabricator.wikimedia.org/T331319 (10XenoRyet) [01:56:51] 10Fundraising-Backlog, 10FR-Adyen: Handle issuer timeouts via Adyen - https://phabricator.wikimedia.org/T331317 (10XenoRyet) [06:53:45] 10Fundraising-Backlog: SmashPig::queue2civicrm find a completed dlocal payment mark as failed and removed from queue table - https://phabricator.wikimedia.org/T309799 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at proj... [06:53:47] 10Fundraising-Backlog, 10Documentation: Document which tables are hit by each process-control job - https://phabricator.wikimedia.org/T322243 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searc... [06:53:49] 10Fundraising-Backlog: Donor directed to "Thank You" page even though payment rejected - https://phabricator.wikimedia.org/T310383 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for task... [06:53:51] 10Fundraising-Backlog: php_network_getaddresses: getaddrinfo failed: Temporary failure in name resolution - https://phabricator.wikimedia.org/T318956 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or... [06:53:53] 10Fundraising-Backlog: Intermittent Apple Pay timeouts when testing locally - https://phabricator.wikimedia.org/T318808 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for tasks in certai... [06:53:55] 10Fundraising-Backlog: AmazonPay script errors when testing locally - https://phabricator.wikimedia.org/T318732 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for tasks in certain projec... [06:53:58] 10Fundraising-Backlog: Adyen: Client side error on Safari even with successful donation - https://phabricator.wikimedia.org/T310737 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for tas... [06:54:00] 10Fundraising-Backlog: Investigate timeouts from Adyen Soap listener - https://phabricator.wikimedia.org/T317422 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for tasks in certain proje... [06:54:06] 10Fundraising-Backlog, 10Documentation: Documentation: Banner data flow and data stores - https://phabricator.wikimedia.org/T296222 (10Aklapper) @XenoRyet: Please associate at least one active team/codebase project tag with a task, so people can find a task when looking at project workboards or searching for t... [09:21:45] 10Fundraising-Backlog, 10FR-Docker: Docker dev setup: Set up process control - https://phabricator.wikimedia.org/T268685 (10Aklapper) a:05jgleeson→03None @jgleeson: Removing task assignee as this open task has been assigned for more than two years - See the email sent to task assignee on Feburary 22nd, 202... [09:30:02] 10Fundraising-Backlog, 10FR-Civi-Prospect, 10Fr-endowment: Need a very special group created - https://phabricator.wikimedia.org/T271333 (10Aklapper) a:05Eileenmcnaughton→03None @Eileenmcnaughton: Removing task assignee as this open task has been assigned for more than two years - See the email sent to t... [09:30:18] 10Fundraising-Backlog: Donors are being removed from Parent Groups by Admin - https://phabricator.wikimedia.org/T261960 (10Aklapper) a:05Eileenmcnaughton→03None @Eileenmcnaughton: Removing task assignee as this open task has been assigned for more than two years - See the email sent to task assignee on Febur... [09:30:26] 10Fundraising Sprint Vagranty McVagrantface, 10Fundraising Sprint We all meet again, 10Fundraising-Backlog, 10fr-matching-gifts: matching gifts values don't match between databases - https://phabricator.wikimedia.org/T266475 (10Aklapper) a:05Eileenmcnaughton→03None @Eileenmcnaughton: Removing task assi... [09:30:30] 10Fundraising Sprint Airline Passenger Experience, 10Fundraising Sprint Yellow hornets of kindness and healing, 10Fundraising Sprint Zeitgeistbusters, 10Fundraising-Backlog, and 2 others: As an email stats updater, I need to find results for a send that is missing ... - https://phabricator.wikimedia.org/T269811 [09:30:34] 10Fundraising Sprint Pseudopretzels, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe: Retrospectively fix mailing_provider data (to the extent possible) for merged contacts - https://phabricator.wikimedia.org/T261264 (10Aklapper) a:05Eileenmcnaughton→03None @Eileenmcnaughton: R... [09:45:50] 10Fundraising Sprint Corrugated super slide, 10Fundraising-Backlog, 10FR-Docker: Fix Docker performance on Mac - https://phabricator.wikimedia.org/T274339 (10Aklapper) a:05AndyRussG→03None @AndyRussG: Removing task assignee as this open task has been assigned for more than two years - See the email sent... [09:45:55] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: Still one more small fix needed for Campaign fallback: local campaign variable. - https://phabricator.wikimedia.org/T236845 (10Aklapper) a:05AndyRussG→03None @AndyRussG: Removing task assignee as this open task has been assigned for more than two... [12:35:29] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Delete pending messages in the recurring queue consumer - https://phabricator.wikimedia.org/T332690 (10jgleeson) [12:44:29] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: PaymentsWiki Error: ResizeObserver loop limit exceeded - https://phabricator.wikimedia.org/T332693 (10jgleeson) [12:44:32] (03PS39) 10Damilare Adedoyin: IP Velocity Filter: Increase count after first run [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/859168 (https://phabricator.wikimedia.org/T323544) (owner: 10AndyRussG) [12:47:51] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: PaymentsWiki Error: ResizeObserver loop limit exceeded - https://phabricator.wikimedia.org/T332693 (10jgleeson) Ok, I think we can ignore this. It looks like it's a non-error error :| according to https://stackoverflow.com/questions/49384120/resizeobserve... [12:48:39] (03PS2) 10Damilare Adedoyin: Remove watchdog from refund_queue_consume drush [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898031 (owner: 10Eileen) [12:52:44] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) [12:53:48] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) a:03jgleeson [12:54:27] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Recurring-Donations, 10fr-donorservices: Ingenico recurrings stopped at status 600 March 5th - https://phabricator.wikimedia.org/T331490 (10jgleeson) a:05jgleeson→03None [12:54:35] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) p:05Triage→03High [14:23:53] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) My current working assumption for this one is that when we get a rejection, we want to delete any relatd pending message and log the rejection. [14:36:36] (03CR) 10Damilare Adedoyin: [C: 03+2] Apply civicrm code formatting, no other change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898030 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [14:38:26] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) It looks like the REJECTED IPN messages don't come with the Auth header we expect for the others. We've had a few in the logs triggering this error : `... [14:50:51] (03Merged) 10jenkins-bot: Apply civicrm code formatting, no other change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898030 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [15:39:45] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10Ejegg) >>! In T332637#8714979, @jgleeson wrote: > My current working assumption for this one is that when we get a rejection, we want to delete any relatd pendin... [15:39:54] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe: Revisit deduper interface with a view to making it possible to do a search in search kit to dedupe - https://phabricator.wikimedia.org/T327728 (10AKanji-WMF) @Eileenmcnaughton noting that in conversation with @SHust this was raised as... [15:40:26] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10AKanji-WMF) @Eileenmcnaughton noting that in conversation with @SHust this was also raised as a DR priority [15:42:49] 10Fundraising Sprint Fish HEAD^, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10AKanji-WMF) [15:43:56] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: Handle REJECTED IPNS for Dlocal - https://phabricator.wikimedia.org/T332637 (10jgleeson) Thanks @Ejegg [15:45:04] XenoRyet: heh, see phab updates from Andre around 11:35 last night (pacific time), there's going to be a bunch of weird/old tasks in the Triage column, maybe worth doing a pre-triage so we don't all have to review them (tl;dr: those tasks somehow lost all tags/projects in our process, so andre added #fr-backlog) [15:45:15] 10Fundraising-Backlog, 10FR-Civi-Dedupe: Alternate name field for donors with different alphabet characters - https://phabricator.wikimedia.org/T275678 (10AKanji-WMF) @XenoRyet I spoke with @SHust about this - let's chat about timing [16:18:17] 10Wikimedia-Fundraising-Banners, 10Wikipedia-iOS-App-Backlog, 10Patch-For-Review: Japan fundraising messages in iOS app - https://phabricator.wikimedia.org/T332229 (10Seddon) [16:21:30] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog, 10Patch-For-Review: Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10Seddon) [16:22:00] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog, 10Patch-For-Review: Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10Seddon) [16:22:55] 10Wikimedia-Fundraising-Banners, 10Patch-For-Review, 10Wikipedia-Android-App-Backlog (Android Release FY2022-23): Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10LGoto) p:05Triage→03High [16:41:50] (03PS1) 10Damilare Adedoyin: Remove Prenotify from non recurring charge Hosted params [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901669 [16:45:55] (03PS2) 10Damilare Adedoyin: Remove Prenotify from non recurring charge Hosted params [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901669 [16:48:10] ejegg I remember you may have mentioned this ^ [16:50:04] thanks, damilare ! will take a look in a sec [16:50:59] tnx! [17:07:33] (03CR) 10Ejegg: [C: 03+2] Remove Prenotify from non recurring charge Hosted params [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901669 (owner: 10Damilare Adedoyin) [17:07:45] looks good for the initial setup call. [17:08:42] (03Merged) 10jenkins-bot: Remove Prenotify from non recurring charge Hosted params [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901669 (owner: 10Damilare Adedoyin) [17:08:46] I think we'll need to add it back for the recurring installment, but I guess it will have to be optional [17:10:14] since we will send prenotify=true on the first try, but then prenotify=false on a retry [17:11:16] (03PS6) 10Ejegg: Add 'wmf_token' CSRF protection to dLocal forms [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/896106 (https://phabricator.wikimedia.org/T330778) (owner: 10Damilare Adedoyin) [17:11:20] (03CR) 10Ejegg: [C: 03+2] Add 'wmf_token' CSRF protection to dLocal forms [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/896106 (https://phabricator.wikimedia.org/T330778) (owner: 10Damilare Adedoyin) [17:11:31] ohh yes it is in the recurring chage param check out RecurringChargeHostedPaymentApiRequestMapper., is that what you mean? [17:11:46] ah i see [17:11:50] not sure about the false on retry [17:11:52] thanks, that makes sense [17:12:22] right, we haven't built the retry logic yet [17:12:39] ohh ok [17:13:24] (03Merged) 10jenkins-bot: Add 'wmf_token' CSRF protection to dLocal forms [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/896106 (https://phabricator.wikimedia.org/T330778) (owner: 10Damilare Adedoyin) [17:13:59] (03PS3) 10Ejegg: Dlocal: Stop mapping submethod to payment_method_id [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900450 (https://phabricator.wikimedia.org/T331475) [17:14:56] I think dlocal must be re-sending those rejection IPNs till it gets a 200 [17:15:15] should we silence the failmail with config for now? [17:17:53] fr-tech who knows how to turn off SmashPig failmail for just one payment processor? [17:18:35] I don't [17:19:03] nor do I [17:19:06] also hiiiiiiii [17:19:23] sounds like a reasonable strategy on their part [17:19:41] yeah they resend until 200 [17:24:23] so it's going to be a setting in settings/SmashPig/config-local/dlocal/main.yaml [17:24:56] overriding the default failmail recipient in provider-defaults.yaml [17:25:05] hmmm [17:25:10] the log stream ejegg ? [17:25:23] ah right jgleeson we can replace the whole log stream [17:25:36] we now have a null log stream [17:26:33] ejegg: where is your settings dir? for fr-dev is that equivalent of config-private/smashpig/dlocal/main.yaml ? [17:26:59] yep, that's the equivalent [17:27:05] ok [17:27:37] see commit 676ed6028352d50a2b56d719cc55d0b12f3e9c89 on frpm1002 for the equivalent patch disabling it for PayPal [17:28:32] and in the localsettings repo on frpm the dir is SmashPig/local-config/dlocal/main.yaml right? [17:29:15] K yeah I see that :) [17:29:40] I can add the equivalent setting for dlocal and push it out for review, I guess? [17:29:52] thanks! [17:43:37] ok that's pushed and ready for review [17:45:10] looks good AndyRussG [17:45:48] so if we really just wanted to shut up the IPN failmail and still get it from payments, we could just rsync to frpig [17:46:06] but i think it's fine to rsync everywhere for now [17:46:20] ejegg: yeah sounds cleaner [17:46:40] just syncing to pig sounds also tricky to keep [17:46:48] just wanted to mention the possibility in case we have something like this for a processor being used in prod. [17:46:59] definitely hard to keep it that way [17:47:03] yeah [17:47:16] oki I'll do that now? anything else we want to push out while I'm at it? [17:47:35] or anything almost about to be +2'ed that I should wait for? [17:47:41] just that one for now [17:47:42] fr-tech ^ [17:49:21] k gonna deploy that [18:00:44] !log update SmashPig config 59a8b2d2 -> 6e651fd [18:00:47] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:01:01] okiii done [18:02:01] 10Wikimedia-Fundraising-Banners, 10Patch-For-Review, 10Wikipedia-Android-App-Backlog (Android Release FY2022-23): Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10Dbrant) a:03Dbrant [18:11:45] 10Wikimedia-Fundraising-Banners, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2022-23: Japan fundraising messages in iOS app - https://phabricator.wikimedia.org/T332229 (10LGoto) p:05Triage→03Medium [18:13:01] 10Wikimedia-Fundraising-Banners, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2022-23: Japan fundraising messages in iOS app - https://phabricator.wikimedia.org/T332229 (10LGoto) Work will be done via Android task https://phabricator.wikimedia.org/T332230 [18:13:03] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Recurring-Donations, 10fr-donorservices: Ingenico recurrings stopped at status 600 March 5th - https://phabricator.wikimedia.org/T331490 (10MBeat33) Nothing similar at Ingenico between Mar 15-21. [18:43:07] 10Fundraising-Backlog, 10fundraising-tech-ops, 10fr-donorservices: Fundraising access request for Salvatore Borriello - https://phabricator.wikimedia.org/T331491 (10Dwisehaupt) Instructions on ssh config file and connections were sent on 20230316. Waiting to see if further assistance is needed. [19:00:46] (03PS2) 10Ejegg: Map payment_submethod to payment_method_id [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/900429 (https://phabricator.wikimedia.org/T331475) [19:12:59] 10Wikimedia-Fundraising-Banners, 10Patch-For-Review, 10Wikipedia-Android-App-Backlog (Android Release FY2022-23): Japan fundraising messages in Android app - https://phabricator.wikimedia.org/T332230 (10Dbrant) 👇 {F36922072} [19:29:15] (03PS1) 10Jgleeson: WIP: add rejected IPN handling to dlocal listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901701 (https://phabricator.wikimedia.org/T332637) [19:29:44] (03CR) 10CI reject: [V: 04-1] WIP: add rejected IPN handling to dlocal listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/901701 (https://phabricator.wikimedia.org/T332637) (owner: 10Jgleeson) [19:40:00] 10Fundraising Sprint Juggalology 2022, 10Fundraising Sprint Kermit hopping principle, 10Fundraising Sprint Localhost Hospitality Studies, 10Fundraising Sprint Machine Unlearning, and 4 others: Currency code/currency symbol in Thank You emails received by Donors iss... - https://phabricator.wikimedia.org/T309349 [19:40:21] 10Fundraising Sprint Amazing grep, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Patch-For-Review: Decommission `civi-data-translate` extension - https://phabricator.wikimedia.org/T325698 (10XenoRyet) [19:42:32] (03PS2) 10Eileen: Remove watchdog from Oanda retriever [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898001 (https://phabricator.wikimedia.org/T288585) [19:42:39] (03PS3) 10Eileen: Remove watchdog from Oanda retriever [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898001 (https://phabricator.wikimedia.org/T288585) [19:50:50] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Delete pending messages in the recurring queue consumer - https://phabricator.wikimedia.org/T332690 (10Ejegg) [19:51:43] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Delete pending messages in the recurring queue consumer - https://phabricator.wikimedia.org/T332690 (10Ejegg) maybe we could just do this instead: https://phabricator.wikimedia.org/T240581 [19:52:46] 10Fundraising-Backlog: Delete pending messages in the recurring queue consumer - https://phabricator.wikimedia.org/T332690 (10jgleeson) [19:58:11] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Recurring-Donations, 10fr-donorservices: Ingenico recurrings stopped at status 600 March 5th - https://phabricator.wikimedia.org/T331490 (10XenoRyet) p:05Triage→03Medium [19:58:18] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: PaymentsWiki Error: ResizeObserver loop limit exceeded - https://phabricator.wikimedia.org/T332693 (10XenoRyet) p:05Triage→03Medium [20:01:59] brb, rebooting for kernel update [20:22:02] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10AKanji-WMF) [20:28:42] fr-tech I sent an email asking about the audit file example [20:29:43] (03PS2) 10Umherirrender: build: Updating npm dependencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/901702 (owner: 10Libraryupgrader) [20:31:19] (03CR) 10Cstone: [C: 03+2] Remove watchdog from Oanda retriever [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898001 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [20:33:07] thanks cstone! [20:34:05] (03CR) 10Umherirrender: [C: 03+1] build: Updating npm dependencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/901702 (owner: 10Libraryupgrader) [20:35:51] ejegg: when writing up how to turn off the failmail is there a provider-defaults on production? i don't see it or is it somewhere special [20:36:00] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10Eileenmcnaughton) @shust are you talking abou this functionality? ie any name pairs saved here will be picked up y th... [20:41:28] AndyRussG: ahhh, that earlier config patch was bad [20:41:46] ejegg ohhh oops oki [20:41:54] the log-streams key needs to go under logging/ [20:42:00] okiii [20:42:02] soory [20:42:09] no worries [20:42:28] cstone: it should be in the base of SmashPig/local-config [20:42:55] oh odd, that's a symlink to /etc/smashpig/provider-defaults.yaml on civi1001 [20:44:26] cstone: ah, it looks like it's managed by puppet [20:44:31] nice spot [20:44:39] fundraising/smashpig/provider-defaults.yaml.erb [20:45:31] (03Merged) 10jenkins-bot: Remove watchdog from Oanda retriever [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898001 (https://phabricator.wikimedia.org/T288585) (owner: 10Eileen) [20:45:51] heh, i wish i'd spotted it earlier [20:48:48] (03PS5) 10Ejegg: Remove orphan_slayer and DonationInterface [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/886207 (https://phabricator.wikimedia.org/T332358) [20:49:16] ejegg: looks like SmashPig/local-config/amazon/main.yaml is also wrong then [20:49:47] nice catch. want to fix it? [20:49:52] oh wow are we finally in a position to slay the orphan slayer and DI composer package [20:50:03] actually, if we're suppressing failmail for Amazon maybe we should stop [20:50:07] jgleeson: yes!!!! [20:50:11] nice! [20:50:29] that was the benefit of that paypal pending resolver [20:51:29] huge benefit then reducing the complexity around that [20:51:43] :) :) [20:52:07] and it unlocks a bit of code removal in DonationInterface (WmfFramework) [20:52:29] ah nice [20:52:29] fr-tech so instead of fixing the Amazon null-logger thingy, remove it, right? [20:52:42] AndyRussG: yes please [20:52:46] oki [20:53:00] also btw pushed a change to fix dlocal failmailstopper [20:53:01] I guess that must have been to quiet down firewall failmails. [20:53:36] or meant to do that anyway [20:54:04] or failwall firemails [20:54:27] firemails sound threatening [20:54:46] i mean, flaming poo is just on the porch [20:54:51] mail comes through the slot [20:54:56] lol [20:56:51] AndyRussG: ahh i see, the failmail change for amazon was just adding the throttle time. I think we actually do want that [20:57:09] ejegg: well it wasn't actually working? [20:57:27] why don't we leave it as is unless we get too many of them again? [20:57:42] or that is, leave it disabled? [20:58:22] ok, I just don't want to risk causing OIT mail server problems again like we did with paypal that time [20:58:49] but yeah, probably lower risk with Amazon [20:59:14] k, justchecked that paypal throttle is correctly configged [20:59:28] ok [20:59:55] so I can deploy the stuff again if it's approved [21:00:01] anyway, your changes look good to deploy [21:01:14] oki deploying [21:02:39] !log update SmashPig config 6e651fd4 -> 035f602a [21:02:42] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [21:02:47] k done [21:02:53] thanks! [21:03:01] thank u! :) [21:05:31] (03CR) 10Ejegg: "PS5: manual rebase, had to regenerate composer.lock" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/886207 (https://phabricator.wikimedia.org/T332358) (owner: 10Ejegg) [21:08:38] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10SHust) @Eileenmcnaughton, I knew about this functionality, thank you. I was talking about the Japanese family names s... [21:13:44] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Dedupe, 10FR-Email: Civi dedupe: automatically dedupe alternate family names - https://phabricator.wikimedia.org/T280696 (10Eileenmcnaughton) @shust - yes - that functionality was always intended for names that are in different scripts, as w... [21:15:55] hmmm well presently no more firemail [21:21:35] nice [21:34:54] (03PS1) 10Ejegg: Delete drupal implementation of WmfFramework [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/901710 (https://phabricator.wikimedia.org/T291785) [21:36:15] ^ ohhh dawn of a new era eh [21:36:26] (03CR) 10CI reject: [V: 04-1] Delete drupal implementation of WmfFramework [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/901710 (https://phabricator.wikimedia.org/T291785) (owner: 10Ejegg) [21:36:46] just a baby step... [21:37:13] oh i see, auoloaded [21:40:17] one small step for a gerrit change, a giant leap for a framework [21:40:27] (03PS2) 10Ejegg: Delete drupal implementation of WmfFramework [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/901710 (https://phabricator.wikimedia.org/T291785) [21:41:00] * AndyRussG watches WmfFramewok get kicked out the airlock [21:46:01] (03CR) 10Cstone: [C: 03+2] Remove watchdog from refund_queue_consume drush [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898031 (owner: 10Eileen) [21:53:41] (03CR) 10Damilare Adedoyin: [C: 03+2] "Thanks for this, tested with UPI and Card recurring/one-time payments. Even though UPI (UI) returned returned a reject, every other mode m" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/900429 (https://phabricator.wikimedia.org/T331475) (owner: 10Ejegg) [21:54:55] (03CR) 10Damilare Adedoyin: [C: 03+2] "As a support to the Smashpig patch, this worked great as well. LGTM" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900450 (https://phabricator.wikimedia.org/T331475) (owner: 10Ejegg) [21:56:52] thanks damilare|away ! [21:57:21] pleasure. thanks for working on those too [21:59:43] (03Merged) 10jenkins-bot: Remove watchdog from refund_queue_consume drush [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/898031 (owner: 10Eileen) [21:59:45] (03Merged) 10jenkins-bot: Map payment_submethod to payment_method_id [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/900429 (https://phabricator.wikimedia.org/T331475) (owner: 10Ejegg) [22:21:33] (03PS4) 10Ejegg: Dlocal: Stop mapping submethod to payment_method_id [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900450 (https://phabricator.wikimedia.org/T331475) [22:22:34] (03CR) 10Ejegg: [C: 03+2] "Re-applying C+2 applied by Damilare to PS3 (only change in PS4 is updating SmashPig version number)" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900450 (https://phabricator.wikimedia.org/T331475) (owner: 10Ejegg) [22:24:16] (03Merged) 10jenkins-bot: Dlocal: Stop mapping submethod to payment_method_id [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/900450 (https://phabricator.wikimedia.org/T331475) (owner: 10Ejegg) [22:38:51] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/901716 [22:38:56] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/901716 (owner: 10Ejegg) [22:39:31] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/901716 (owner: 10Ejegg) [22:41:13] ejegg: Joaquin is asking for an example audit file, I see we are downloading the current one with the audit tools repo but not seeing where its coming from to download it [22:42:05] lemme see, that's probably in a puppet-managed file [22:44:12] oh do we download it from their ftp? [22:44:25] yep [22:45:39] (03PS1) 10Ejegg: Update composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/901718 [22:45:44] (03CR) 10Ejegg: [C: 03+2] Update composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/901718 (owner: 10Ejegg) [22:47:38] !log payments-wiki upgraded from 0fd66b1f to ab0a55a2 [22:47:41] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [22:52:49] (03Merged) 10jenkins-bot: Update composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/901718 (owner: 10Ejegg) [23:10:03] 10fundraising-tech-ops: reimage fundraising_database role servers to Bullseye - https://phabricator.wikimedia.org/T315601 (10Dwisehaupt) [23:15:38] 10fundraising-tech-ops, 10User-greg: Renew FR-Tech nessus license - https://phabricator.wikimedia.org/T332097 (10greg) a:05greg→03Dwisehaupt Thanks to the awesome Security team (hi Jennifer!) they'll cover the cost for this renewal this time. Dallas: can you find any renewal/contract/etc that I can use to... [23:33:02] 10fundraising-tech-ops, 10User-greg: Renew FR-Tech nessus license - https://phabricator.wikimedia.org/T332097 (10Dwisehaupt) Forwarded the only email thread I have on it.