[00:07:04] (03PS9) 10Eileen: Update Fidelity Import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136833 (https://phabricator.wikimedia.org/T386017) [00:07:04] (03PS7) 10Eileen: Upstream fix for uploaded file with template [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137096 [00:07:04] (03PS10) 10Eileen: mgd file for Fidelity import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1136834 [00:07:04] (03PS6) 10Eileen: Superficial cleanup in test class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138140 [00:07:05] (03PS4) 10Eileen: port date patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138477 (https://phabricator.wikimedia.org/T359219) [00:07:07] (03PS2) 10Eileen: Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) [00:07:11] (03PS1) 10Eileen: Convert getUserJob and getGatway to be non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139203 (https://phabricator.wikimedia.org/T359219) [00:07:15] (03PS1) 10Eileen: Make SetTimeOfDayIfStockDonation non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139204 (https://phabricator.wikimedia.org/T359219) [00:07:19] (03PS1) 10Eileen: Extract inValidateModeDoNotRequireTotalAmount [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139205 (https://phabricator.wikimedia.org/T359219) [00:07:24] (03PS1) 10Eileen: Benevity import feature 1 - filter not-shared-info [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139206 (https://phabricator.wikimedia.org/T359219) [00:07:28] (03PS1) 10Eileen: Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 [00:07:32] (03PS1) 10Eileen: Benevity feature 2 - field transformations on Gift Sourc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139208 (https://phabricator.wikimedia.org/T359219) [00:07:36] (03PS1) 10Eileen: Add Benevity mgd file [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139209 [00:39:19] (03CR) 10CI reject: [V:04-1] Start to make Import hook class non-static [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [00:41:33] (03CR) 10CI reject: [V:04-1] Minor function extractions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 (owner: 10Eileen) [06:52:28] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1138598 (https://phabricator.wikimedia.org/T359219) (owner: 10Eileen) [06:52:56] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139207 (owner: 10Eileen) [11:10:39] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fix bugs in smash-pig/Maintenance/BuildDonationMessageFromPaymentLogs - https://phabricator.wikimedia.org/T392536#10772068 (10jgleeson) →14Duplicate dup:03T381012 [11:10:39] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10772070 (10jgleeson) [11:10:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10772071 (10jgleeson) a:03jgleeson [11:23:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Create WMF standalone config - https://phabricator.wikimedia.org/T372005#10772115 (10jgleeson) a:03jgleeson [11:41:24] (03CR) 10Jforrester: "check experimental" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139122 (owner: 10Umherirrender) [11:45:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Potential FundraiseUp Data Integration Issue - https://phabricator.wikimedia.org/T391126#10772195 (10Damilare) Hi @SHust looks like another contribution came in for this cancelled recurring with subscription ID: RDHWCPFN, it was can... [11:45:41] (03CR) 10Jforrester: [C:03+2] Use namespaced classes compatible with 1.41 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139122 (owner: 10Umherirrender) [11:54:35] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap, 07Epic: Ensure Fundraiseup recurring record have the Processor ID - https://phabricator.wikimedia.org/T392808 (10Damilare) 03NEW [11:54:58] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap, 07Epic: Ensure Fundraiseup recurring record have the Processor ID - https://phabricator.wikimedia.org/T392808#10772229 (10Damilare) a:03Damilare [12:04:33] (03Merged) 10jenkins-bot: Use namespaced classes compatible with 1.41 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139122 (owner: 10Umherirrender) [13:41:24] (03PS1) 10Ejegg: Delete obsolete boilerplate [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139472 [13:41:33] hi fr-tech [13:44:53] ooh, that's nice, releng has put the web front-end on the docker registry [13:45:00] so you get a nice interface when you hit https://docker-registry.wikimedia.org/ [13:45:09] and can figure out what the latest version of each image is [13:55:50] 10fundraising-tech-ops: Migrate backup fundraising analytics server from fran1001 to fran1002 - https://phabricator.wikimedia.org/T392818#10772723 (10Jgreen) 05Open→03In progress p:05Triage→03Medium [13:56:25] 10fundraising-tech-ops: Migrate backup fundraising analytics server from fran1001 to fran1002 - https://phabricator.wikimedia.org/T392818#10772739 (10Jgreen) a:03Jgreen [13:56:55] 10fundraising-tech-ops: Q1:rack/setup/install fran1002 - https://phabricator.wikimedia.org/T369940#10772742 (10Jgreen) [13:56:56] 10fundraising-tech-ops: Migrate backup fundraising analytics server from fran1001 to fran1002 - https://phabricator.wikimedia.org/T392818#10772743 (10Jgreen) [13:57:08] 10fundraising-tech-ops: Q1:rack/setup/install fran1002 - https://phabricator.wikimedia.org/T369940#10772744 (10Jgreen) 05In progress→03Resolved [14:19:49] hi ejegg [14:20:06] I feel like I've seen that UI before [14:20:47] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139472 (owner: 10Ejegg) [14:21:28] ejegg: do you know if that new fun-dev CiviCRM we added with the new wmf-cv tool baked in get deployed [14:22:54] jgleeson: yep! [14:23:00] thanks! [14:23:04] it's all set [14:23:11] (03Merged) 10jenkins-bot: Delete obsolete boilerplate [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139472 (owner: 10Ejegg) [14:23:16] :) [14:25:49] oh wait, was that 1.1.0 jgleeson ? [14:26:28] civicrm-bookworm-php82-apache2:1.1.0 [14:26:34] yeah, that's deployed [14:27:00] so what's the issue with this https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/1139198 [14:27:03] ? [14:27:06] I [14:27:13] I'm getting [14:27:14] service "wmf-cv" is not running [14:27:30] oh it needs the service name first [14:27:37] i.e. civicrm before wmf-cv [14:27:41] lemme edit that [14:28:34] ok, now it's running, just failing with Failed to determine contactID for user=root [14:32:01] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: eqiad: determine second frack - https://phabricator.wikimedia.org/T392007#10772843 (10ayounsi) p:05Triage→03Medium [14:32:22] (03PS2) 10Ejegg: Switch thank you to new api, update eoyemail.send [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139198 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [14:37:11] jgleeson: it looks like I can run with --user=admin [14:37:17] but --user=root doesn't work [14:37:31] it might already be doing that [14:38:21] https://gitlab.wikimedia.org/repos/releng/dev-images/-/merge_requests/74/diffs?commit_id=a80e5211100f1cc42e4ae020d4266ea22938eb5a#efa129c9fab310e66aad03d023ba1c9a1382868f [14:40:05] yep, that's the commit I've got locally [14:40:22] And it can't find any civi user named 'root' [14:40:32] ah I see [14:40:33] but we do have an 'admin' user installed by default [14:40:43] did that change with standalone maybe? [14:41:03] is this the OS user or the CiviCRM user [14:41:53] I took that script from the one on prod so we might need to update that to fit dev if the user doesn't match [14:56:43] ohh i see [14:56:49] i think it's the civi user jgleeson [14:57:54] 06Fundraising-Backlog, 10Collection, 10EntitySchema, 10MediaWiki-extensions-CentralAuth, and 11 others: Update references to legacy namespace for Skin in various WMF deployed code - https://phabricator.wikimedia.org/T392463#10772937 (10matmarex) [14:58:35] ah ok ejegg [14:58:44] I'll push up an update to default to admin [14:59:04] unfortunately it'll need to go through the usual deploy steps :( [14:59:28] that's probably easier than shimming in a root user do you think? [15:06:43] well, does the user change to root in standalone? [15:08:06] hmm [15:09:32] nope ejegg it's admin there too [15:09:54] I just pushed up http://gitlab.wikimedia.org/repos/releng/dev-images/-/commit/482e816cae9956c4eefbf09419a1a93afba17aa0 [15:14:48] just adding the other patch. The image built fine for me [15:15:32] great! [15:15:52] trying to see how to merge your request, just a sec [15:16:12] hang fire I'm adding notes to the MR [15:16:29] ok [15:16:51] let's also delete the older OS images to clean up a bit [15:16:57] I'll do that in another MR [15:19:45] (03PS1) 10Jgleeson: Update CiviCRM image to 1.1.1 [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139500 [15:21:06] (03PS2) 10Jgleeson: Update CiviCRM image to 1.1.1 [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139500 (https://phabricator.wikimedia.org/T383097) [15:21:22] ejegg: that should be everything [15:22:19] also, that command `docker-pkg -c dockerfiles/config.yaml build --no-pull --select 'docker-registry.wikimedia.org/dev/fundraising-civicrm-bookworm-php82-apache2:1.1.1' dockerfiles/` worked for me but I recall it not working for you at the offsite. I can't remember why [15:29:39] hmm, I think I did eventually get it to build though [15:30:05] 06Fundraising-Backlog, 07Fundraising-FRUP, 06Fundraising-Tech-Roadmap, 07Epic: Ensure Fundraiseup recurring record have the Processor ID - https://phabricator.wikimedia.org/T392808#10773122 (10Damilare) Fundraiseup contribution recur rows with missing Processor IDs have now being filled. [15:43:07] yeah ejegg I think we needed a wildcard somewhere [15:43:24] the paths in that command look relative tho so it's odd that it didn't work [15:43:54] wfan: the bug in that script was due to a different in how we log gravy messages vs other processors [15:44:15] well, the command in your instructions jsut worked perfectlyh for me! [15:44:36] awesome! Another good news dlocal just messaged me that stitch should be working now, validating that [15:44:52] OK, it's merged [15:44:59] great! [15:45:06] in releng they mentioned some steps for publishing it [15:45:08] must be location-dependant :) [15:45:15] ah cool [15:45:19] I wonder if that's anything we can do, or if we just have to ask them [15:45:41] s/different/difference/ [15:45:54] hmm, it says 'We use a script for deployment to the Wikimedia Docker registry: [15:45:57] $ fab deploy_devimages' [15:46:02] but nothing about where to deploy that [15:46:16] guessing it's a box that releng has shell access to [15:46:32] ah that would make sense [15:46:50] I feel like hashhar might have mentioned he had to hop on to a box in the past [15:47:01] our recurring "how does that work" subject [15:52:48] https://payments.wikimedia.org/index.php?title=Special:GravyGateway&appeal=JimmyQuote&payment_method=bt&recurring=0&uselang=en&language=en¤cy=ZAR&amount=55&country=ZA&first_name=Jimmy&last_name=Wales&email=jwales%40example.com not able to test that in prod, but it loading something instead of error before, while local still showing authorization_declined, messaging them that, see if they can fix the sandbox first. [15:58:49] (03PS1) 10Wfan: update smash-pig requirement to 0.9.5.1 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139507 [15:59:40] (03CR) 10CI reject: [V:04-1] update smash-pig requirement to 0.9.5.1 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139507 (owner: 10Wfan) [16:01:20] (03CR) 10Wfan: "recheck" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139507 (owner: 10Wfan) [16:01:56] oh i see, the 'fab' script is in the root of the dev-images repo [16:02:14] and it runs some remote commands on contint*.wikimedia.org via ssh [16:02:31] so if they gave us shell there we could probably publish ourselves [16:02:49] (and also maybe make a mess of CI...) [16:04:35] (03CR) 10Wfan: [C:03+2] update smash-pig requirement to 0.9.5.1 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139507 (owner: 10Wfan) [16:21:36] gonna just move djangobannerstats off bullseye in fr-dev to match how it is in prod [16:22:00] then we can delete all the fundraising*bullseye* images [16:22:16] (03Merged) 10jenkins-bot: update smash-pig requirement to 0.9.5.1 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1139507 (owner: 10Wfan) [16:25:44] nice ejegg [16:25:49] back later [16:26:45] 06Fundraising-Backlog: Ingenico changes to Ideal flow for our roadmap - https://phabricator.wikimedia.org/T239924#10773441 (10EMartin) 05Open→03Resolved a:03EMartin [16:30:32] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 07payments-orchestration: Switch Credit Cards from Adyen to Gravy - https://phabricator.wikimedia.org/T392147#10773462 (10EMartin) @AKanji-WMF @XenoRyet Hi both, Online have consented to cut over Cards, Apple Pay and Gpay to Gr4vy as of today: https:/... [16:32:53] dang, it's not running [16:32:58] ImportError: cannot import name 'Iterator' from 'collections' (/usr/lib/python3.11/collections/__init__.py) [16:33:11] so do we have some extra package installed on fran2001 ? [16:33:46] it's running 3.11.2, same as fundraising-bookworm-python [16:37:05] oh hmm, there were some warnings about target directories existing [16:37:09] let's clear those out and see [16:40:17] hmph, nope [16:40:31] so it's the version of django that's wrong apparently [16:40:56] well, it's inside django that it's importing from the wrong colletions [16:41:04] let's see what's up on fran [16:41:40] ahh I see, and we're pulling specific package versions in the build script [16:41:46] OK, lemme update those to match prod [16:43:49] (03PS1) 10Wfan: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1139519 [16:44:03] (03CR) 10Wfan: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1139519 (owner: 10Wfan) [16:52:43] ok, getting closer [16:52:50] just needs some config updates [16:53:32] for the caches I think [16:54:18] oh hmm, does that need to be in the image? it's in /etc/fundraising [16:54:59] nope, looks like it's in config [16:55:25] woo, and it's running! [16:57:24] (03PS1) 10Ejegg: Use bookworm for DjangoBannerStats [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139520 [17:10:39] (03CR) 10Damilare Adedoyin: "Hi jgleeson, thanks for catching this. I just saw your comment on the main patch, although not sure I fully understand, perhaps we can tal" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 (owner: 10Jgleeson) [17:11:33] (03CR) 10Damilare Adedoyin: [C:03+1] "Hi jgleeson, thanks for catching this. I just saw your comment on the main patch, although not sure I fully understand, perhaps we can tal" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 (owner: 10Jgleeson) [17:14:55] fr-tech I've got a merge request in dev-images to delete our bullseye images (releng requested we clean them up) [17:15:22] We just need to stop running django-banner-stats on bullseye in dev (to match what we're doing inproduction) [17:15:30] so I've got another patch to make that work [17:15:36] https://gitlab.wikimedia.org/repos/releng/dev-images/-/merge_requests/77 [17:15:41] https://gerrit.wikimedia.org/r/1139520 [17:25:10] 10fundraising-tech-ops, 10BDC-Implementation: SurveyMonkey load data into platform - https://phabricator.wikimedia.org/T385743#10773661 (10AStein-WMF) i can pull data from the api through dlt. now i need to: 1. configure minio as a source 2. add the script to dagster [17:40:16] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1139519 (owner: 10Wfan) [17:43:25] !log payments-wiki upgraded from e7f66569 to 3d60091d [17:43:28] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [17:45:48] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10MW-1.44-notes (1.44.0-wmf.27; 2025-04-29), 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10773723 (10AnnWF) >>! In T389547#10749680, @AnnWF wrote: > @RKumar_WMF see a ticket for use gravy for al... [17:51:43] (03CR) 10Jgleeson: [V:03+2 C:03+2] Update CiviCRM image to 1.1.1 [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139500 (https://phabricator.wikimedia.org/T383097) (owner: 10Jgleeson) [17:52:47] (03PS2) 10Ejegg: Use bookworm for DjangoBannerStats [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139520 [17:54:54] (03CR) 10Jgleeson: [C:03+2] "Installs fine for me" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139520 (owner: 10Ejegg) [17:54:58] (03CR) 10Jgleeson: [V:03+2 C:03+2] Use bookworm for DjangoBannerStats [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139520 (owner: 10Ejegg) [18:05:14] thanks! [18:08:13] oh, my donor portal patches for the crm repo got merged! [18:08:24] Lemme do a follow-on fixup patch [18:09:24] (03Abandoned) 10C. Scott Ananian: DNM: CI test after update to wikimedia/remex-html [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1138475 (owner: 10C. Scott Ananian) [18:19:42] (03CR) 10Jgleeson: "Yeah, let's go through this on tech-talk. It'll be easier to explain." [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 (owner: 10Jgleeson) [18:25:10] damilare|brb: out of interest, did you try out `docker compose cp source target` over an existing running site? I'm assuming you did first before deciding to delete first, and then copy, but I might be wrong so wanted to confirm. [18:26:01] when I tried way back when, I wasn't able to `docker-compose cp source target` over an existing directory. It kept giving me errors that a file with the same name already existed [18:26:14] I was hoping it would "merge" the two if you know what I mean [18:27:03] 10fundraising-tech-ops: Migrate backup fundraising analytics server from fran1001 to fran1002 - https://phabricator.wikimedia.org/T392818#10773935 (10Jgreen) [18:36:44] damilare|brb: it does merge! or at least it did in the test I just did [18:37:03] meaning we don't have to delete first and then copy [18:37:58] so instead of reverting, we can update the scripts to `docker compose cp source target` over the existing site when pushing [18:42:22] (03CR) 10Jgleeson: "Okay, it looks like `docker compose cp` is allowing me to copy an existing directory structure from the container. Maybe we can do this af" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 (owner: 10Jgleeson) [18:43:26] (03CR) 10Jgleeson: ""is allowing me to copy an existing directory structure from local* to* the container"" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1138360 (owner: 10Jgleeson) [18:50:31] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-dlocal, 06FR-donorrelations: dLocal audit file question + missing reversed transaction - https://phabricator.wikimedia.org/T389245#10773998 (10MBeat33) The canceled transactions from the prior comment are not in Civi as of today. [19:05:55] 06Fundraising-Backlog: There is no active payment service for this transaction. response from gravy, do we want failmail? - https://phabricator.wikimedia.org/T392545#10774079 (10jgleeson) This means we haven't set up the payment service in the Gravy console. I saw a few of these when mistakenly testing live for... [19:14:07] (03PS1) 10Ejegg: Fixes for donor portal patches [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139540 (https://phabricator.wikimedia.org/T388739) [19:14:52] PROBLEM - Host civi2002 is DOWN: PING CRITICAL - Packet loss = 100% [19:15:06] (03PS1) 10Ejegg: Set base URL for donor portal in Civi [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139541 (https://phabricator.wikimedia.org/T388739) [19:15:14] ^^ that's me sneaking in some updates. [19:19:54] RECOVERY - Host civi2002 is UP: PING OK - Packet loss = 0%, RTA = 30.42 ms [19:20:42] PROBLEM - Host frauth2002 is DOWN: PING CRITICAL - Packet loss = 100% [19:23:07] sounds good jgleeson, sorry I went to have dinner, I also tried it to confirm my assertions before the comment. [19:23:26] I'd push up a follo on patch removing the clean up lines [19:23:31] follow* [19:23:34] thanks damilare ! [19:25:12] PROBLEM - Host frpig2002 is DOWN: PING CRITICAL - Packet loss = 100% [19:25:44] RECOVERY - Host frauth2002 is UP: PING OK - Packet loss = 0%, RTA = 30.50 ms [19:28:21] RECOVERY - Host frpig2002 is UP: PING OK - Packet loss = 0%, RTA = 30.38 ms [19:41:54] 14Fundraising Sprint: Abula, 06Fundraising-Backlog: Duplicates after import - https://phabricator.wikimedia.org/T382516#10774134 (10XenoRyet) [19:42:40] 06Fundraising-Backlog, 10donate.wikimedia.org: Provide a donate skin for donate.wikimedia.org - https://phabricator.wikimedia.org/T257862#10774135 (10XenoRyet) [19:44:55] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy error: Argument #1 ($amount) must be of type float, string given - https://phabricator.wikimedia.org/T392655#10774138 (10XenoRyet) p:05Triage→03High [19:45:12] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Create WMF standalone config - https://phabricator.wikimedia.org/T372005#10774143 (10XenoRyet) p:05Triage→03Medium [19:46:11] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Smashpig, 07payments-orchestration: Gravy API::getTransaction should handle bad response more gracefully - https://phabricator.wikimedia.org/T392383#10774146 (10XenoRyet) p:05Triage→03Medium [19:46:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: Log 'pending' details JSON blob for all authorized Gravy transactions - https://phabricator.wikimedia.org/T392156#10774147 (10XenoRyet) p:05Triage→03High [19:47:45] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-AutoTY-Email, 13Patch-For-Review: One CID with an on-hold email address is getting a TY mail activity on every run - https://phabricator.wikimedia.org/T392387#10774149 (10XenoRyet) 05Open→03Resolved [19:50:50] PROBLEM - Host frqueue2003 is DOWN: PING CRITICAL - Packet loss = 100% [19:52:48] PROBLEM - Host pay-lb2002 is DOWN: PING CRITICAL - Packet loss = 100% [19:55:52] RECOVERY - Host frqueue2003 is UP: PING OK - Packet loss = 0%, RTA = 30.42 ms [19:57:10] RECOVERY - Host pay-lb2002 is UP: PING OK - Packet loss = 0%, RTA = 30.43 ms [19:58:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Transactions stuck in "Authorised" status in Adyen - Urgent - https://phabricator.wikimedia.org/T392000#10774186 (10AnnWF) @MBeat33 yes donor might get multiple refund since the way we check for the duplicate charge is by looking for their donation in the... [19:58:50] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Lots of snooze activities on one contact - https://phabricator.wikimedia.org/T392726#10774200 (10AKanji-WMF) p:05Triage→03Low [19:59:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Lots of snooze activities on one contact - https://phabricator.wikimedia.org/T392726#10774202 (10AKanji-WMF) p:05Low→03Medium [20:01:59] 06Fundraising-Backlog: DAF batch data entry - match to Fidelity rules - https://phabricator.wikimedia.org/T392723#10774217 (10AKanji-WMF) If you're able to make these changes now relatively easy in the form @MDemosWMF we'd advise you to go ahead, that said we hope to have an update on the status of the batch ent... [20:02:31] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failmail: Command "ThankYou.BatchSend" is not defined - https://phabricator.wikimedia.org/T392685#10774219 (10AKanji-WMF) p:05Triage→03High [20:05:17] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: LATAM test issues - Gr4vy - https://phabricator.wikimedia.org/T392677#10774228 (10AKanji-WMF) p:05Triage→03Medium [20:07:23] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog, 10MW-1.44-notes (1.44.0-wmf.27; 2025-04-29), 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10774232 (10AnnWF) [20:14:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Uptick in Dutch donors experiencing issues donating - https://phabricator.wikimedia.org/T372571#10774251 (10AKanji-WMF) @EMartin would you or @RKumar_WMF be able to set up iDeal testing that we could sit in on to try to see if this is still occurring and... [20:17:33] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Fix thank you to default to total_amount & currency when original_amount & original_currency empty, - https://phabricator.wikimedia.org/T392646#10774262 (10AKanji-WMF) Didn't find this issue on more recent data, not a priority but could be good to addres... [20:19:40] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: We are trying to send thank yous to 2 contacts with unusable emails - https://phabricator.wikimedia.org/T392548#10774274 (10AKanji-WMF) p:05Triage→03Medium Should we have emails associated with Anonymous record CID 72 at all? [20:21:52] 06Fundraising-Backlog, 10FR-email-preferences: Comms preference centre - scope adding to Acoustic email footers - https://phabricator.wikimedia.org/T392742#10774294 (10AKanji-WMF) Thanks @ppenloglou will coordinate a call after your offsite with FR-Tech to discuss next steps. [20:26:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: There is no active payment service for this transaction. response from gravy, do we want failmail? - https://phabricator.wikimedia.org/T392545#10774324 (10AKanji-WMF) p:05Triage→03Medium [20:28:01] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Please set a default rule in donation_rules.yaml in logs for gravy transactions - https://phabricator.wikimedia.org/T392544#10774330 (10AKanji-WMF) p:05Triage→03High [20:28:09] 03Fundraising Sprint: Hot Pot!, 06Fundraising-Backlog: Setting up consolidated settlement reporting for all PSPs via Gr4vy for new recon process - https://phabricator.wikimedia.org/T391496#10774332 (10EMartin) ok. Just seeing this. Thanks@cstone. [20:31:50] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: clean up yaml files for gravy - https://phabricator.wikimedia.org/T392855 (10AnnWF) 03NEW [20:31:56] 06Fundraising-Backlog: DAF batch data entry - match to Fidelity rules - https://phabricator.wikimedia.org/T392723#10774373 (10MDemosWMF) @AKanji-WMF Unfortunately I don't think I know how to add a second soft credit line to the same contribution in the form. If it's an easy lift on your end great, but if not we... [20:32:50] 06Fundraising-Backlog, 10fundraising-tech-ops, 13Patch-For-Review: Can we get more MariaDB deadlock tools? - https://phabricator.wikimedia.org/T392396#10774381 (10AKanji-WMF) May be worth looking in the slow logs... [20:34:34] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: utm_campaign update request for mailing_id sp74105938 - https://phabricator.wikimedia.org/T386742#10774398 (10AKanji-WMF) [20:34:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: We are trying to send thank yous to 2 contacts with unusable emails - https://phabricator.wikimedia.org/T392548#10774400 (10AKanji-WMF) [20:34:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Civi merge is not saving the state when a mailing address is present. - https://phabricator.wikimedia.org/T379684#10774401 (10AKanji-WMF) [20:36:13] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: clean up yaml files for gravy - https://phabricator.wikimedia.org/T392855#10774418 (10AnnWF) [20:39:58] 06Fundraising-Backlog, 10FR-email-preferences: Comms preference centre - scope adding to Acoustic email footers - https://phabricator.wikimedia.org/T392742#10774442 (10AnnWF) FYI: here is the link for epc: https://donorpreferences.wikimedia.org/index.php?title=Special:EmailPreferences/emailPreferences&contact_... [20:43:30] it would be great to get the Fidelity stuff Melanie has tested out https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1136833 [20:47:31] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: New contact merged but with recurring contribution - https://phabricator.wikimedia.org/T388079#10774470 (10Eileenmcnaughton) p:05High→03Low [20:48:26] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: New contact merged but with recurring contribution - https://phabricator.wikimedia.org/T388079#10774481 (10Eileenmcnaughton) I've dropped this down to low priority because we now have a mechanism to pick up new instances... [21:07:07] looks like damilare|away & wfan are both missing in our root composer.json [21:12:04] (03PS1) 10Eileen: Fix authors in composer.json [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139558 [21:13:14] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139558 (owner: 10Eileen) [21:40:56] (03CR) 10CI reject: [V:04-1] Fix authors in composer.json [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139558 (owner: 10Eileen) [21:45:35] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139558 (owner: 10Eileen) [21:46:01] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1138934 should ease those test fails [22:07:12] PROBLEM - check_ipsec on frban1001 is CRITICAL: Strongswan CRITICAL - ok: 0 not-conn: fran1001_v4, fran1002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1001&service=check_ipsec [22:12:14] RECOVERY - check_ipsec on frban1001 is OK: Strongswan OK - 2 ESP OK https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1001&service=check_ipsec [22:28:30] looking at that upgrade and the mgd speed fix [22:31:16] (03PS1) 10Jgleeson: WIP: Fix issues in donation message build script [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139564 (https://phabricator.wikimedia.org/T381012) [22:36:52] hmm, css is a little wacky on 6.1.2 [22:37:07] eileen: does 6.1.2 mess up the contact summary CSS for you too? [22:37:23] I get the sidebar above the main content instead of next to it [22:37:23] ejegg: hmm - I'll switch to it [22:37:34] ah, let me make sure i've cleared all caches [22:37:51] ejegg: hmm - just note probably locally but NOT on prod - cos I left theisland enabled on prod [22:38:25] ah right, riverlea is only local, huh? [22:38:28] locally riverlea has been a bit flakey - for reasons that I assume will only affect drupal [22:38:42] PROBLEM - Host frbast2002 is DOWN: PING CRITICAL - Packet loss = 100% [22:38:43] I don't see any other problems though [22:38:50] yeah - when I've enabled it on prod the screen jumps around, intermittently [22:38:58] just gonna try a snooze [22:39:04] night [22:40:18] hah, i mean an acoustic snooze :) [22:40:39] have tested queue consumers and mail sending so far [22:43:10] RECOVERY - Host frbast2002 is UP: PING OK - Packet loss = 0%, RTA = 30.41 ms [22:46:47] cool - it's a pretty light update TBH [22:47:10] (03PS1) 10Wfan: Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) [22:47:35] (03CR) 10CI reject: [V:04-1] Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) (owner: 10Wfan) [22:47:41] ejegg: on the sending mail - I put this up too https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/1139198 (we really need to switch out all drush commands - can't recall if I created a phab) [22:49:06] oh Failed to determine contactID for user=root - that is weird - that is an issue with our build huh? I mean I guess it applies on new builds if no-one has ever logged in to create the uf match? [22:50:09] Delayed ejegg but yeah I've been seeing that riverlea local issue for a while [22:58:59] (03PS2) 10Wfan: Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) [23:01:29] eileen: that user=root issue was with the wmf-cv script in the image [23:01:39] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: Log 'pending' details JSON blob for all authorized Gravy transactions - https://phabricator.wikimedia.org/T392156#10774915 (10AnnWF) a:03AnnWF [23:01:40] we fixed it by updating the image to use user=admin [23:01:45] oh ok [23:02:08] (03PS3) 10Ejegg: Switch thank you to new api, update eoyemail.send [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139198 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [23:02:12] (03CR) 10Ejegg: [C:03+2] Switch thank you to new api, update eoyemail.send [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139198 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [23:02:17] (03CR) 10Ejegg: [V:03+2 C:03+2] Switch thank you to new api, update eoyemail.send [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1139198 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [23:02:22] cool [23:02:25] ok, merged! [23:09:39] cool [23:10:17] now we can delete thank_you module!!! https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1139197 [23:17:52] rip [23:18:43] (03CR) 10Cstone: [C:03+2] "thank you thank_you" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139197 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [23:18:51] woohoo [23:19:02] Anyone want to check for the amount ticket? Then I can deploy to prod tomorrow to quite the error [23:20:12] (03CR) 10Eileen: [C:03+2] Fixes for donor portal patches [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139540 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [23:20:38] wfan: which one is that? [23:21:05] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1139567 [23:21:24] (03PS4) 10Eileen: CiviCRM upstream updates to 6.1.2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137097 [23:21:40] hmm wfan is the validator not catching that? [23:22:38] amount is listed as a requried field [23:22:44] This is used for all payment methods like card Apple Pay and so on. So there is a chance amount in string also I see many unit test using string amount [23:23:05] Yes required just add the amount as other places to check in case empty [23:23:18] it would probably make more sense to have that check in the validator though? since they were broken out [23:24:09] Hmm, make sense [23:26:19] (03CR) 10Ejegg: [C:03+2] CiviCRM upstream updates to 6.1.2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1137097 (owner: 10Eileen) [23:37:51] validator check empty but not sure it should be the place to check type, will just remove the empty check then [23:39:26] (03PS3) 10Wfan: Add amount check before create payment request in gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1139567 (https://phabricator.wikimedia.org/T392655) [23:39:46] cstone: also if you got check that api3 to api4 is ready to test [23:42:38] (03Merged) 10jenkins-bot: Finish removing thank_you module [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1139197 (https://phabricator.wikimedia.org/T270674) (owner: 10Eileen) [23:54:16] wfan: want to update the commit message of the check one now that the check is gone? [23:54:57] im going to eat dinner then ill check the other patch [23:55:18] what check message? [23:55:38] the commit message of https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1139567 [23:55:47] since its not checking anymore just float converting [23:56:13] ah right, since all validator have the require fields check, and amount are there [23:57:44] so no longer need to add the empty check~ validateCreatePaymentInput have those covered~ [23:57:52] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139574 [23:58:23] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1139574 (owner: 10Eileen)