[00:06:34] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for jtanner - https://phabricator.wikimedia.org/T415729 (10Dwisehaupt) 03NEW [00:06:36] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for sredkina - https://phabricator.wikimedia.org/T415730 (10Dwisehaupt) 03NEW [00:06:39] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for scampos - https://phabricator.wikimedia.org/T415731 (10Dwisehaupt) 03NEW [00:06:43] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for anil - https://phabricator.wikimedia.org/T415732 (10Dwisehaupt) 03NEW [00:06:47] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for seddon - https://phabricator.wikimedia.org/T415733 (10Dwisehaupt) 03NEW [00:06:49] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for dbautista - https://phabricator.wikimedia.org/T415734 (10Dwisehaupt) 03NEW [00:06:51] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for jcuriel - https://phabricator.wikimedia.org/T415735 (10Dwisehaupt) 03NEW [00:06:54] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for abeckford - https://phabricator.wikimedia.org/T415736 (10Dwisehaupt) 03NEW [00:06:58] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for sguseman - https://phabricator.wikimedia.org/T415737 (10Dwisehaupt) 03NEW [00:33:35] (03PS2) 10Eileen: Fix sort_key to cope with new filename syntax [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 [00:35:47] (03CR) 10CI reject: [V:04-1] Fix sort_key to cope with new filename syntax [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [00:36:25] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [00:38:07] 10fundraising-tech-ops: Add date of commit to fundraising_code_update -i output - https://phabricator.wikimedia.org/T415739 (10Dwisehaupt) 03NEW [00:49:17] (03PS9) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [00:50:05] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [00:50:27] (03CR) 10CI reject: [V:04-1] Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [00:51:34] ejegg: error ES2018 'Promise.prototype.finally' method is forbidden I will add to then for success but not using finally then [00:53:53] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 0725 Dec Q3 Planning: Create Recurring Upgrade and Recurring Downgrade activities when completed by DR through CiviCRM UI - https://phabricator.wikimedia.org/T409097#11560808 (10Lars) a:03Lars [00:54:02] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [00:58:35] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Allow creation of addt soft credits and other entities on import - https://phabricator.wikimedia.org/T415163#11560827 (10Lars) [00:58:36] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing, and 2 others: Chariot import mapping template - 2 soft credits - https://phabricator.wikimedia.org/T411555#11560826 (10Lars) [00:59:48] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing: Chariot import fails when there is no exact match found for organization name - https://phabricator.wikimedia.org/T415295#11560829 (10Lars) [00:59:50] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing, and 2 others: Chariot import mapping template - 2 soft credits - https://phabricator.wikimedia.org/T411555#11560828 (10Lars) [01:00:24] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing: Chariot import mapping template - 2 soft credits - https://phabricator.wikimedia.org/T411555#11560833 (10Lars) [01:05:43] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [01:42:12] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [01:46:49] (03PS10) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [01:48:01] (03CR) 10Wfan: [C:03+2] Fix sort_key to cope with new filename syntax [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [01:48:09] (03CR) 10CI reject: [V:04-1] Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [01:48:41] (03CR) 10Wfan: "recheck" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [01:51:55] (03PS11) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [01:53:38] (03CR) 10CI reject: [V:04-1] Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [02:06:24] thanks wfan [02:07:01] (03Merged) 10jenkins-bot: Fix sort_key to cope with new filename syntax [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1232854 (owner: 10Eileen) [02:09:19] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1233951 [02:09:26] (03PS12) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [02:09:31] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1233951 (owner: 10Eileen) [02:10:34] (03Merged) 10jenkins-bot: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1233951 (owner: 10Eileen) [02:10:56] (03CR) 10CI reject: [V:04-1] Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [02:13:26] !log civicrm upgraded from 7299615a to 64267a34 [02:13:27] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [02:17:07] (03PS13) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [02:18:24] (03CR) 10CI reject: [V:04-1] Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) (owner: 10Wfan) [02:19:37] (03PS2) 10Wfan: Update wording to indicate the external_identifier used for venmo and paypal payer id [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1233252 (https://phabricator.wikimedia.org/T400765) [02:46:35] (03PS1) 10Eileen: Civix upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233965 [03:07:22] (03CR) 10Cstone: [C:03+2] Load gravy paypal from pending/transactionLog even when paypal id differs [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233880 (https://phabricator.wikimedia.org/T409362) (owner: 10Eileen) [03:10:11] (03PS1) 10Eileen: Paypal grant wip [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233971 [03:20:40] thanks cstone [03:27:47] (03CR) 10CI reject: [V:04-1] Paypal grant wip [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233971 (owner: 10Eileen) [03:30:00] (03Merged) 10jenkins-bot: Load gravy paypal from pending/transactionLog even when paypal id differs [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233880 (https://phabricator.wikimedia.org/T409362) (owner: 10Eileen) [03:32:28] (03PS2) 10Eileen: Paypal grant wip [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233971 [03:32:52] (03PS3) 10Eileen: Paypal grant wip [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233971 [03:37:52] (03PS3) 10Eileen: Bubble up grant_provider from paypal trr files [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1233837 (https://phabricator.wikimedia.org/T414999) [03:40:45] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1233974 [03:41:20] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1233974 (owner: 10Eileen) [03:42:36] !log civicrm upgraded from 64267a34 to 32f9a10d [03:42:37] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [03:49:57] (03CR) 10CI reject: [V:04-1] Paypal grant wip [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233971 (owner: 10Eileen) [04:35:46] !log config revision changed from ed0808a9 to ef6ef5f2 [04:35:46] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [04:40:46] 14Fundraising Sprint: ARMchair, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 07Epic, 07fr-current-sprint: Show which contacts have double opted in / still need double opt in in Acoustic - https://phabricator.wikimedia.org/T413812#11560923 (10Lars) @NNgu-WMF Now done, we have the double out... [05:04:00] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing: Chariot import fails when there is no exact match found for organization name - https://phabricator.wikimedia.org/T415295#11560947 (10Lars) @MDemosWMF We'll take that check out then, which wil... [05:04:34] 06Fundraising-Backlog, 06FR-donorrelations, 07fr-current-sprint, 07payments-orchestration: Handle Gravy giving us wrong or inconsistent paypal trxn ID - https://phabricator.wikimedia.org/T415744 (10Eileenmcnaughton) 03NEW [05:05:40] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Add link in Zendesk to contact in CiviCRM - https://phabricator.wikimedia.org/T415378#11560967 (10Lars) p:05Triage→03Medium [05:05:58] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 06FR-donorrelations: Add one-click ‘copy email address’ function - https://phabricator.wikimedia.org/T415374#11560969 (10Lars) p:05Triage→03Medium [05:36:43] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog: Queue messages are logged twice - https://phabricator.wikimedia.org/T384561#11560985 (10Eileenmcnaughton) I have an explanation for this ... We have 2 standard out loggers - in theory the `wmf_cli` one logs for `wmf` channel (only) for info and abov... [05:57:35] (03Abandoned) 10Cstone: Move the recurring failure sending and activity creation to a helper [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1230706 (owner: 10Cstone) [05:59:29] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: BaseAuditProcessor improvements - https://phabricator.wikimedia.org/T415746 (10Eileenmcnaughton) 03NEW [05:59:50] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-WMF-Audit: BaseAuditProcessor improvements - https://phabricator.wikimedia.org/T415746#11561022 (10Eileenmcnaughton) [06:01:17] (03PS1) 10Eileen: Get to the good stuff quicker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234126 (https://phabricator.wikimedia.org/T415746) [06:01:46] (03PS2) 10Eileen: Get to the good stuff quicker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234126 (https://phabricator.wikimedia.org/T415746) [06:09:48] (03PS1) 10Cstone: WIP: Add ability to send 2nd recurring failure from the UI [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234131 (https://phabricator.wikimedia.org/T365488) [06:10:24] (03PS2) 10Cstone: WIP: Add ability to send 2nd recurring failure from the UI [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234131 (https://phabricator.wikimedia.org/T365488) [06:18:44] (03CR) 10CI reject: [V:04-1] Get to the good stuff quicker [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234126 (https://phabricator.wikimedia.org/T415746) (owner: 10Eileen) [06:24:03] (03PS1) 10Eileen: Take what we can get [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234142 (https://phabricator.wikimedia.org/T415744) [06:26:42] (03CR) 10Cstone: [C:03+2] Take what we can get [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234142 (https://phabricator.wikimedia.org/T415744) (owner: 10Eileen) [06:47:34] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234144 [06:47:53] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234144 (owner: 10Eileen) [06:50:20] (03Merged) 10jenkins-bot: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234144 (owner: 10Eileen) [06:51:57] !log * civicrm upgraded from 32f9a10d to 600b21a6 [06:51:57] * config revision is d3eb916c [06:51:58] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [07:06:14] (03PS1) 10Eileen: Fix transactionDetail to store [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234153 (https://phabricator.wikimedia.org/T415744) [07:06:45] (03CR) 10Eileen: [C:03+2] "sneaky self-+2 cos it's the same in spirit - just not as wrong" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234153 (https://phabricator.wikimedia.org/T415744) (owner: 10Eileen) [07:27:12] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234162 [07:27:38] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234162 (owner: 10Eileen) [07:30:05] !log civicrm upgraded from 600b21a6 to 56c222da [07:30:05] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [07:41:23] 06Fundraising-Backlog, 06FR-donorrelations, 07fr-current-sprint, 07payments-orchestration: Handle Gravy giving us wrong or inconsistent paypal trxn ID - https://phabricator.wikimedia.org/T415744#11561165 (10Eileenmcnaughton) OK - that worked - it has a transaction now!! Imported from the PayPal reconciliat... [07:41:47] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 06FR-donorrelations, 07fr-current-sprint, 07payments-orchestration: Handle Gravy giving us wrong or inconsistent paypal trxn ID - https://phabricator.wikimedia.org/T415744#11561167 (10Eileenmcnaughton) [07:44:57] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 06FR-donorrelations, 07fr-current-sprint, 07payments-orchestration: Handle Gravy giving us wrong or inconsistent paypal trxn ID - https://phabricator.wikimedia.org/T415744#11561184 (10Eileenmcnaughton) In terms of volume - looks like there are 3 of t... [11:31:14] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 06FR-donorrelations, 07fr-current-sprint, 07payments-orchestration: Handle Gravy giving us wrong or inconsistent paypal trxn ID - https://phabricator.wikimedia.org/T415744#11561623 (10Eileenmcnaughton) After deploying the fix for this I re-processed... [12:11:43] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1234343 [12:15:48] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1234343 (owner: 10Jgleeson) [12:16:23] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1234343 (owner: 10Jgleeson) [12:22:02] (03PS1) 10Jgleeson: Update SmashPig to v1.2.0 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 [12:23:40] (03CR) 10CI reject: [V:04-1] Update SmashPig to v1.2.0 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 (owner: 10Jgleeson) [12:52:52] (03PS1) 10Jgleeson: Update PHPUnit to 9.6.34 to fix security advisory [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234362 [12:56:48] (03CR) 10Jgleeson: [C:03+2] Update PHPUnit to 9.6.34 to fix security advisory [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234362 (owner: 10Jgleeson) [13:09:50] (03Merged) 10jenkins-bot: Update PHPUnit to 9.6.34 to fix security advisory [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234362 (owner: 10Jgleeson) [13:12:32] (03CR) 10Jgleeson: "resubmit" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 (owner: 10Jgleeson) [13:14:06] that moment of "is it resubmit or recheck?!" [13:14:25] (03CR) 10Jgleeson: "recheck" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 (owner: 10Jgleeson) [13:15:03] recheck it seems [13:17:55] (03CR) 10Jgleeson: [C:03+2] Update SmashPig to v1.2.0 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 (owner: 10Jgleeson) [13:28:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Snooze Activity in Civi creating numerous activities on a record - https://phabricator.wikimedia.org/T415642#11562138 (10RLewis) @Lars - can we just ping you the cids if we see this repeat snooze activity on any records... [13:47:17] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Donor-portal, 0725 Dec Q3 Planning: Logout for Donor Portal - https://phabricator.wikimedia.org/T407454#11562251 (10jgleeson) a:03jgleeson [13:49:59] damilare|brb: I've been brave and assigned myself to https://phabricator.wikimedia.org/T407454 and put you down as a co-assignee. Is that a good ticket to help get onboarded with how it all works? If so, could we grab an hour later today or even tomorrow, as we're stacked with calls this evening, and you could give me an overview of the donor portal architecture and best first tasks? Thanks in advance! [13:53:17] (03Merged) 10jenkins-bot: Update SmashPig to v1.2.0 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234345 (owner: 10Jgleeson) [13:54:14] jgleeson: sure thing, I think that's definitely a good one to get started with especially with the invalidate checksum api action larssandergreen set up in the last sprint [13:56:10] tomorrow sounds good too for the meeting, I'd block a time on your calendar [13:56:32] thanks damilare ! [13:57:35] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1234401 [14:00:02] done jgleeson, please feel free to move it to a more convenient time for you also [14:00:44] that slot looks good for me damilare thanks [14:01:10] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1234401 (owner: 10Jgleeson) [14:01:12] cool, thanks! [14:06:22] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10FR-Donor-portal: When civiproxy is down display a better donor preference page - https://phabricator.wikimedia.org/T415393#11562351 (10Damilare) a:03Damilare [14:07:56] TIL https://en.wikipedia.org/wiki/Wow!_signal [14:35:42] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1234401 (owner: 10Jgleeson) [14:36:33] (03PS1) 10Jgleeson: Update composer.lock file for SmashPig update [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234426 [14:36:43] (03CR) 10Jgleeson: [C:03+2] Update composer.lock file for SmashPig update [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234426 (owner: 10Jgleeson) [14:41:02] !log payments-wiki upgraded from 24915bdb to 81d9f614 [14:41:02] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [14:41:49] ok! we have structured API timings log data [14:41:51] Jan 28 14:40:41 payments1005 SmashPig-Gravy: gravy::244699877:244699877.1 | (APITimings) [gravy|cc|capture|request|time] 0.828603s | | [14:42:01] Jan 28 14:40:48 payments1006 SmashPig-Gravy: gravy::244699891:244699891.1 | (APITimings) [gravy|cc|createsession|request|time] 0.171039s | | [14:42:05] awesome [14:42:13] I'll prep the deploy to civi [14:44:17] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 07fr-current-sprint: Add in tracking (and graphs) for how long api calls to the payment processors take - https://phabricator.wikimedia.org/T411673#11562571 (10jgleeson) We deployed this to the Payments cluster a minute ago, and I can now see structured... [14:47:39] (03Merged) 10jenkins-bot: Update composer.lock file for SmashPig update [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1234426 (owner: 10Jgleeson) [14:54:39] (03PS1) 10Jgleeson: Upgrade SmashPig to v1.2.0 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234433 [14:56:18] (03CR) 10Jgleeson: [C:03+2] Upgrade SmashPig to v1.2.0 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234433 (owner: 10Jgleeson) [14:57:21] (03PS1) 10Jgleeson: Update SmashPig to v1.2.0 [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1234435 [14:57:43] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234436 [14:57:58] (03CR) 10Jgleeson: [C:03+2] Update SmashPig to v1.2.0 [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1234435 (owner: 10Jgleeson) [15:13:56] (03Merged) 10jenkins-bot: Upgrade SmashPig to v1.2.0 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234433 (owner: 10Jgleeson) [15:14:55] (03Merged) 10jenkins-bot: Update SmashPig to v1.2.0 [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1234435 (owner: 10Jgleeson) [15:36:47] (03PS1) 10Ejegg: Upgrade composer packages [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1234443 [15:52:32] (03CR) 10Ejegg: "Just noticing that the "Transaction Event Code" seems to be T0001 for all of them and for precious little else." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1233837 (https://phabricator.wikimedia.org/T414999) (owner: 10Eileen) [15:52:53] (03CR) 10Ejegg: [C:03+2] Upgrade composer packages [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1234443 (owner: 10Ejegg) [15:54:00] (03Merged) 10jenkins-bot: Upgrade composer packages [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1234443 (owner: 10Ejegg) [15:56:14] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Snooze Activity in Civi creating numerous activities on a record - https://phabricator.wikimedia.org/T415642#11562848 (10RLewis) @Lars - here's another one I came across [15:56:36] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for rlewis - https://phabricator.wikimedia.org/T415612#11562850 (10Dwisehaupt) certificate renewed and sent via email. password sent via sms. [15:58:55] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for pcoombe - https://phabricator.wikimedia.org/T415614#11562871 (10Dwisehaupt) certificate renewed and sent via email. password sent via sms. [16:00:31] (03PS1) 10Ejegg: Update phpunit and dependencies [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234445 [16:04:30] (03CR) 10Ejegg: [C:03+2] Update phpunit and dependencies [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234445 (owner: 10Ejegg) [16:04:50] 10fundraising-tech-ops: Metabase access for Sarah Holko - https://phabricator.wikimedia.org/T415385#11562890 (10AStein-WMF) [16:05:52] 10fundraising-tech-ops: Metabase access for Sarah Holko - https://phabricator.wikimedia.org/T415385#11562892 (10AStein-WMF) waiting on her phone number so i can send the ssl cert password [16:12:00] 06Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for sguseman - https://phabricator.wikimedia.org/T415737#11562926 (10Dwisehaupt) certificate renewed and sent via email. password sent via sms. [16:14:36] (03PS1) 10Damilare Adedoyin: WIP: Gracefully handle civiproxy errors on DonorPortal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234446 (https://phabricator.wikimedia.org/T415393) [16:17:33] (03CR) 10CI reject: [V:04-1] WIP: Gracefully handle civiproxy errors on DonorPortal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1234446 (https://phabricator.wikimedia.org/T415393) (owner: 10Damilare Adedoyin) [16:22:16] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Handle snoozed emails in dedupe - https://phabricator.wikimedia.org/T415797 (10Lars) 03NEW [16:22:33] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Handle snoozed emails in dedupe - https://phabricator.wikimedia.org/T415797#11562949 (10Lars) p:05Triage→03Medium [16:26:37] (03Merged) 10jenkins-bot: Update phpunit and dependencies [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234445 (owner: 10Ejegg) [16:30:24] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Handle snoozed emails in dedupe - https://phabricator.wikimedia.org/T415797#11562980 (10Lars) [16:48:42] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Snooze Activity in Civi creating numerous activities on a record - https://phabricator.wikimedia.org/T415642#11563048 (10Lars) @RLewis I don't think cleaning this up manually would make sense. We have ~10k contacts affec... [17:00:10] (03PS14) 10Wfan: Apple session already exist issue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1222510 (https://phabricator.wikimedia.org/T407584) [17:03:38] (03PS3) 10Wfan: Update wording to indicate the external_identifier used for venmo and paypal payer id [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1233252 (https://phabricator.wikimedia.org/T400765) [17:04:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Trouble donating via Venmo does not move to TY page automatically + error message displayed still charged - https://phabricator.wikimedia.org/T413671#11563123 (10AnnWF) a:03AnnWF [17:06:38] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234436 (owner: 10Jgleeson) [17:07:30] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1234436 (owner: 10Jgleeson) [17:19:22] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Improve automatic deduping - https://phabricator.wikimedia.org/T415806 (10Lars) 03NEW [17:19:46] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Improve automatic deduping - https://phabricator.wikimedia.org/T415806#11563262 (10Lars) p:05Triage→03Medium [17:21:21] (03CR) 10Wfan: "check coverage" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1233252 (https://phabricator.wikimedia.org/T400765) (owner: 10Wfan) [17:23:32] 10fundraising-tech-ops: Metabase access for Sarah Holko - https://phabricator.wikimedia.org/T415385#11563284 (10AStein-WMF) 05Open→03Resolved [17:28:04] (03CR) 10Lars SG: [C:03+2] Civix upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233965 (owner: 10Eileen) [17:46:57] (03Merged) 10jenkins-bot: Civix upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1233965 (owner: 10Eileen) [17:53:55] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Snooze Activity in Civi creating numerous activities on a record - https://phabricator.wikimedia.org/T415642#11563416 (10RLewis) @Lars sorry here is the cid 49844569 and yes that makes sense since there as so many. In r... [17:55:52] (03CR) 10Lars SG: [C:03+2] Avoid validating just-generated checksum [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1230947 (https://phabricator.wikimedia.org/T402837) (owner: 10Ejegg) [17:58:21] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Snooze Activity in Civi creating numerous activities on a record - https://phabricator.wikimedia.org/T415642#11563439 (10Lars) a:03Lars @RLewis I'll add it to the list. [17:58:30] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Remove extra snooze activities - https://phabricator.wikimedia.org/T415642#11563441 (10Lars) [18:16:57] (03Merged) 10jenkins-bot: Avoid validating just-generated checksum [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1230947 (https://phabricator.wikimedia.org/T402837) (owner: 10Ejegg) [18:17:08] sorry fr-tech internet connection slowness [18:40:00] (03PS1) 10Ejegg: Datacheck for duplicate relationships [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1234476 (https://phabricator.wikimedia.org/T413042) [18:40:57] 03Fundraising Sprint: Ball pit, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donation-Processing, 0725 Dec Q3 Planning: Share address and email between DAF and DAF owner when importing donations - https://phabricator.wikimedia.org/T411639#11563616 (10Lars) @MDemosWMF or @RLewis In the cas... [18:51:26] !logdog civicrm upgraded from 56c222da to 5c065c4e [18:51:38] ok that's civicrm updated with the latest log timing stuff [18:51:56] I'll quickly deploy smashpig standlone and then it's out [18:53:38] !log SmashPig upgraded from 96a6224d to b42f2de4 [18:53:39] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [18:53:43] oops, I keep doing !logdog [18:53:51] !log civicrm upgraded from 56c222da to 5c065c4e [18:53:52] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [18:56:35] dwisehaupt: Jeff_Green just to confirm, with the structured log strings, you want me to do the following: [18:56:48] For gravy [gravy|adyen|cc|authorize|request|time] 2.572354s [18:57:02] sure, that works [18:57:03] for direct integrations [|adyen|cc|authorize|request|time] 2.572354s [18:57:12] (omitting the first part) [18:57:20] or did we agree on something else [18:57:46] the [| being the part where we leave that field empty, ya? [18:57:54] yeah [18:57:58] sure this works [18:58:06] ok cool I'll work that in [18:58:07] i thought it was: processor|backend|type|method|request|time [18:58:08] thanks [18:58:20] was it dwisehaupt ? [18:58:24] so what would that look like [18:58:28] i could be wrong. [18:58:44] let's back up a sec [18:58:45] ok so this is that format [gravy|adyen|cc|authorize|request|time] 2.572354s [18:58:52] the gravy would be the same. the adyen would be adyen||cc|authorize|request|time [18:59:06] the tricky part is, for direct integrations we don't have the first key [18:59:07] what are the label names you want for each field? [18:59:25] jgleeson: oh. interesting. [18:59:35] so the complete format is [18:59:46] as long as there's consistency between the number of delimited slots and label names it'll run good [18:59:49] orchestrator|processor|type|method|request|time [19:00:02] type = payment_type [19:00:05] ok. ignore me. it's good. [19:00:22] that'll work as long as it's consistent. [19:00:35] ok great [19:00:41] one thing just to call out [19:00:49] you are going to see one that looks confusing [19:01:03] [gravy|paypal|paypal|authorize|request|time] [19:01:07] that's legit [19:01:13] as paypal is the backend and the method [19:01:19] looks fine to me. :) [19:01:30] cool cool [19:01:43] eventually that might turn into [gravy|braintree|paypal|authorize|request|time] [19:02:12] I can also foresee [gravy|paypal|paypal|authorize|request|time] 23 hours [19:02:14] :) [19:02:26] need to add paypal as a orchestrator then we can have paypal|paypal|paypal akin to https://en.wikipedia.org/wiki/Badgers_(animation) [19:02:41] turtles|turtles|turtles [19:02:50] dwisehaupt: oh no [19:03:02] badger|badger|badger|mushrooooooooommm [19:03:04] ok. gotta get this dog out for his walk after dropping that earworm. :) [19:03:14] hf [19:04:05] so that translates to: whatever_we_name_this_metric{orchestrator=gravy, processor=paypal, type=paypal, method=request, time=time} 0.005 [19:04:26] lemme see [19:04:46] err the last field (time) confuses me, what is that? [19:06:44] so it would map like this I think [19:06:46] [gravy|adyen|cc|authorize|request|time] 2.654072 [19:06:48] would get you [19:06:50] jgleeson: were yu planning on coming to fraud club [19:06:56] whatever_we_name_this_metric{orchestrator=gravy, processor=adyen, payment_method=cc, api_operation=authorize, metric=request, unit=time} 2.654072 [19:06:58] coming cstone ! [19:07:09] Jeff_Green: I just made up metric and unit [19:09:08] hmm. lemme cook up the example collector config so we can discuss [19:09:14] what shall we call this metric? [19:09:26] payment_timing ? [19:09:35] processor_timing? [19:09:44] processing_time... [19:09:47] badgertime [19:14:58] the regex would look something like this: [19:15:24] pattern: '\[(?[^\|]*)\|(?[^\|]*)\|(?[^\|]*)\|(?[^\|]*)\|(?[^\|]*)\|(?