[04:55:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CI failing for DonationInterface - https://phabricator.wikimedia.org/T377327#10245052 (10hashar) [05:02:15] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CI failing for DonationInterface - https://phabricator.wikimedia.org/T377327#10245065 (10hashar) 05Resolved→03Open I am reopening due to T377644: the cldr extension has the same issue (ex: [[ https://gerrit.wikimedia.org/r/c/mediawiki/extensions/cldr/... [05:12:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: CI failing for DonationInterface - https://phabricator.wikimedia.org/T377327#10245074 (10hashar) 05Open→03Resolved [12:47:08] (03CR) 10Jgleeson: Ensure no recurring is initiated for gravy one-time only methods. (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) (owner: 10Damilare Adedoyin) [12:50:40] jgleeson: I think so, I believe I ran a test trxn for that, but lemme rerun another test to be double sure. [13:11:25] (03PS4) 10Damilare Adedoyin: Ensure no recurring is initiated for gravy one-time only methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) [13:13:05] (03CR) 10CI reject: [V:04-1] Ensure no recurring is initiated for gravy one-time only methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) (owner: 10Damilare Adedoyin) [13:18:40] (03PS5) 10Damilare Adedoyin: Ensure no recurring is initiated for gravy one-time only methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) [13:20:17] (03CR) 10CI reject: [V:04-1] Ensure no recurring is initiated for gravy one-time only methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) (owner: 10Damilare Adedoyin) [13:27:13] (03PS6) 10Damilare Adedoyin: Ensure no recurring is initiated for gravy one-time only methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) [13:33:11] (03CR) 10Damilare Adedoyin: "Thanks for the jgleeson, I responded to the question inline and added a test to confirm the use of that line." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1080730 (https://phabricator.wikimedia.org/T377335) (owner: 10Damilare Adedoyin) [13:40:38] fr-tech I'm thinking of dropping the word 'Reconciliation' from the newly proposed payment_orchestrator_reconciliation_id field, related to this ticket https://phabricator.wikimedia.org/T374144. I don't like it because we use the word Reconciliation in other contexts, namely audits, and we're essentially adding gravy-specific naming to something that is intended to be [13:40:40] payment-orchestrator agnostic. I'm not mad about this alternative name but I feel like something along the lines of payment_orchestrator_alt_id or payment_orchestrator_alternative_id, which we map the gravy reconciliation_id to, makes more sense. We could add a label on the CiviCRM UI to tell users that it maps to the Gravy reconciliation ID if we're worried they might not know. What [13:40:42] do you think? Any other suggestions? [13:41:23] payment_orchestrator_reference_id maybe [13:41:50] payment_orchestrator_link_id / payment_orchestrator_connector_id [13:42:05] hmm they suck too [13:43:47] i want to be sure before we lock this name in as it will need adding to payments, smashpig and civi so it's unlikely we'll ever change it [13:45:29] payment_orchestrator_cross_system_id [13:45:42] payment_orchestrator_global_id [13:45:58] payment_orchestrator_external_id [13:46:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Already cancelled Paypal cancel dates at 1969-12-31 - need to update to actual cancel date - https://phabricator.wikimedia.org/T373162#10246388 (10MSuijkerbuijk_WMF) Thanks @Ejegg does this mean this issue is fixed? @Cstone I believe... [13:47:05] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10246389 (10MSuijkerbuijk_WMF) @Ejegg as per above, can you confirm it's all set? Thanks! [13:47:08] I feel like they are all ugly but cross_system does a better job of explaining its purpose over reconciliation_id [13:47:44] I'll mull it over on the school run [14:13:08] 06Fundraising-Backlog: FundraiseUp testing Q2 - 2425 - https://phabricator.wikimedia.org/T377732 (10MSuijkerbuijk_WMF) 03NEW [14:17:13] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10246545 (10ERoden-WMF) Hi @Dwisehaupt @Jgreen @greg Ilse has received her computer, could you let us know when you'll be able to get her access set up? cc @IAckerman-WMF [14:24:29] 10fundraising-tech-ops, 10BDC-Implementation: [Hive] Setup and configuration - https://phabricator.wikimedia.org/T377368#10246564 (10Jgreen) 05Duplicate→03Resolved [14:25:57] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: Frack eqiad network upgrade: design, installation and configuration - https://phabricator.wikimedia.org/T377381#10246585 (10Jgreen) @cmooney Dallas did a survey of existing servers at eqiad, and none of them have 10G in... [14:31:50] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: Frack eqiad network upgrade: design, installation and configuration - https://phabricator.wikimedia.org/T377381#10246614 (10cmooney) >>! In T377381#10246582, @Jgreen wrote: > @cmooney Dallas did a survey of existing ser... [14:33:12] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: Frack eqiad network upgrade: design, installation and configuration - https://phabricator.wikimedia.org/T377381#10246616 (10cmooney) [14:34:13] jgleeson|skrun: if you don't want to be so specific as reconciliation, I'd go for the general 'alterntive' [14:53:04] !log disabled failing CiviCRM contact dedupe job [14:53:06] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [15:03:31] ejegg: so one of the grumbles I have to the 'alternative' option is that there are no other payment_orchestration IDs. The only other new ID is called backend_processor_trxn_ID so we're got a bit of a mismatch there naming wise [15:03:59] we've* [15:04:54] oh [15:10:42] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: Frack eqiad network upgrade: design, installation and configuration - https://phabricator.wikimedia.org/T377381#10246886 (10Jgreen) > That's a bit of a shame in some ways but no problems We'll get there next procuremen... [15:16:22] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10246938 (10Ejegg) Thanks @RLopez-WMF - that 'WMF Thank You' English template looks great.... [15:23:19] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10FR-email-preferences: Email pref centre - Solve email overwrite issue from recurring - https://phabricator.wikimedia.org/T353376#10246955 (10Damilare) I'm working on this ticket but in order to understand the requirements/proposed solution, I've... [15:23:32] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Already cancelled Paypal cancel dates at 1969-12-31 - need to update to actual cancel date - https://phabricator.wikimedia.org/T373162#10246957 (10Ejegg) The dates are fixed, but I haven't cancelled them on paypal's side yet. I'm goi... [15:24:28] damilare: we've got two tests I'm looking at and I'm confused about the names, testSuccessfulCreatePaymentFromTokenGuest() and testSuccessfulCreatePaymentFromTokenGuestCheckout(), the difference being 'Checkout' in the name. The both use the same mock response, and the latter populates the buyer object. Do those names still make sense to you? [15:24:30] https://github.com/wikimedia/wikimedia-fundraising-SmashPig/blob/44d7261ccd46a0e092e35936fb6a1bc4f3aaedb8/PaymentProviders/Gravy/Tests/phpunit/CardPaymentProviderTest.php#L78 [15:25:27] ohh [15:25:30] (I'm updating them to check for the new field with no name) [15:25:52] the girl has no name [15:26:37] looks like the one with just "Guest" is a duplicate of testSuccessfulCreatePaymentFromTokenWithProcessorContactId [15:26:52] I have to step off now brb [15:27:03] https://www.youtube.com/watch?v=EcoSYR0Ss7Q [15:27:09] ah gotcha [15:37:15] PROBLEM - Host frban2002 is DOWN: PING CRITICAL - Packet loss = 100% [15:38:11] PROBLEM - check_disk on frdb1004 is CRITICAL: DISK CRITICAL - free space: /dev 257276 MB (100% inode=99%): /run 51458 MB (99% inode=99%): / 5000 MB (71% inode=89%): /dev/shm 257299 MB (100% inode=99%): /run/lock 5 MB (100% inode=99%): /home 12348 MB (99% inode=99%): /tmp 12350 MB (99% inode=99%): /var 25122 MB (94% inode=99%): /var/tmp 422 MB (99% inode=99%): /var/log 23718 MB (89% inode=99%): /var/log/audit 422 MB (99% inode=99%): /boot [15:38:11] 84% inode=99%): /srv 499624 MB (10% inode=99%): https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_disk [15:40:31] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 not-conn: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [15:43:11] RECOVERY - check_disk on frdb1004 is OK: DISK OK - free space: /dev 257276 MB (100% inode=99%): /run 51458 MB (99% inode=99%): / 5000 MB (71% inode=89%): /dev/shm 257299 MB (100% inode=99%): /run/lock 5 MB (100% inode=99%): /home 12348 MB (99% inode=99%): /tmp 12350 MB (99% inode=99%): /var 25122 MB (94% inode=99%): /var/tmp 422 MB (99% inode=99%): /var/log 23718 MB (89% inode=99%): /var/log/audit 422 MB (99% inode=99%): /boot 575 MB (84 [15:43:11] 9%): /srv 1642250 MB (36% inode=99%): https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_disk [15:45:33] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 not-conn: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [15:47:01] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:servers migration task - https://phabricator.wikimedia.org/T375151#10247070 (10Papaul) [15:50:33] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 not-conn: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [15:51:52] Thanks ejegg for disable the failing dedupe job. [15:54:12] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Civi-Dedupe, 13Patch-For-Review: Fail mail on dedupe - https://phabricator.wikimedia.org/T363965#10247100 (10AnnWF) Some failMail related: Return value of CRM_Deduper_BAO_MergeHandler::getEmailConflicts() must be of the type array, null returned... [15:55:29] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247107 (10Ejegg) @RLopez-WMF Oops I think we need a fix to the WMF Thank You template. I... [15:57:21] PROBLEM - Host fran2001 is DOWN: PING CRITICAL - Packet loss = 100% [16:00:09] RECOVERY - Host fran2001 is UP: PING OK - Packet loss = 0%, RTA = 30.52 ms [16:01:20] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247122 (10Ejegg) OK, it might be a code issue - I'm not seeing where we feed the frequenc... [16:01:22] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247123 (10RLopez-WMF) @Ejegg i will add the other templates to my to do list for this wee... [16:02:10] fr-tech I'mma disable TY mails for a bit [16:02:23] I think we need a quick code fix to add the frequency_unit. [16:04:08] !log disabled fundraising Thank You mail send jobs [16:04:10] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [16:05:07] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 connecting: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [16:07:50] (03PS1) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) [16:07:55] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247172 (10RLopez-WMF) >>! In T371848#10247122, @Ejegg wrote: > OK, it might be a code iss... [16:08:15] Let me see if I can get the latest TY email template into our dev setups to test ^^^ [16:10:09] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 connecting: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [16:10:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 34.91, 32.93, 18.35 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:10:15] (03PS2) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) [16:10:30] OK, that's the template updated. Let's see what to run to get that into the DB [16:15:09] PROBLEM - check_ipsec on fran2001 is CRITICAL: Strongswan CRITICAL - ok: 0 connecting: frban2002_v4 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [16:15:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 37.30, 34.56, 22.90 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:15:45] maybe just managed reconcile? [16:18:19] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247224 (10Ejegg) Hi @RLopez-WMF looks like it's something I have to fix in the PHP code t... [16:18:41] RECOVERY - Host frban2002 is UP: PING OK - Packet loss = 0%, RTA = 30.42 ms [16:19:24] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1082019 (owner: 10L10n-bot) [16:20:07] RECOVERY - check_ipsec on fran2001 is OK: Strongswan OK - 1 ESP OK https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_ipsec [16:20:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 41.20, 37.67, 27.44 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:21:29] dang, managed reconcile isn't doing it [16:22:08] well, copypaste works to test for now [16:25:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 34.33, 34.08, 28.54 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:25:44] hmm, still not doing it [16:28:24] ok, it's getting into $params [16:30:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 10.14, 26.62, 27.51 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:31:11] (03CR) 10CI reject: [V:04-1] Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) (owner: 10Ejegg) [16:34:13] (03PS1) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082037 (https://phabricator.wikimedia.org/T371848) [16:34:21] ok, ^^^ is working for me locally [16:34:38] ah, let's see what that V-1 for the last PS was [16:34:52] ack, wait, why did that make a new changeset? [16:35:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 32.90, 27.50, 27.20 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:35:49] (03PS3) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) [16:35:54] (03Abandoned) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082037 (https://phabricator.wikimedia.org/T371848) (owner: 10Ejegg) [16:36:28] ok, V-1 was for the no such field error, that should be fixed in PS3 [16:39:33] PROBLEM - Host frdb2005 is DOWN: PING CRITICAL - Packet loss = 100% [16:40:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 31.20, 32.27, 29.68 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:41:27] fr-tech can anyone check this change? I'll need to either deploy that or roll back the TY mail template to a previous version before I can turn the thank you mailer back on: https://gerrit.wikimedia.org/r/1082005 [16:41:40] I've been able to smoke test it locally and get the expected tokens in the text [16:43:03] PROBLEM - Host payments2006 is DOWN: PING CRITICAL - Packet loss = 100% [16:44:55] ACKNOWLEDGEMENT - Host payments2006 is DOWN: PING CRITICAL - Packet loss = 100% Jeff_Green rack reog [16:45:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 35.89, 35.79, 32.01 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:45:25] ACKNOWLEDGEMENT - Host frdb2005 is DOWN: PING CRITICAL - Packet loss = 100% Jeff_Green rack reorg [16:50:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 22.71, 29.99, 30.69 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:55:09] PROBLEM - check_load on fran2001 is CRITICAL: LOAD CRITICAL - total load average: 0.45, 13.12, 23.50 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [16:55:10] gonna grab lunch but I'll be looking out for review on https://gerrit.wikimedia.org/r/1082005 in case anyone has a chance to look [16:59:21] (03CR) 10CI reject: [V:04-1] Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) (owner: 10Ejegg) [17:00:11] RECOVERY - Host frdb2005 is UP: PING OK - Packet loss = 0%, RTA = 30.49 ms [17:00:51] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06Content-Transform-Team-WIP, 10MediaWiki-extensions-CentralNotice, and 3 others: JavaScript building message boxes should now use mw.util.messageBox - https://phabricator.wikimedia.org/T375902#10247490 (10Jdlrobson) * CentralNotice was fixed in h... [17:01:45] dang, more CI fail [17:02:36] oh, template text in the test [17:08:53] RECOVERY - Host payments2006 is UP: PING OK - Packet loss = 0%, RTA = 30.43 ms [17:10:30] hmm, new template has no venmo username? [17:11:26] I can review when the CI gods are happy ejegg|food [17:11:57] thanks jgleeson [17:12:15] also scripts/civicrm-update-managed-entities.sh does the update you need [17:12:20] (03PS1) 10Wfan: Add descriptor for paypal onetime and recurring donations [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082048 (https://phabricator.wikimedia.org/T376077) [17:12:35] I'll add the venmo bit in test, and just mentioned to RLopex that we need it back in prod too [17:12:40] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247571 (10Ejegg) I think we also need to add the Venmo username conditional back to the T... [17:13:13] jgleeson: ohh, does that do something special to get the translations? [17:13:18] looking [17:13:19] (03PS2) 10Wfan: Add descriptor for paypal onetime and recurring donations [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082048 (https://phabricator.wikimedia.org/T376077) [17:13:58] oh no [17:14:01] it just wraps the main call [17:14:37] well if it works for you all the better! i had to cut-n-paste, not sure why [17:15:04] sorry. no I haven't even pulled down the patc yet [17:15:06] patch* [17:15:18] I didn't realise it wasn't working [17:15:24] (the command) [17:15:27] misread the commit [17:25:08] RECOVERY - check_load on fran2001 is OK: LOAD OK - total load average: 0.06, 0.10, 3.65 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fran2001&service=check_load [17:26:12] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247684 (10RLopez-WMF) Hey @Ejegg I was asked by DR to remove the venmo_user_name part due... [17:34:48] (03PS4) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) [17:51:26] (03PS5) 10Ejegg: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) [17:51:44] ok jgleeson I think PS5 is the one that will pass in CI [17:52:07] new template has some extra conditionals around formatting USD vs non-USD receipts [17:56:08] (03PS1) 10Wfan: Do nothing if no email differences [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082060 (https://phabricator.wikimedia.org/T363965) [17:58:38] (03PS1) 10Jgleeson: Remove duplicated test [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082061 [17:58:38] (03PS1) 10Jgleeson: Capture Payment Orchestrator Reconciliation ID. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082062 (https://phabricator.wikimedia.org/T374144) [17:58:44] looking ejegg [17:59:06] (03CR) 10CI reject: [V:04-1] Capture Payment Orchestrator Reconciliation ID. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082062 (https://phabricator.wikimedia.org/T374144) (owner: 10Jgleeson) [18:01:36] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10247855 (10Ejegg) Ahh, gotcha. Thanks! I think we can have the PHP fix deployed soon. [18:05:09] (03PS2) 10Jgleeson: Capture Payment Orchestrator Reconciliation ID. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1082062 (https://phabricator.wikimedia.org/T374144) [18:05:53] (03CR) 10Wfan: [C:03+2] "LGTM, fail page with "Your transaction could not be accepted" is now display" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1081173 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:08:29] (03PS1) 10Jgleeson: Add new Payment Orchestrator Reconciliation ID field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082064 (https://phabricator.wikimedia.org/T374144) [18:11:33] rebuilding civi ejegg to pull in those templates. probably be quick enough [18:12:31] k [18:13:52] (03CR) 10Wfan: [C:03+2] "Seems like we need to update this error message if we have more params check in the future, maybe add a todo here? But great to have the e" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1081177 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:14:24] (03Merged) 10jenkins-bot: Log error when sending bad params to createPayment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1081177 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:19:09] thanks wfan [18:20:09] ok that's built [18:20:11] 6m not bad [18:21:09] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10247913 (10IAckerman-WMF) [18:21:12] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10247915 (10Jgreen) @IAckerman-WMF & @ERoden-WMF: did ITS send Ilse a yubikey with the laptop? [18:22:07] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10247916 (10IAckerman-WMF) I have a Yubikey already, so we can skip IT sending me one. [18:25:11] (03CR) 10Jgleeson: [C:03+2] "Working well for me!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) (owner: 10Ejegg) [18:26:13] looks like wfan added this handy script https://github.com/wikimedia/wikimedia-fundraising-dev/blob/ad29c1751c80e910f0a33627be24f0955cd564f7/scripts/send-mails.sh#L5-L4 [18:26:20] we might wanna break those up [18:26:25] 2 for the price one 1 [18:26:28] of* [18:26:38] thanks jgleeson ! [18:26:46] np [18:26:53] I will prep the deploy [18:30:27] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1082068 [18:31:06] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1082068 (owner: 10Ejegg) [18:32:13] (03CR) 10Wfan: [C:03+2] "Skip update for cancelled donation looks right to me, while I am not see how is this related with that paypal cancel_date?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1081414 (https://phabricator.wikimedia.org/T373162) (owner: 10Ejegg) [18:40:21] (03Merged) 10jenkins-bot: Send donor to fail page on Adyen fail [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1081173 (https://phabricator.wikimedia.org/T374132) (owner: 10Ejegg) [18:43:42] I pushed up a job into process control that will check the data upload has completed (if someone could OK it) - it should cause a fail mail in about an hour & if that works I'll push up the job that prevents it [18:46:58] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10247993 (10Jgreen) [18:48:42] (03Merged) 10jenkins-bot: Add frequency_unit to params for TY mails [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082005 (https://phabricator.wikimedia.org/T371848) (owner: 10Ejegg) [18:48:43] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1082068 (owner: 10Ejegg) [18:48:45] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248006 (10Jgreen) >>! In T377039#10247916, @IAckerman-WMF wrote: > I have a Yubikey already, so we can skip IT sending me one. Is it the same one we set up before, for T357456? [18:51:21] !log fundraising civicrm upgraded from cfb0def0 to 36660cb3 [18:51:24] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:51:27] ok, let's send out some TY mails [18:54:32] (03Merged) 10jenkins-bot: Drop subscr_failed messages for cancelled recurrings [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1081414 (https://phabricator.wikimedia.org/T373162) (owner: 10Ejegg) [19:07:33] ok, those look good. turning the job back on. [19:09:00] eileen: I think that job is fine - shouldn't need the '-' in front of a single command but it will worok [19:09:14] I'mma push out process control to turn the TY mail back on [19:11:51] !log re-enabled fundraising thank you mailer [19:11:54] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:14:05] ah you pushed it out already [19:16:44] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 13Patch-For-Review, 07recurring upgrade: Update text for recurring upgrade thank you email - https://phabricator.wikimedia.org/T371848#10248089 (10Ejegg) OK @RLopez-WMF I've deployed the PHP fix and checked some recently-sent... [19:52:17] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Acoustic SMS stage 4 re-map our export mapping - https://phabricator.wikimedia.org/T376690#10248222 (10XenoRyet) 05Open→03Resolved [19:53:02] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Annual Recurring - Tagging and segmenting in Civi and Acoustic - https://phabricator.wikimedia.org/T372378#10248229 (10XenoRyet) 05Open→03Resolved [19:53:20] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Set up production Gravy Adyen 3DSecure Scheme config - https://phabricator.wikimedia.org/T375214#10248240 (10XenoRyet) 05Open→03Resolved [19:53:52] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Add refund activity with refund reason - https://phabricator.wikimedia.org/T373555#10248243 (10XenoRyet) 05Open→03Resolved [19:54:37] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248250 (10IAckerman-WMF) Yep, same one! [19:54:38] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Recurring-Donations: Implement Post-Payment Monthly Convert variant for testing - https://phabricator.wikimedia.org/T371523#10248251 (10XenoRyet) 05Open→03Resolved [19:54:43] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Acoustic SMS stage 2 Update api calls - https://phabricator.wikimedia.org/T376689#10248252 (10XenoRyet) 05Open→03Resolved [19:55:12] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Acoustic SMS stage 2 Update api calls - https://phabricator.wikimedia.org/T376689#10248255 (10XenoRyet) 05Resolved→03Open [19:55:29] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Civi point version update - https://phabricator.wikimedia.org/T374485#10248258 (10XenoRyet) 05Open→03Resolved [19:55:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Civi-Dedupe, 13Patch-For-Review: Fail mail on dedupe - https://phabricator.wikimedia.org/T363965#10248260 (10Eileenmcnaughton) The specific contact we are hitting an issue on is 59517950 - the contact has a snooze set which is causing the hiccup... [19:56:18] (03CR) 10Eileen: "Just noting that these grumpy type hints actually exposed errors in other cases - in this case the issue seems to be on the custom field (" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082060 (https://phabricator.wikimedia.org/T363965) (owner: 10Wfan) [19:56:25] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07Epic, 07payments-orchestration: Ensure we are receiving notifications on Venmo transactions via Gravy webhooks - https://phabricator.wikimedia.org/T376799#10248261 (10XenoRyet) 05Open→03Resolved [19:56:46] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 07payments-orchestration: Gravy PayPal Webhooks - https://phabricator.wikimedia.org/T376473#10248266 (10XenoRyet) 05Open→03Resolved [19:57:50] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog: Acoustic SMS stage 2 Update api calls - https://phabricator.wikimedia.org/T376689#10248274 (10Eileenmcnaughton) 05Open→03Resolved [20:00:22] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.43-notes (1.43.0-wmf.27; 2024-10-15): Monthly convert buttons leave modal area on smaller phone screens - https://phabricator.wikimedia.org/T375545#10248285 (10XenoRyet) 05Open→03Resolved [20:00:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.43-notes (1.43.0-wmf.27; 2024-10-15): Foundation logo overlapping monthly convert modal in smaller screen sizes - https://phabricator.wikimedia.org/T374074#10248287 (10XenoRyet) 05Open→03Resolved [20:01:11] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Acoustic SMS stage 2 Data merge on email to acoustic is not updating name - https://phabricator.wikimedia.org/T376347#10248293 (10XenoRyet) 05Open→03Resolved [20:01:18] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Update dLocal India form to require house number field - https://phabricator.wikimedia.org/T374726#10248281 (10XenoRyet) 05Open→03Resolved [20:01:31] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Validate email snooze date to ensure it's not in the past - https://phabricator.wikimedia.org/T371526#10248299 (10XenoRyet) 05Open→03Resolved [20:02:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Banner Closed Status Code Increase Year over Year - https://phabricator.wikimedia.org/T371632#10248302 (10XenoRyet) 05Open→03Resolved [20:02:38] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), 13Patch-For-Review: Add Address field, Description of service and PAN for any donation above INR 250k INR /3K USD - https://phabricator.wikimedia.org/T370583#10248278 (10XenoRyet) 05Open→03Resolved [20:02:47] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06MediaWiki-Platform-Team, 10MediaWiki-Vendor, and 5 others: mediawiki/core and mediawiki/vendor both skip composer.lock checks - https://phabricator.wikimedia.org/T370380#10248306 (10XenoRyet) 05Open→03Resolved [20:02:51] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices: Snooze date is gone from summary page - https://phabricator.wikimedia.org/T376962#10248304 (10XenoRyet) 05Open→03Resolved [20:12:13] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248321 (10Jgreen) Great, I'll reactivate that key. Next step is for you to generate a new ssh keypair for use in the FR environment. Instructions are here. https://wikitech.wikimedia.org/wiki/Fundraising/t... [20:13:00] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248322 (10Jgreen) [20:29:42] ejegg: can you check the new job. [20:29:42] Also - that dedupe job was just failing on one contact (now merged) - running ok on CLI now [20:30:54] ah ok - can you remind me how you find the contact it was failing on? [20:31:12] I looked up the CID from the setting and tried doing a deduper search starting there [20:31:35] but the first few ones that showed up didn't seem to have any email issues [20:47:46] eileen: that new job looks fine [21:02:20] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248434 (10IAckerman-WMF) Thanks. The public key is `cccccbdfehjr`. [21:10:13] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248447 (10Jgreen) >>! In T377039#10248434, @IAckerman-WMF wrote: > Thanks. The public key is `cccccbdfehjr`. Please post the ssh public key too. [21:13:46] 10fundraising-tech-ops, 10BDC-Implementation: [dbt] unblock packages install - https://phabricator.wikimedia.org/T377369#10248451 (10Davidbooke) @Jgreen Just wanted to give an update that I was able to install the dbt packages via the proxy and tarball approach. [21:40:48] ejegg: yeah so basically I bisect it [21:41:43] I did fix the one from before though so you won't find it [21:43:26] !log config revision changed from d240bcfb to a1c7759c [21:43:28] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [21:44:52] 10fundraising-tech-ops: Fundraising Access For Ilse Ackerman - https://phabricator.wikimedia.org/T377039#10248529 (10IAckerman-WMF) Here you go: `ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIMQvSzk2WIncyDP2sc52U9mxHag9u2jta+5+mRahMZB6 iackerman@wikimedia.org` ✨ [21:52:42] (03PS1) 10Eileen: Fix if grouping [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1082098 (https://phabricator.wikimedia.org/T377605) [21:52:55] ^^ is a partial fix but would be good to push out [22:16:22] 06Fundraising-Backlog, 13Patch-For-Review: Engage import_Matching - Ind contact import *updated - https://phabricator.wikimedia.org/T377605#10248575 (10Eileenmcnaughton) OK - I took a look and the mapping is messed up here - the organization contact ID - ie 246526 is mapped to the contribution ID - so it ha... [22:21:22] !log config revision changed from a1c7759c to 3bbf553d [22:21:24] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [22:26:14] 06Fundraising-Backlog, 13Patch-For-Review: Engage import_Matching - Ind contact import *updated - https://phabricator.wikimedia.org/T377605#10248600 (10Eileenmcnaughton) The mapping error seems to be specifically in that import - rather than the underlying import - @Elbar53 did you adjust the mapping while imp... [22:34:49] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failed to load zendesk - https://phabricator.wikimedia.org/T377788 (10AnnWF) 03NEW [22:35:34] 06Fundraising-Backlog, 13Patch-For-Review: Engage import_Matching - Ind contact import *updated - https://phabricator.wikimedia.org/T377605#10248617 (10Eileenmcnaughton) OK - I've put that donation against the original contact and got rid of the extra soft credits. There are still some floating payment records... [22:39:01] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failed to load zendesk - https://phabricator.wikimedia.org/T377788#10248618 (10AnnWF) [22:40:02] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Failed to load zendesk - https://phabricator.wikimedia.org/T377788#10248619 (10AnnWF) [23:04:15] 03Fundraising Sprint: trickOrTreat(), 06Fundraising-Backlog, 10FR-donorservices: Civi "thank you letter summary" feature request - https://phabricator.wikimedia.org/T299096#10248713 (10Eileenmcnaughton) @CDenes_WMF assuming you are not planning on changing the text this year I don't think we need any localiz...