[00:12:31] eileen: i think its fine to leave it too if you don't want to rebase were mainly looking at those files now if something breaks them [00:13:21] (03CR) 10Wfan: [C:03+2] "Works well on my local, remember if mac, use ./setup sync to sync local CiviCRM files to containers before flush the system to see the new" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) (owner: 10Jgleeson) [00:13:35] cstone: well I guess I'm fine to rebase that one out if it gets it merged quicker - just so we can see what it does to the output & what we want to do [00:31:05] sun is intense [00:31:25] (03Merged) 10jenkins-bot: Add Zendesk tab to contact summary screen [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1050045 (https://phabricator.wikimedia.org/T368689) (owner: 10Jgleeson) [00:39:39] (03PS2) 10Cstone: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [00:41:25] (03CR) 10CI reject: [V:04-1] Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [00:43:33] (03PS3) 10Cstone: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [00:45:17] (03CR) 10CI reject: [V:04-1] Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [00:46:11] (03PS4) 10Cstone: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [00:47:49] (03CR) 10CI reject: [V:04-1] Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [00:57:57] (03PS5) 10Cstone: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [00:59:45] (03CR) 10CI reject: [V:04-1] Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [01:01:08] (03PS6) 10Cstone: Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 [01:01:14] linter please [01:02:56] (03CR) 10CI reject: [V:04-1] Add special page to show our configurable payment settings [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1057014 (owner: 10Cstone) [01:09:11] (03CR) 10Cstone: [C:03+2] Remove some calls to deprecated function from with test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056643 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [01:27:36] (03Merged) 10jenkins-bot: Remove some calls to deprecated function from with test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056643 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [05:09:00] 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#10017002 (10RKumar_WMF) Thank you @Cstone [05:17:17] 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#10017004 (10RKumar_WMF) @Cstone - Are we not ad... [13:59:44] (03PS9) 10Damilare Adedoyin: WIP: Setup listener for transaction event message. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1056531 (https://phabricator.wikimedia.org/T370797) [14:00:10] (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) [15:07:40] CentralNotice regression reported in https://app.slack.com/client/E012JBDTTHA/C05C80HBTHB with the deploy this week [15:07:47] I'm going to take a quick look locally [15:07:58] see if I can find the cause with git bisect [15:08:40] (I really should have noticed that when I was reviewing the merge!) [15:11:40] til git pull --unshallow https://stackoverflow.com/questions/29270058/how-to-fetch-all-git-history-after-i-clone-the-repo-with-depth-1 [15:23:48] We're far from the shallow now [16:23:09] (03PS1) 10Jgleeson: Remove Zendesk tab clutter from wmf_civicrm_civicrm_contactSummaryBlocks() [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1057238 (https://phabricator.wikimedia.org/T368689) [17:58:48] 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 13Patch-For-Review: CentralNotice should catch banner errors and log them to its own channel - https://phabricator.wikimedia.org/T361680#10019020 (10Jdlrobson) @pcoombe what do you think of the above proposal? Is it viable? [18:59:30] 10fundraising-tech-ops, 06SRE: Q1:rack/setup/install frqueue2003, pay-lb2001, pay-lb2002 - https://phabricator.wikimedia.org/T369566#10019218 (10Dwisehaupt) Awaiting deployment of pfw changes to add the new mgmt subnet to pfw config (T371137). Then we can build the hosts. [19:28:24] (03CR) 10Cstone: [C:03+2] Remove Zendesk tab clutter from wmf_civicrm_civicrm_contactSummaryBlocks() [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1057238 (https://phabricator.wikimedia.org/T368689) (owner: 10Jgleeson) [19:29:42] (03CR) 10Cstone: [C:03+2] Move function to only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056644 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [19:47:55] (03Merged) 10jenkins-bot: Remove Zendesk tab clutter from wmf_civicrm_civicrm_contactSummaryBlocks() [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1057238 (https://phabricator.wikimedia.org/T368689) (owner: 10Jgleeson) [19:51:15] (03Merged) 10jenkins-bot: Move function to only caller [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1056644 (https://phabricator.wikimedia.org/T365415) (owner: 10Eileen) [20:33:22] (03Abandoned) 10Jdlrobson: Log central notice banner errors to their own channel [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016759 (https://phabricator.wikimedia.org/T361680) (owner: 10Jdlrobson) [21:42:42] (03PS1) 10Ejegg: Fix multiselects [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1057271 [21:43:00] fr-tech anyone around to take a quick look at that? ^^^ [21:43:16] You can test it on the local donutwiki by going to any campaign edit screen [21:43:39] e.g. https://localhost:9010/w/index.php?title=Special:CentralNotice&subaction=noticeDetail¬ice=soso [21:45:38] Sure ejegg [21:48:05] thanks cstone [21:48:17] (03CR) 10Cstone: [C:03+2] Fix multiselects [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1057271 (owner: 10Ejegg) [21:51:41] (03Merged) 10jenkins-bot: Fix multiselects [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1057271 (owner: 10Ejegg) [21:51:50] thanks cstone! [22:44:43] 10fundraising-tech-ops, 06SRE: Q1:rack/setup/install frqueue2003, pay-lb2001, pay-lb2002 - https://phabricator.wikimedia.org/T369566#10019829 (10Dwisehaupt) iptables changes applied for frqueue host. No iptables changes needed for pay-lb hosts. [22:56:30] (03PS1) 10Ejegg: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/1057291 [22:56:34] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/1057291 (owner: 10Ejegg) [23:00:01] (03Merged) 10jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/1057291 (owner: 10Ejegg)