[00:03:06] (03PS2) 10Eileen: Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) [00:03:06] (03PS2) 10Eileen: Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) [00:03:06] (03PS1) 10Eileen: Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) [00:03:41] I think once the listener is merged failures like this might stop [00:03:41] 11:54:13 1) RefundTest::testMarkRefundWithType [00:03:41] 11:54:13 Civi\API\Exception\UnauthorizedException: Authorization failed: CiviCRM APIv4 (Contribution::get) [00:05:44] (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/1056628 [00:05:56] (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/1056628 (owner: 10Eileen) [00:07:07] (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/1056628 (owner: 10Eileen) [00:11:23] !log civicrm upgraded from c656ab2f to 1dc4f944 [00:11:26] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [00:44:35] (03PS2) 10Eileen: Move maxContributionID, maxContactID to our test trait [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056278 [00:45:06] (03CR) 10Ejegg: [C:03+2] Move maxContributionID, maxContactID to our test trait [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056278 (owner: 10Eileen) [00:50:34] (03PS8) 10Eileen: Slight improvement in test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056260 [00:50:43] (03CR) 10Ejegg: [C:03+2] Slight improvement in test (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056260 (owner: 10Eileen) [00:50:58] (03PS4) 10Eileen: Remove unnecessary setup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056280 [00:51:01] (03CR) 10Ejegg: [C:03+2] Remove unnecessary setup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056280 (owner: 10Eileen) [00:55:38] (03PS1) 10Cstone: Rename number to house number [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1056633 (https://phabricator.wikimedia.org/T370583) [01:01:19] (03PS2) 10Eileen: Do not use transactional interface in geocode [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056443 [01:01:23] (03CR) 10Ejegg: [C:03+2] Do not use transactional interface in geocode [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056443 (owner: 10Eileen) [01:01:47] (03PS2) 10Eileen: Remove transactional from OmniMail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056453 [01:01:50] (03CR) 10Ejegg: [C:03+2] Remove transactional from OmniMail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056453 (owner: 10Eileen) [01:04:42] (03Merged) 10jenkins-bot: Move maxContributionID, maxContactID to our test trait [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056278 (owner: 10Eileen) [01:10:44] (03PS19) 10Eileen: Add test listener to make test clean up more consistent [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1054971 (https://phabricator.wikimedia.org/T370371) [01:10:46] (03Merged) 10jenkins-bot: Slight improvement in test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056260 (owner: 10Eileen) [01:10:47] (03Merged) 10jenkins-bot: Remove unnecessary setup [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056280 (owner: 10Eileen) [01:10:57] (03CR) 10Ejegg: [C:03+2] Add test listener to make test clean up more consistent [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1054971 (https://phabricator.wikimedia.org/T370371) (owner: 10Eileen) [01:11:33] (03PS3) 10Eileen: stop stray contact being left in fundraise audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056446 [01:11:41] (03CR) 10Ejegg: [C:03+2] stop stray contact being left in fundraise audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056446 (owner: 10Eileen) [01:12:03] (03PS2) 10Eileen: Ensure very unique gets deleted [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056455 [01:12:05] (03CR) 10Ejegg: [C:03+2] Ensure very unique gets deleted [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056455 (owner: 10Eileen) [01:14:04] ejegg: what is the status of the matching gifts drush job / configuration - ie admin/config/wmf_civicrm/matching_gifts [01:14:04] - the todo is 'delete or migrate to a setting' [01:14:52] (03PS3) 10Eileen: Fix DataCheck tests to not use transactional [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056583 [01:14:55] (03CR) 10Ejegg: [C:03+2] Fix DataCheck tests to not use transactional [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056583 (owner: 10Eileen) [01:20:08] (03Merged) 10jenkins-bot: Do not use transactional interface in geocode [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056443 (owner: 10Eileen) [01:20:09] (03Merged) 10jenkins-bot: Remove transactional from OmniMail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056453 (owner: 10Eileen) [01:27:05] (03CR) 10Ejegg: "Looks like a good first step! One little gotcha with the test setup" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1055981 (https://phabricator.wikimedia.org/T367807) (owner: 10Damilare Adedoyin) [01:30:30] (03Merged) 10jenkins-bot: Add test listener to make test clean up more consistent [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1054971 (https://phabricator.wikimedia.org/T370371) (owner: 10Eileen) [01:31:20] (03Merged) 10jenkins-bot: stop stray contact being left in fundraise audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056446 (owner: 10Eileen) [01:31:21] (03Merged) 10jenkins-bot: Ensure very unique gets deleted [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056455 (owner: 10Eileen) [01:32:36] (03Merged) 10jenkins-bot: Fix DataCheck tests to not use transactional [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056583 (owner: 10Eileen) [01:50:01] (03PS2) 10Eileen: Put Minnie on strike [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056593 [01:50:38] Wanna +2 this one too - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1056593 - it's the last in that chain - now over master - although I guess I did dead-name Daisy [01:51:47] (03PS2) 10Eileen: Copy wmf_civicrm.module address functions to ChecksFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056607 [01:51:48] (03PS4) 10Eileen: Do not call mark_refund from Audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056610 [01:51:48] (03PS5) 10Eileen: Move remaining address update functions from wmf_civicrm to WMFContact::Save [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056608 (https://phabricator.wikimedia.org/T365415) [01:51:48] (03PS3) 10Eileen: Fix refundTest to test via queue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056616 (https://phabricator.wikimedia.org/T365415) [01:51:49] (03PS3) 10Eileen: Long live Maisy (update WMF Donor Refund test to go via queue) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056617 (https://phabricator.wikimedia.org/T365415) [01:51:50] (03PS3) 10Eileen: Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) [01:51:54] (03PS3) 10Eileen: Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) [01:51:58] (03PS2) 10Eileen: Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) [01:52:02] (03PS1) 10Eileen: Move RefundTest tests from legacy class to RefundQueueTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056637 (https://phabricator.wikimedia.org/T365415) [01:52:06] (03PS1) 10Eileen: Fully remove RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056638 (https://phabricator.wikimedia.org/T365415) [01:52:10] (03PS1) 10Eileen: Move markRefund to the refund queue consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056639 (https://phabricator.wikimedia.org/T365415) [01:52:14] (03PS1) 10Eileen: Move permission check to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056640 (https://phabricator.wikimedia.org/T365415) [01:52:18] (03PS1) 10Eileen: Fold function back into only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056641 (https://phabricator.wikimedia.org/T365415) [01:52:22] (03PS1) 10Eileen: Move checkLanguageExists to WMFContact::save (only caller) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056642 (https://phabricator.wikimedia.org/T365415) [01:52:26] (03PS1) 10Eileen: Remove some calls to deprecated function from with test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056643 (https://phabricator.wikimedia.org/T365415) [01:52:30] (03PS1) 10Eileen: Move function to only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056644 (https://phabricator.wikimedia.org/T365415) [01:52:34] (03PS1) 10Eileen: Move first attempt at queue function to Omnimail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056645 (https://phabricator.wikimedia.org/T365415) [01:52:38] (03PS1) 10Eileen: Remove call to legacy function from test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056646 (https://phabricator.wikimedia.org/T365415) [01:52:42] (03PS1) 10Eileen: Upgrade our flock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056647 (https://phabricator.wikimedia.org/T365415) [02:25:11] (03CR) 10CI reject: [V:04-1] Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [02:33:59] (03CR) 10Ejegg: [C:03+2] Put Minnie on strike [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056593 (owner: 10Eileen) [02:34:26] (03CR) 10Ejegg: Setup Gravy webhook listener (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1055981 (https://phabricator.wikimedia.org/T367807) (owner: 10Damilare Adedoyin) [02:36:42] (03PS2) 10Eileen: Enable wfmfraud tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1034144 [02:36:51] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1034144 (owner: 10Eileen) [02:52:02] (03Merged) 10jenkins-bot: Put Minnie on strike [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056593 (owner: 10Eileen) [03:13:50] 06Fundraising-Backlog, 07I18n: Plural in fundraising email with "badges" - https://phabricator.wikimedia.org/T244873#10013585 (10srishakatux) @CDenes_WMF This sounds like an issue that was specific to an email sent to donors in 2020. Is this task still valid, or should it be closed as invalid? [03:38:15] (03PS1) 10Eileen: Fix WMFFraud tests to actually run (and pass) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056649 (https://phabricator.wikimedia.org/T365415) [03:38:40] 14Fundraising Sprint: justWork(), 14Fundraising Sprint: meow(), 03Fundraising Sprint: neveruse(), 06Fundraising-Backlog, and 4 others: Move a bunch of code out of wmf_civicrm.module - https://phabricator.wikimedia.org/T365415#10013590 (10Eileenmcnaughton) a:03Eileenmcnaughton [03:38:57] (03Abandoned) 10Eileen: Enable wfmfraud tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1034144 (owner: 10Eileen) [03:41:43] (03PS2) 10Eileen: Upgrade our flock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056647 (https://phabricator.wikimedia.org/T365415) [03:41:43] (03PS2) 10Eileen: Remove call to legacy function from test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056646 (https://phabricator.wikimedia.org/T365415) [03:41:43] (03PS1) 10Eileen: Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) [03:41:45] (03PS1) 10Eileen: Remove functions with no callers [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056651 [03:41:45] (03PS1) 10Eileen: Remove ContributionTrackingTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056652 (https://phabricator.wikimedia.org/T365415) [04:01:06] (03CR) 10CI reject: [V:04-1] Remove call to legacy function from test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056646 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [04:01:06] (03CR) 10CI reject: [V:04-1] Remove ContributionTrackingTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056652 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [04:02:10] (03CR) 10CI reject: [V:04-1] Remove functions with no callers [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056651 (owner: 10Eileen) [04:04:01] (03CR) 10CI reject: [V:04-1] Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [04:24:27] (03PS3) 10Eileen: Copy wmf_civicrm.module address functions to ChecksFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056607 [04:24:27] (03PS5) 10Eileen: Do not call mark_refund from Audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056610 [04:24:27] (03PS6) 10Eileen: Move remaining address update functions from wmf_civicrm to WMFContact::Save [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056608 (https://phabricator.wikimedia.org/T365415) [04:24:28] (03PS4) 10Eileen: Fix refundTest to test via queue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056616 (https://phabricator.wikimedia.org/T365415) [04:24:29] (03PS4) 10Eileen: Long live Maisy (update WMF Donor Refund test to go via queue) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056617 (https://phabricator.wikimedia.org/T365415) [04:24:30] (03PS4) 10Eileen: Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) [04:24:34] (03PS4) 10Eileen: Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) [04:24:38] (03PS3) 10Eileen: Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) [04:24:42] (03PS2) 10Eileen: Move RefundTest tests from legacy class to RefundQueueTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056637 (https://phabricator.wikimedia.org/T365415) [04:24:46] (03PS2) 10Eileen: Fully remove RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056638 (https://phabricator.wikimedia.org/T365415) [04:24:50] (03PS2) 10Eileen: Move markRefund to the refund queue consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056639 (https://phabricator.wikimedia.org/T365415) [04:24:54] (03PS2) 10Eileen: Move permission check to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056640 (https://phabricator.wikimedia.org/T365415) [04:24:58] (03PS2) 10Eileen: Fold function back into only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056641 (https://phabricator.wikimedia.org/T365415) [04:25:02] (03PS2) 10Eileen: Move checkLanguageExists to WMFContact::save (only caller) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056642 (https://phabricator.wikimedia.org/T365415) [04:25:06] (03PS2) 10Eileen: Remove some calls to deprecated function from with test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056643 (https://phabricator.wikimedia.org/T365415) [04:25:10] (03PS2) 10Eileen: Move function to only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056644 (https://phabricator.wikimedia.org/T365415) [04:25:14] (03PS2) 10Eileen: Move first attempt at queue function to Omnimail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056645 (https://phabricator.wikimedia.org/T365415) [04:25:18] (03PS3) 10Eileen: Remove call to legacy function from test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056646 (https://phabricator.wikimedia.org/T365415) [04:27:13] (03PS2) 10Eileen: Remove functions with no callers [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056651 [04:28:06] (03PS2) 10Eileen: Fix WMFFraud tests to actually run (and pass) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056649 (https://phabricator.wikimedia.org/T365415) [04:28:06] (03PS3) 10Eileen: Upgrade our flock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056647 (https://phabricator.wikimedia.org/T365415) [04:28:06] (03PS2) 10Eileen: Remove ContributionTrackingTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056652 (https://phabricator.wikimedia.org/T365415) [04:28:07] (03PS2) 10Eileen: Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) [05:10:24] (03CR) 10CI reject: [V:04-1] Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [09:23:15] 06Fundraising-Backlog: Scoping: 2nd Recurring Failure Email in Civi - https://phabricator.wikimedia.org/T365488#10013920 (10MSuijkerbuijk_WMF) @AKanji-WMF Thanks! This is clear. We can send the second email 7 days after the first one. @Cstone would that work for you? Also, I've added the correct URLs in the cont... [11:35:22] (03PS5) 10Jgleeson: WIP: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [11:43:07] (03PS6) 10Jgleeson: WIP: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [12:39:57] (03PS10) 10Damilare Adedoyin: Setup Gravy webhook listener [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1055981 (https://phabricator.wikimedia.org/T367807) [12:44:04] (03CR) 10Damilare Adedoyin: "Thanks for the CR ejegg and jgleeson. All comments have been resolved." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1055981 (https://phabricator.wikimedia.org/T367807) (owner: 10Damilare Adedoyin) [12:44:12] (03PS6) 10Damilare Adedoyin: WIP [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056531 (https://phabricator.wikimedia.org/T370797) [14:58:33] (03CR) 10Ejegg: [C:03+2] Fix WMFFraud tests to actually run (and pass) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056649 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:00:18] (03PS7) 10Jgleeson: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [15:01:36] (03PS8) 10Jgleeson: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [15:02:14] (03CR) 10Ejegg: [C:03+2] "Quack!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056647 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:02:29] fr-tech: that patch gives us zendesk tickets on civicrm. If anyone fancies reviewing it lemme know and I'll share my zendesk trial API credentials to help with testing [15:02:29] (03CR) 10Ejegg: [C:03+2] Remove ContributionTrackingTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056652 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:02:48] (until we get wmf sandbox api creds that work) [15:03:38] jgleeson ejegg around for architecture call? [15:03:46] oh yerah [15:03:48] -r [15:05:42] (03PS6) 10Jgleeson: Map gravy buyer ID to external_identifier queue message field. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1056527 (https://phabricator.wikimedia.org/T368039) [15:05:43] (03CR) 10Ejegg: [C:03+2] Map gravy buyer ID to external_identifier queue message field. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1056527 (https://phabricator.wikimedia.org/T368039) (owner: 10Jgleeson) [15:06:34] (03PS9) 10Jgleeson: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [15:06:39] ack, be right there [15:06:43] sorry damilare [15:16:55] (03Merged) 10jenkins-bot: Fix WMFFraud tests to actually run (and pass) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056649 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:20:04] (03Merged) 10jenkins-bot: Upgrade our flock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056647 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:22:41] (03Merged) 10jenkins-bot: Remove ContributionTrackingTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056652 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [15:30:12] (03PS3) 10Wfan: Force lowercase for buyer email for gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056009 (https://phabricator.wikimedia.org/T370700) [15:30:53] (03CR) 10Wfan: Force lowercase for buyer email for gravy (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056009 (https://phabricator.wikimedia.org/T370700) (owner: 10Wfan) [15:36:31] 14Fundraising Sprint: meow(), 03Fundraising Sprint: neveruse(), 06Fundraising-Backlog, 13Patch-For-Review: Add Zendesk tab to contact summary screen - https://phabricator.wikimedia.org/T368689#10015344 (10jgleeson) [15:38:33] \o/ pictures! [15:40:10] greg-g: https://wikimedia.slack.com/archives/G015YLP3BLP/p1721921981451049 [15:42:31] jgleeson: ugh, slack link isn't working for me, which message in donorrelations? [15:42:43] the latest [15:42:44] oh yours wasn't showing up! weird [15:43:17] that last query param on slack worries me https://app.slack.com/client/E012JBDTTHA/G015YLP3BLP?cdn_fallback=1 [15:43:42] cue the crowdstrike bluescreen [15:48:50] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw: Q1:rack/setup/install civi2002, frpig2002, frpm2002 - https://phabricator.wikimedia.org/T369937#10015377 (10Jhancock.wm) a:03Jhancock.wm [15:50:48] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw: Q1:rack/setup/install civi2002, frpig2002, frpm2002 - https://phabricator.wikimedia.org/T369937#10015395 (10Jhancock.wm) [16:45:48] (03PS10) 10Jgleeson: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [16:48:08] (03PS11) 10Jgleeson: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) [17:02:09] 06Fundraising-Backlog, 10FR-donorservices: Endowment 'Thank You' email/receipt sent in English to non-English donors - https://phabricator.wikimedia.org/T367594#10015680 (10CDenes_WMF) Hi @Ejegg, are these counts of all time? Not time boxed? Sweden we ran a Swedish and English Endowment campaign back in 2020... [17:02:12] 06Fundraising-Backlog, 07I18n: Plural in fundraising email with "badges" - https://phabricator.wikimedia.org/T244873#10015681 (10CDenes_WMF) can be closed invalid, we don't have badges anymore [17:06:34] (03CR) 10Jgleeson: [C:03+2] Rename number to house number [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1056633 (https://phabricator.wikimedia.org/T370583) (owner: 10Cstone) [17:08:29] (03Merged) 10jenkins-bot: Rename number to house number [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1056633 (https://phabricator.wikimedia.org/T370583) (owner: 10Cstone) [17:12:05] (03PS1) 10Cstone: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1056988 [17:21:40] (03CR) 10Cstone: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1056988 (owner: 10Cstone) [17:42:38] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1056988 (owner: 10Cstone) [17:46:47] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10015794 (10greg) @nisrael hey quick question, who manages this verification key? As in which account with Yahoo Sender Hub is creating it? My reading of the documentation says they will send a verif... [17:50:01] 06Fundraising-Backlog, 10fundraising-tech-ops: Consider moving our mail to a subdomain - https://phabricator.wikimedia.org/T365637#10015829 (10Dwisehaupt) [17:52:29] 06Fundraising-Backlog, 07I18n: Plural in fundraising email with "badges" - https://phabricator.wikimedia.org/T244873#10015833 (10srishakatux) 05Open→03Invalid [18:20:15] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), 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#10015904 (10CaitVirtue) @RKumar_WMF - Thanks, t... [18:25:37] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10015923 (10nisrael) Just so I'm not playing middle man, I'm tagging in @bsisolak who likely has more specific details on how to implement this. [18:29:20] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10015927 (10bsisolak) This will be in Message Digital's Yahoo Sender Hub. There will be no verification email to postmaster@wikimedia.org, just this DNS entry. As for timeline, if it takes longer tha... [18:34:00] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10015935 (10greg) Can you share the documentation from yahoo about this here? [18:57:10] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10015968 (10bsisolak) https://senders.yahooinc.com/complaint-feedback-loop/ and here https://blog.postmaster.yahooinc.com/post/750658900050706432/its-here-meet-the-new-sender-hub-dashboard [19:16:15] !log payments-wiki upgraded from a37746fe to 91624a2e [19:16:19] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:18:59] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MW-1.43-notes (1.43.0-wmf.16; 2024-07-30), 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#10016013 (10Cstone) Fields were added for the a... [19:39:31] (03PS3) 10Jgleeson: Preserve .idea config during reclones for Payments [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1056536 [19:39:31] (03PS1) 10Jgleeson: Preserve .idea config during reclones for CiviCRM WMFF [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1057007 [19:52:22] did donor relations used to be called donor services or did I make that up? [19:58:31] you are right [19:58:41] they changed it like 2 years ago? [19:58:43] thank you! [20:03:29] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: fundraising-dev: civicrm install errors - https://phabricator.wikimedia.org/T371054 (10jgleeson) 03NEW [20:06:04] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: fundraising-dev: civicrm install errors - https://phabricator.wikimedia.org/T371054#10016135 (10jgleeson) [20:11:32] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10016148 (10Dwisehaupt) @bsisolak Just want to make sure this is clear to me. The setup and configuration is happening with Yahoo, but through Message Digital? If that is correct, are there any contac... [20:17:39] (03CR) 10Jgleeson: "We could potentially upstream this or turn it into an ext for civicrm users" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) (owner: 10Jgleeson) [20:19:14] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10016163 (10bsisolak) There is no data access in Yahoo, and there is no way in Sender Hub to share that. The feedback loop emails are sent to Acoustic for processing. This is just setup, in which we p... [20:25:55] (03PS4) 10Eileen: Copy wmf_civicrm.module address functions to ChecksFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056607 [20:26:06] (03CR) 10Ejegg: [C:03+2] Copy wmf_civicrm.module address functions to ChecksFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056607 (owner: 10Eileen) [20:35:42] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10016236 (10Dwisehaupt) >>! In T370963#10016163, @bsisolak wrote: > This only applies to emails sent form Acoustic, if you ever wanted to setup FBLs for other senders on the domain, you can do that in... [20:37:40] (03PS7) 10Damilare Adedoyin: WIP: Setup listener for transaction event message. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056531 (https://phabricator.wikimedia.org/T370797) [20:38:08] (03CR) 10CI reject: [V:04-1] WIP: Setup listener for transaction event message. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056531 (https://phabricator.wikimedia.org/T370797) (owner: 10Damilare Adedoyin) [20:41:06] (03PS8) 10Damilare Adedoyin: WIP: Setup listener for transaction event message. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056531 (https://phabricator.wikimedia.org/T370797) [20:45:16] (03Merged) 10jenkins-bot: Copy wmf_civicrm.module address functions to ChecksFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056607 (owner: 10Eileen) [20:50:32] (03PS1) 10Cstone: WIP: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [20:52:23] (03CR) 10CI reject: [V:04-1] WIP: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [20:54:12] (03CR) 10Damilare Adedoyin: [C:03+2] Force lowercase for buyer email for gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056009 (https://phabricator.wikimedia.org/T370700) (owner: 10Wfan) [20:54:36] (03PS6) 10Eileen: Do not call mark_refund from Audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056610 [20:54:38] (03Merged) 10jenkins-bot: Force lowercase for buyer email for gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056009 (https://phabricator.wikimedia.org/T370700) (owner: 10Wfan) [20:54:49] (03CR) 10Damilare Adedoyin: [C:03+2] "Thanks for spotting that edge case, approved!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056009 (https://phabricator.wikimedia.org/T370700) (owner: 10Wfan) [20:55:00] thx dami~ [20:55:30] (03CR) 10Ejegg: [C:03+2] Do not call mark_refund from Audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056610 (owner: 10Eileen) [21:01:09] (03PS3) 10Eileen: Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) [21:03:12] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10016312 (10bsisolak) Ah... I'm only setting this up for the domain wikimedia.org with the DKIM selector of spop1014 (which is Acoustic). [21:10:36] (03PS7) 10Eileen: Move remaining address update functions from wmf_civicrm to WMFContact::Save [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056608 (https://phabricator.wikimedia.org/T365415) [21:10:38] (03CR) 10Ejegg: [C:03+2] Move remaining address update functions from wmf_civicrm to WMFContact::Save [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056608 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:12:16] (03PS5) 10Eileen: Fix refundTest to test via queue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056616 (https://phabricator.wikimedia.org/T365415) [21:12:23] (03CR) 10Ejegg: [C:03+2] Fix refundTest to test via queue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056616 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:14:13] (03Merged) 10jenkins-bot: Do not call mark_refund from Audit test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056610 (owner: 10Eileen) [21:15:10] (03PS5) 10Eileen: Long live Maisy (update WMF Donor Refund test to go via queue) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056617 (https://phabricator.wikimedia.org/T365415) [21:15:14] (03CR) 10Ejegg: [C:03+2] Long live Maisy (update WMF Donor Refund test to go via queue) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056617 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:18:23] oh Maisy it in!!! [21:19:10] (03CR) 10CI reject: [V:04-1] Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:21:21] (03PS5) 10Eileen: Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) [21:21:24] (03CR) 10Ejegg: [C:03+2] Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:29:34] (03Merged) 10jenkins-bot: Move remaining address update functions from wmf_civicrm to WMFContact::Save [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056608 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:31:24] (03Merged) 10jenkins-bot: Fix refundTest to test via queue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056616 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:33:53] (03Merged) 10jenkins-bot: Long live Maisy (update WMF Donor Refund test to go via queue) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056617 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:40:04] (03Merged) 10jenkins-bot: Remove deprecated function calls from RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056621 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [21:46:58] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw, 06SRE: Q1:rack/setup/install frqueue2003, pay-lb2001, pay-lb2002 - https://phabricator.wikimedia.org/T369566#10016447 (10Papaul) ` papaul@fasw-c-codfw# show | compare [edit interfaces interface-range disabled] - member ge-0/0/32; - member ge-1/0/32; [ed... [21:48:22] 10fundraising-tech-ops, 06SRE: Q1:rack/setup/install fransc2001 - https://phabricator.wikimedia.org/T367816#10016450 (10Papaul) [21:49:03] 10fundraising-tech-ops, 06SRE: Q1:rack/setup/install frand200[12] - https://phabricator.wikimedia.org/T367804#10016451 (10Papaul) [21:49:29] 10fundraising-tech-ops, 06SRE: Q1:rack/setup/install franio200[1-3] - https://phabricator.wikimedia.org/T367819#10016457 (10Papaul) [22:04:57] (03PS5) 10Eileen: Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) [22:05:25] (03CR) 10Ejegg: [C:03+2] Switch remining refund tests to use RefundQueue [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056620 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:07:47] (03PS4) 10Eileen: Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) [22:07:51] (03CR) 10Ejegg: [C:03+2] Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:09:14] (03PS3) 10Eileen: Move RefundTest tests from legacy class to RefundQueueTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056637 (https://phabricator.wikimedia.org/T365415) [22:09:16] (03CR) 10Ejegg: [C:03+2] Move RefundTest tests from legacy class to RefundQueueTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056637 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:09:26] (03PS3) 10Eileen: Fully remove RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056638 (https://phabricator.wikimedia.org/T365415) [22:09:28] (03CR) 10Ejegg: [C:03+2] Fully remove RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056638 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:13:29] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw, 06SRE: Q1:rack/setup/install fransw200[1-3].frack.codfw.wmnet - https://phabricator.wikimedia.org/T367800#10016559 (10Papaul) ` [edit interfaces interface-range disabled] - member ge-0/0/29; - member ge-0/0/30; - member ge-0/0/31; - member ge-1/0/2... [22:13:31] 10fundraising-tech-ops: Add a TXT record to the Yahoo sending domain - https://phabricator.wikimedia.org/T370963#10016560 (10Dwisehaupt) Oh. That makes more sense. I'm assuming you mean spop1024 since spop1014 does not exist. Let me have some time to figure out how to track this internally and build up a change... [22:14:59] (03PS3) 10Eileen: Move markRefund to the refund queue consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056639 (https://phabricator.wikimedia.org/T365415) [22:15:03] (03CR) 10Ejegg: [C:03+2] Move markRefund to the refund queue consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056639 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:16:32] (03PS3) 10Eileen: Move permission check to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056640 (https://phabricator.wikimedia.org/T365415) [22:16:35] (03CR) 10Ejegg: [C:03+2] Move permission check to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056640 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:17:40] (03PS3) 10Eileen: Fold function back into only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056641 (https://phabricator.wikimedia.org/T365415) [22:17:45] (03CR) 10Ejegg: [C:03+2] Fold function back into only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056641 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:22:22] (03PS3) 10Eileen: Move checkLanguageExists to WMFContact::save (only caller) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056642 (https://phabricator.wikimedia.org/T365415) [22:22:25] (03CR) 10Ejegg: [C:03+2] Move checkLanguageExists to WMFContact::save (only caller) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056642 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:40:49] 10fundraising-tech-ops, 06cloud-services-team, 10VPS-Projects, 10Puppet (Puppet 7.0): Update puppet civicrm-prototype puppetmaster - https://phabricator.wikimedia.org/T361595#10016625 (10Dwisehaupt) Migrated to the production VPS today. Hopefully we will be able to start the rebuild of the cloudVPS host ne... [22:44:19] (03Merged) 10jenkins-bot: Extract set-up functionality, preparatory to moving to RefundQueueConsumerTest class [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056627 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:45:56] 06Fundraising-Backlog, 10fundraising-tech-ops: Make the full list of languages available to custom fields on Community CiviCRM instance - https://phabricator.wikimedia.org/T343488#10016630 (10Dwisehaupt) Is this still needed? I have a recollection that it has been solved but no documentation stating so. [22:46:04] 06Fundraising-Backlog, 10fundraising-tech-ops: Make the full list of languages available to custom fields on Community CiviCRM instance - https://phabricator.wikimedia.org/T343488#10016632 (10Dwisehaupt) [22:46:09] (03Merged) 10jenkins-bot: Move RefundTest tests from legacy class to RefundQueueTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056637 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:46:11] (03Merged) 10jenkins-bot: Fully remove RefundTest [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056638 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:46:13] (03Merged) 10jenkins-bot: Move markRefund to the refund queue consumer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056639 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:46:13] (03Merged) 10jenkins-bot: Move permission check to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056640 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:47:22] (03Merged) 10jenkins-bot: Fold function back into only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056641 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [22:47:23] (03Merged) 10jenkins-bot: Move checkLanguageExists to WMFContact::save (only caller) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056642 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [23:02:07] 10fundraising-tech-ops, 06SRE, 13Patch-For-Review: Q1:rack/setup/install frqueue2003, pay-lb2001, pay-lb2002 - https://phabricator.wikimedia.org/T369566#10016660 (10Papaul) [23:02:45] 10fundraising-tech-ops, 06SRE, 13Patch-For-Review: Q1:rack/setup/install frqueue2003, pay-lb2001, pay-lb2002 - https://phabricator.wikimedia.org/T369566#10016663 (10Papaul) a:05Papaul→03Dwisehaupt @Dwisehaupt all your's [23:04:55] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw, 06SRE, 13Patch-For-Review: Q1:rack/setup/install fransw200[1-3].frack.codfw.wmnet - https://phabricator.wikimedia.org/T367800#10016668 (10Papaul) [23:05:54] 10fundraising-tech-ops, 06SRE, 13Patch-For-Review: Q1:rack/setup/install fransw200[1-3].frack.codfw.wmnet - https://phabricator.wikimedia.org/T367800#10016675 (10Papaul) [23:07:16] 10fundraising-tech-ops, 06SRE, 13Patch-For-Review: Q1:rack/setup/install fransw200[1-3].frack.codfw.wmnet - https://phabricator.wikimedia.org/T367800#10016672 (10Papaul) a:05Papaul→03Dwisehaupt @Dwisehaupt all your's [23:10:34] (03PS3) 10Eileen: Remove some calls to deprecated function from with test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056643 (https://phabricator.wikimedia.org/T365415) [23:10:34] (03PS3) 10Eileen: Move function to only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056644 (https://phabricator.wikimedia.org/T365415) [23:10:34] (03PS3) 10Eileen: Move first attempt at queue function to Omnimail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056645 (https://phabricator.wikimedia.org/T365415) [23:10:35] (03PS4) 10Eileen: Remove call to legacy function from test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056646 (https://phabricator.wikimedia.org/T365415) [23:10:36] (03PS3) 10Eileen: Remove functions with no callers [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056651 [23:10:39] (03PS4) 10Eileen: Upgrade another test away from legacy functions [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056650 (https://phabricator.wikimedia.org/T365415) [23:13:55] (03PS9) 10Eileen: Add WMFAudit.parse function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056053 (https://phabricator.wikimedia.org/T270675) [23:13:55] (03PS4) 10Eileen: Minor clean up on the parseReconciliationFile [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056270 [23:13:55] (03PS9) 10Eileen: Replace a chunk of single character outputs with summary text [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056271 (https://phabricator.wikimedia.org/T270675) [23:13:56] (03PS7) 10Eileen: Remove grumble [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056274 [23:13:57] (03PS3) 10Eileen: Remove single char output in making section [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056444 [23:13:58] (03PS7) 10Eileen: Replace count construct with for-each [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056275 [23:20:36] (03PS5) 10Eileen: Call WMFAudit::parse from within drush [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056054 (https://phabricator.wikimedia.org/T270675) [23:20:36] (03PS5) 10Eileen: Stop enabling wmf_audit sub-modules [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056055 (https://phabricator.wikimedia.org/T270675) [23:20:36] (03PS4) 10Eileen: Switch to using \Civi::log() for logging [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056044 (https://phabricator.wikimedia.org/T270675) [23:20:37] (03PS4) 10Eileen: Remove now-unused audit_runtime [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056046 (https://phabricator.wikimedia.org/T270675) [23:21:51] cstone: this one is the one we did on the call yesterday - it isn't in either of the chains (just rebased them) but gets rid of the function output https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1056596/1/drupal/sites/default/civicrm/extensions/wmf-civicrm/Civi/WMFAudit/BaseAuditProcessor.php [23:22:26] the chains are looking saner thanks to ejegg's review :-) [23:22:32] (03CR) 10Cstone: [C:03+2] "thanks!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056596 (owner: 10Eileen) [23:22:35] nice eileen [23:22:47] yeah i didn't want to join in and cause review chaos hah [23:23:02] cstone: this is also not in a chain - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1056588/1/drupal/sites/default/civicrm/extensions/wmf-civicrm/CRM/Wmf/Upgrader.php & should be a no brainer [23:23:29] (03CR) 10Cstone: [C:03+2] Fix exceptions in a few places [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056588 (owner: 10Eileen) [23:23:32] hah yep [23:24:19] also - I'm thinking about rebasing this one out of the chain? - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1056271 -it's the output we talked about & it might be nice to see what it looks like on prod so we can make further output decisions [23:24:45] (03CR) 10Eileen: Replace a chunk of single character outputs with summary text (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056271 (https://phabricator.wikimedia.org/T270675) (owner: 10Eileen) [23:40:13] omg i think there is actual mouse in here [23:40:16] i can hear chewing [23:41:45] ugh. i hate that sounds. [23:42:11] (03Merged) 10jenkins-bot: Remove extraneous command output [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056596 (owner: 10Eileen) [23:42:15] (03Merged) 10jenkins-bot: Fix exceptions in a few places [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056588 (owner: 10Eileen) [23:54:08] cstone: run it through our unit tests - it won't survive