[00:22:53] (03PS3) 10Wfan: add is_opt_id option as to deduper ui with ability to edit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/967066 [00:23:37] 10Fundraising Sprint Uptown cURL, 10Fundraising-Backlog, 10FR-donorservices, 10Patch-For-Review: Deduper and legacy merging overwriting issue - https://phabricator.wikimedia.org/T345206 (10AnnWF) [00:25:15] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-donorservices: URGENT: Recent Paypal donations without an email address in Civi! - https://phabricator.wikimedia.org/T348760 (10AnnWF) Latest donation with no email was on Oct 29th, and already emailed Paypal team for investigation [00:39:31] 10Fundraising Sprint Uptown cURL, 10Fundraising-Backlog, 10FR-donorservices, 10Patch-For-Review: Deduper and legacy merging overwriting issue - https://phabricator.wikimedia.org/T345206 (10AnnWF) @SHust Hi Sandra, our current logic for is_opt_out to silverpop is either conctact have is_opt_out true, mark i... [00:40:52] (03CR) 10Eileen: [C: 03+2] add is_opt_id option as to deduper ui with ability to edit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/967066 (owner: 10Wfan) [00:41:14] Thanks Eileen~ and here is the note for Sandra https://phabricator.wikimedia.org/T345206#9303899 [00:57:46] (03Merged) 10jenkins-bot: add is_opt_id option as to deduper ui with ability to edit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/967066 (owner: 10Wfan) [03:25:18] (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/971339 [03:25:33] (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/971339 (owner: 10Eileen) [03:26:22] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/971339 (owner: 10Eileen) [04:01:14] !log civicrm upgraded from 84ec2957 to 5be02f1b [04:01:20] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [05:24:37] 10Fundraising Sprint Uptown cURL: Enable all payment methods for LATAM pre-test in Q2 - https://phabricator.wikimedia.org/T349857 (10RKumar_WMF) @AnnWF - Great. Thank you. [08:00:44] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971365 (owner: 10L10n-bot) [08:10:18] (03CR) 10Raimond Spekking: [C: 03+2] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971365 (owner: 10L10n-bot) [11:22:34] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Successful Adyen transactions not making it into CiviCRM - https://phabricator.wikimedia.org/T345736 (10SBorriello) [11:23:09] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Successful Adyen transactions not making it into CiviCRM - https://phabricator.wikimedia.org/T345736 (10SBorriello) [11:57:39] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Successful Adyen transactions not making it into CiviCRM - https://phabricator.wikimedia.org/T345736 (10SBorriello) [13:57:52] (03CR) 10Jgleeson: "Thanks for the updates! It looks really close. I have an additional ask on the new code and a question about the uncommented key." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/968725 (https://phabricator.wikimedia.org/T348544) (owner: 10Damilare Adedoyin) [14:26:05] (03CR) 10Jgleeson: Upgrade recur amount when donor upgrades their recurring in Fundraiseup (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/968725 (https://phabricator.wikimedia.org/T348544) (owner: 10Damilare Adedoyin) [14:27:26] (03CR) 10Jgleeson: Upgrade recur amount when donor upgrades their recurring in Fundraiseup (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/968725 (https://phabricator.wikimedia.org/T348544) (owner: 10Damilare Adedoyin) [14:40:10] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-Email: Civi to Acoustic Export: Job Status Failed - https://phabricator.wikimedia.org/T350386 (10LMedley-WMF) Thank you so much, @Cstone and @KHaggard!! :-) [14:50:04] (03PS1) 10Jgleeson: Add unit test for Recurring Downgrade scenario [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/971462 (https://phabricator.wikimedia.org/T348544) [14:57:25] (03PS1) 10Jgleeson: Tweak exception messages for recurring upgrade checks [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/971466 (https://phabricator.wikimedia.org/T348544) [15:03:52] 10Fundraising-Backlog, 10fundraising-tech-ops, 10Patch-For-Review: Set up Community CiviCRM instance. - https://phabricator.wikimedia.org/T343486 (10greg) Because it took me too long to find this morning, here's the task for the production VM request: {T349402} [15:24:45] (03PS20) 10Wfan: Add api to remove recurring payment token then delete customer from Venmo vault for one-time donor [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/963853 (https://phabricator.wikimedia.org/T347606) [15:50:17] 10Fundraising-Backlog: Venmo non-native browser flow posing checkout issues for donors - https://phabricator.wikimedia.org/T348699 (10EMartin) Second pretest scheduled for Monday 6 Nov. [15:55:52] (03PS4) 10Ejegg: Upgrade civitoken [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/970892 (https://phabricator.wikimedia.org/T350208) (owner: 10Eileen) [15:55:57] (03CR) 10Ejegg: [C: 03+2] Upgrade civitoken [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/970892 (https://phabricator.wikimedia.org/T350208) (owner: 10Eileen) [15:59:23] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog (Android Release - FY2023-24), 10Wikipedia-iOS-App-Backlog (Design), 10iOS Release FY2023-24: For smaller screens (SE) ensure that the ability to scroll is visible to users - https://phabricator.wikimedia.org/T348695 (10cmadeo) @Tsevener look... [16:14:20] (03Merged) 10jenkins-bot: Upgrade civitoken [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/970892 (https://phabricator.wikimedia.org/T350208) (owner: 10Eileen) [16:48:06] oh darn, trying the latest civiproxy patch locally and getting errors [16:48:10] let's see [16:59:14] (03PS1) 10Wfan: Add normalized property for auto-rescue refusal reason [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/971493 (https://phabricator.wikimedia.org/T340159) [17:03:29] 10Fundraising-Backlog, 10Recurring-Donations: Auto-Rescue Edge Cases - https://phabricator.wikimedia.org/T350501 (10Cstone) [17:05:25] (03PS9) 10Wfan: Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) [17:07:42] well, i was able to install a lappy firmware upgrade without trashing grub this time, anyway [17:07:53] ok, back to looking at those civiproxy errors [17:09:43] (03PS2) 10Wfan: Add normalized property for auto-rescue refusal reason [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/971493 (https://phabricator.wikimedia.org/T340159) [17:09:50] (03CR) 10CI reject: [V: 04-1] Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [17:15:23] (03CR) 10Cstone: [C: 03+2] "thanks!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/971493 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [17:16:14] (03Merged) 10jenkins-bot: Add normalized property for auto-rescue refusal reason [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/971493 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [17:33:45] (03PS10) 10Wfan: Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) [17:51:18] (03CR) 10CI reject: [V: 04-1] Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [17:53:03] ahh i can't debug through because my civiproxy image is back on buster and I already updated the xdebug config [17:54:01] fr-tech it would be nice to get this one merged, to get our dev civiproxy on the same php version as prod: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/970006 [17:57:24] looking ejegg [17:57:53] ah thanks jgleeson - i haven't made any changes, just filed the phab task to rename the 'mediawiki' image [17:59:59] motion denied [18:00:06] j/k :) [18:00:09] :P [18:00:27] hmm I still need to figure out why my local docker doesn't like those platform config keys [18:00:37] I'm guessing upgrading docker will clear it [18:01:17] ah right, as you sleuthed out, they really aren't in the v3 schema [18:03:01] (03PS11) 10Wfan: Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) [18:06:20] well, ok, the recur upgrade forms work again once I got civiproxy back on bullseye [18:06:33] so let me just try to test a couple more auth flows [18:07:05] cool, header and xheader both seem to work [18:08:01] ah, param flow is busted [18:08:12] seems to be missing the param names, just sending the values [18:08:28] I was gonna say do you have good url to test with the new image [18:09:10] jgleeson: so you can do a basic check by following the 'Email Preferences link' from any contact in your local wmff civi [18:09:25] that will load an email prefs page that uses civiproxy to get the info [18:13:12] ah nice [18:13:23] just reinstallin pref cntr [18:13:25] (03PS5) 10Ejegg: Pass through site_key in config copy of civiproxy [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/970411 (https://phabricator.wikimedia.org/T349946) [18:15:46] (03PS7) 10Ejegg: Support site_key and multiple auth flows [wikimedia/fundraising/crm/civiproxy] - 10https://gerrit.wikimedia.org/r/970443 (https://phabricator.wikimedia.org/T349946) [18:17:43] so... not expecting legacyrest to actually work internally, but let's try it [18:18:58] hmm, that's not actually trying legacyrest [18:19:06] did the overlay mount fail? [18:19:21] (03CR) 10CI reject: [V: 04-1] Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [18:19:23] seems to be mounted [18:20:10] guh, mounted but not showing the same file contents as the external file [18:20:20] git stash did something spooky [18:20:49] oh I see why it's not found SmashPig\PaymentProviders\Responses\CreatePaymentWithProcessorRetryResponse, needs to add version for civi~ [18:21:19] oh hey, legacyrest DOES work on the api4 endpoint [18:21:20] nice [18:21:35] ok, that's all the internal flows smoke tested [18:23:25] so the 1001 albums project has pointed me to Nick Drake today, and I gotta say it's not as mopey as I was expecting [18:23:47] lyrics are not all smiles, but there's some pretty lively rhythm behind them [18:24:09] Digging the piano and guitar on 'Man In a Shed' [18:25:29] so let's see, to test the other external auth flows I guess I could just modify the CiviProxyConnect class [18:27:03] sad story re:Nick Drake ejegg [18:27:04] oh jeez, we're putting those keys in class constants? [18:27:09] never heard of him before [18:27:12] oh jgleeson ? [18:27:25] I'll give https://en.wikipedia.org/wiki/Life_in_a_Northern_Town a listen [18:27:39] I'd never listened, just always had the idea his stuff was all really depressing [18:28:01] probably just pigeonholed him based on who i heard talking about him in high school [18:28:07] that's the jist I get reading the summary https://en.wikipedia.org/wiki/Nick_Drake [18:29:08] ah man my civi is broke now [18:29:13] hmm lemme make a new donation [18:29:26] civiproxy -> email pref and now -> civi [18:29:41] ah dang [18:30:07] SELECT ct.id, ct.label FROM civicrm_tag ct INNER JOIN civicrm_entity_tag et ON ( ct.id = et.tag_id AND et.entity_id = 848 AND et.entity_table = 'civicrm_contact' AND ct.is_tagset = 0 ) [nativecode=1054 ** Unknown column 'ct.label' in 'field list'] [18:30:11] would a vidchat help debug the brokenness? [18:30:21] ohhh [18:30:27] probably needs a cvupdb [18:30:28] I'll just rebuild. I'm guessing it's a new field I'm missing [18:30:32] ah ok yeah [18:30:57] we just merged an upgrade from i think 5.66 to 5.68rc [18:31:06] ah cool [18:34:24] it's running but taking its time [18:35:36] docker@civicrm:/$ drush @wmff cvupdb [18:35:38] Xdebug: [Step Debug] Could not connect to debugging client. Tried: 172.18.0.1:9000 (through xdebug.client_host/xdebug.client_port) :-( [18:35:40] Upgrade outputs: [18:35:42] docker@civicrm:/$ [18:35:48] not chatty today [18:36:41] ah I must need a drupal db update [18:36:44] Unknown column 'wmf_external_contact_identifiers.venmo_user_name' in 'field list'] [18:37:08] hmm [18:37:12] no updates required [18:37:39] ah, that might be an extension upgrade [18:37:46] cvapi extension.upgrade [18:38:00] does that run them all? [18:38:04] or even our special custom fields sync [18:38:19] I thought I'd done this to review the change [18:38:20] jgleeson: extension.upgrade just runs db updates for extensions [18:38:40] drush @wmff cvapi WMFConfig.SyncCustomFields version=4 [18:38:49] is our 'special' custom fields sync [18:38:56] pretty sure I did that to review the venmo external identifier patch ha [18:38:59] I think that's actually the one you need for the venmo user name [18:39:03] oh weird [18:39:34] running again [18:39:42] and then I'll try extension upgrade [18:40:32] whoa, van morrison has a new album out today? [18:40:40] had no idea he was still releasing stuff [18:40:46] https://en.wikipedia.org/wiki/Accentuate_the_Positive_(Van_Morrison_album) [18:42:14] wow [18:44:08] (03PS1) 10Cstone: Add more logging [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971505 [18:48:29] extension upgrade still running [18:50:26] (03PS2) 10Cstone: Add more logging [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971505 [18:52:53] I should have rebuilt [18:52:56] :> [18:57:41] (03CR) 10Jgleeson: [C: 03+2] "Working locally for me! thanks" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971505 (owner: 10Cstone) [18:57:50] thanks jgleeson ! [18:58:18] np [18:58:58] ejegg: I'm just gonna rebuild civicrm. this is killing me [19:03:28] so buildkit is broke now [19:03:35] composer install is failing [19:03:57] oh jeez [19:04:04] version conflict? [19:05:10] (03Merged) 10jenkins-bot: Add more logging [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/971505 (owner: 10Cstone) [19:06:17] ahh silly me i was modfying civiproxyconnect to send authx style params instead of legacyrest params, and i swapped the api key and site key [19:06:25] took me half an hour to figure that out [19:07:05] removin the vendor dir fixed it [19:07:07] https://phabricator.wikimedia.org/F41439383 [19:07:22] oh huh [19:12:54] (03PS1) 10Cstone: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/971515 [19:12:59] im gona deploy that log change [19:13:08] (03PS12) 10Wfan: Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) [19:13:18] (03CR) 10CI reject: [V: 04-1] Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) (owner: 10Wfan) [19:13:51] (03CR) 10Cstone: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/971515 (owner: 10Cstone) [19:14:23] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/971515 (owner: 10Cstone) [19:14:41] (03PS13) 10Wfan: Add log and cancel_reason to indicate the failed reason [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/958563 (https://phabricator.wikimedia.org/T340159) [19:15:46] !log payments-wiki upgraded from cf9f8e52 to 1d66a20f [19:15:48] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:20:05] ok, that's all the external auth methods tested and working except for the standard Authorization header [19:20:14] so... is that Apache removing it? [19:21:10] ah right, I can get rid of looking for a 'Basic: ' prefix - that's only for username/pw [19:21:30] which neither civi nor civiproxy supports [19:23:37] ah, just for consistency I guess I shoudl prefix those fns with civiproxy_ [19:25:13] eh, and maybe move the header fn into the same file with the get_parameters [19:31:19] finally [19:31:22] it worked! [19:34:52] woohoo! [19:38:43] (03CR) 10Jgleeson: [C: 03+2] "Working locally for me. Thanks for the renaming ticket!" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/970006 (owner: 10Ejegg) [19:40:01] argggghh i'm getting really frequent gnome crashes [19:40:20] "Oh no! Something has gone wrone and the system can't recover" [19:40:34] like 5 times today [19:40:38] have to log out and log in again [19:41:02] this last time i was only running shell, firefox, and phpstorm [19:41:44] thanks for the CR jgleeson ! [19:42:45] could uninstall tweak tool i guess [19:45:30] np [19:45:33] right calling it a night [19:45:38] have a good weekend fr-tech [19:45:49] bye for now [19:49:28] k, working on this compy is really frustrating right now [19:49:42] and i want to get out of the house at least once before the sun goes down [19:49:58] I think I may just head out early and take a few more hours of PTO [19:50:13] let's just push up one more pair of patches for civiproxy [19:50:51] (03CR) 10Ejegg: [V: 03+2] Use bullseye for civiproxy [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/970006 (owner: 10Ejegg) [19:58:04] (03PS8) 10Ejegg: Support site_key and multiple auth flows [wikimedia/fundraising/crm/civiproxy] - 10https://gerrit.wikimedia.org/r/970443 (https://phabricator.wikimedia.org/T349946) [20:01:53] ok fr-tech, I am going to spend another few hrs of PTO. Have a good weekend! [20:55:48] 10Fundraising Sprint Uptown cURL: Enable all payment methods for LATAM pre-test in Q2 - https://phabricator.wikimedia.org/T349857 (10Pcoombe) I've added the requested additional payment methods. Adding @HNordeenWMF for info. Note that Apple Pay only shows up if you use Safari browser (or include `&forceApplePay=... [21:09:48] 10fundraising-tech-ops: encrypt fundraising database client->server communication - https://phabricator.wikimedia.org/T170321 (10Jgreen) [21:09:51] 10Fundraising Sprint Interstitial ads halfway down the coaster hill, 10Fundraising Sprint Ketchup Flume Ride, 10Fundraising Sprint Owner of a Lonely Product, 10Fundraising-Backlog, and 3 others: Enable SSL for CiviCRM DB connections - https://phabricator.wikimedia.org/T280080 (10Jgreen) 05Resolved→03Ope... [21:31:32] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-Email: Civi to Acoustic Export: Job Status Failed - https://phabricator.wikimedia.org/T350386 (10KHaggard) 05Open→03Resolved a:03KHaggard This looks to be resolved as we had successful import jobs today. Thanks everyone! Resolving this task.