[00:05:53] (03PS1) 10Eileen: Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 [00:06:16] cstone: ^^ I added in booking date / settled_date as you suggested [00:06:22] (03CR) 10CI reject: [V:04-1] Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 (owner: 10Eileen) [00:06:39] - although now i have a test fight :-) [00:17:55] (03PS1) 10Wfan: sync civi config to local [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1180250 [00:28:00] (03PS2) 10Eileen: Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 [00:28:30] (03CR) 10CI reject: [V:04-1] Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 (owner: 10Eileen) [00:29:55] (03PS3) 10Eileen: Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 [00:47:26] (03CR) 10Wfan: [C:03+2] Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 (owner: 10Eileen) [01:06:33] thanks wfan - there was one in the chain before that before it will merge [01:26:31] (03PS4) 10Eileen: Reduce passing around of arrays [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179991 [01:26:34] (03CR) 10Ejegg: [C:03+2] Reduce passing around of arrays [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179991 (owner: 10Eileen) [01:32:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Gravy-PayPal duplicate donation in Civi: what to do - https://phabricator.wikimedia.org/T401866#11100794 (10Cstone) [01:38:23] (03CR) 10Ejegg: "There's a behavior change in this that makes me want to review a lot of messages before merging" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:39:30] (03CR) 10Eileen: Add order_id to fields normalized in the AuditMessage class (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:43:15] (03CR) 10Ejegg: Add order_id to fields normalized in the AuditMessage class (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:44:31] (03CR) 10Eileen: Add order_id to fields normalized in the AuditMessage class (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:47:55] (03Merged) 10jenkins-bot: Reduce passing around of arrays [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179991 (owner: 10Eileen) [01:48:45] (03CR) 10Ejegg: Add order_id to fields normalized in the AuditMessage class (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:50:33] (03CR) 10Ejegg: "Oh hah, gravy is already setting invoice_id as well as order_id, both to the same value. I'll check on the PayPal parser (still in python)" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179992 (owner: 10Eileen) [01:50:57] ejegg: might be worth clearing this & the next one out that might be simpler (earlier in the chain - mostly) https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1179989 [01:54:27] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM: Civi user account deleted when deduping INTO linked contact_id - https://phabricator.wikimedia.org/T400065#11100811 (10Cstone) Happened to me too now [01:57:34] (03CR) 10Cstone: [C:03+2] "long time coming to actually look at the fees!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1178130 (owner: 10Eileen) [01:58:04] (03Merged) 10jenkins-bot: Add settled_currency, batch_number, exchange_rate [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1178130 (owner: 10Eileen) [01:58:06] (03Merged) 10jenkins-bot: Add settled_date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180248 (owner: 10Eileen) [02:23:13] PROBLEM - check_load on frban1002 is CRITICAL: LOAD CRITICAL - total load average: 18.05, 44.27, 31.16 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [02:33:16] (03PS1) 10Ejegg: Ignore Gravy txns in PayPal audit [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180256 (https://phabricator.wikimedia.org/T401866) [02:33:32] ok cstone I think that works ^^^ [02:34:11] (03PS6) 10Eileen: Remove get_parent_order_id as this is in gateway_txn_id now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179295 [02:34:13] (03CR) 10Ejegg: [C:03+2] Remove get_parent_order_id as this is in gateway_txn_id now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179295 (owner: 10Eileen) [02:36:01] (03PS2) 10Eileen: Further logging simplification [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179989 [02:36:04] (03CR) 10Ejegg: [C:03+2] Further logging simplification [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179989 (owner: 10Eileen) [02:36:25] cstone I added another check on the length of the custom field [02:37:06] thanks ejegg ! [02:37:57] just hoping that doesn't drop any legit ones by mistake - looks like we check a few different columns for the order_id [02:38:04] but... I think it's ok [02:41:18] with the three checks you added though it looks safe? [02:41:28] I hope so! [02:41:55] I'm willing to deploy that, anyway [02:42:48] do you want to check to see if any of the donors with a |dup started a recurring, and if that got recorded correctly? [02:43:31] yeah ill look into that next there is some other issue going on with missing donor info though [02:43:39] but i think stopping the dups was the most urgent part [02:46:29] (03CR) 10Cstone: [C:03+2] "thanks for adding this!" [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180256 (https://phabricator.wikimedia.org/T401866) (owner: 10Ejegg) [02:46:39] ty [02:47:43] (03Merged) 10jenkins-bot: Ignore Gravy txns in PayPal audit [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180256 (https://phabricator.wikimedia.org/T401866) (owner: 10Ejegg) [02:48:11] RECOVERY - check_load on frban1002 is OK: LOAD OK - total load average: 1.39, 1.86, 7.72 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [02:50:00] (03PS1) 10Ejegg: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180257 [02:50:02] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180257 (owner: 10Ejegg) [02:50:26] (03Merged) 10jenkins-bot: Merge branch 'master' into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180257 (owner: 10Ejegg) [02:50:52] (03Merged) 10jenkins-bot: Remove get_parent_order_id as this is in gateway_txn_id now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179295 (owner: 10Eileen) [02:51:33] (03Merged) 10jenkins-bot: Further logging simplification [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179989 (owner: 10Eileen) [03:05:38] oh hmm, what's that silverpop change? [03:06:18] what, we still hadn't deployed the recurring upgrade type hardcoding fix?? [03:06:21] I guess not [03:07:20] !log fundraising python tools upgraded from 6ddfb22f to 0d6df34a [03:07:23] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [03:34:27] (03PS1) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [03:34:54] (03CR) 10CI reject: [V:04-1] Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 (owner: 10Eileen) [03:42:12] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Gravy-PayPal duplicate donation in Civi: what to do - https://phabricator.wikimedia.org/T401866#11100878 (10Cstone) For the duplicate issue, looks like they are coming from the paypal audit. It's not all of them though which is inte... [03:51:57] (03PS2) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [03:52:24] (03CR) 10CI reject: [V:04-1] Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 (owner: 10Eileen) [04:03:04] (03PS3) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [04:03:29] (03CR) 10CI reject: [V:04-1] Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 (owner: 10Eileen) [04:14:54] (03PS4) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [04:15:20] (03CR) 10CI reject: [V:04-1] Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 (owner: 10Eileen) [05:59:00] (03PS5) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [05:59:28] (03CR) 10CI reject: [V:04-1] Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 (owner: 10Eileen) [06:04:36] (03PS3) 10Eileen: Extract all the statistic wrangling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1179990 [06:07:48] (03PS6) 10Eileen: Fill out our amount fields [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180259 [06:26:39] (03PS1) 10Eileen: Rename settlement_batch_number to settlement_batch_reference [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180267 [06:49:14] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_44) - 10https://gerrit.wikimedia.org/r/1180312 (owner: 10L10n-bot) [10:52:39] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_44) - 10https://gerrit.wikimedia.org/r/1180312 (owner: 10L10n-bot) [11:27:28] going to kick off the deployment of the gravy paypal fallback [11:29:06] (03PS1) 10Damilare Adedoyin: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180530 [11:32:36] (03CR) 10Damilare Adedoyin: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180530 (owner: 10Damilare Adedoyin) [12:01:35] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180530 (owner: 10Damilare Adedoyin) [12:02:37] (03PS1) 10Damilare Adedoyin: Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 [12:02:43] (03CR) 10Zuul test: "Merge Failed." [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 (owner: 10Damilare Adedoyin) [12:03:00] (03CR) 10CI reject: [V:04-1] Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 (owner: 10Damilare Adedoyin) [12:05:51] (03PS2) 10Damilare Adedoyin: Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 [12:15:20] (03PS3) 10Damilare Adedoyin: Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 [12:17:35] (03PS4) 10Damilare Adedoyin: Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 [12:44:03] (03CR) 10Damilare Adedoyin: [C:03+2] Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 (owner: 10Damilare Adedoyin) [12:56:27] (03Merged) 10jenkins-bot: Update smashpig to v0.9.7.11 [core] (fundraising/REL1_43) - 10https://gerrit.wikimedia.org/r/1180538 (owner: 10Damilare Adedoyin) [12:58:23] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Settled Adyen Donation - Settled Gravy Donation - Not Appearing in Civi - https://phabricator.wikimedia.org/T395340#11102125 (10jgleeson) a:03jgleeson [13:14:13] (03CR) 10Jgleeson: [C:03+2] Remove unused recurring-upgrade queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180171 (owner: 10Ejegg) [13:14:44] (03Merged) 10jenkins-bot: Remove unused recurring-upgrade queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180171 (owner: 10Ejegg) [13:23:09] PROBLEM - check_load on frban1002 is CRITICAL: LOAD CRITICAL - total load average: 78.72, 76.76, 40.47 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [13:25:17] PROBLEM - check_load on frban2002 is CRITICAL: LOAD CRITICAL - total load average: 43.05, 58.56, 37.03 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban2002&service=check_load [13:28:13] PROBLEM - check_load on frban1002 is CRITICAL: LOAD CRITICAL - total load average: 21.29, 52.13, 40.95 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [13:30:17] PROBLEM - check_load on frban2002 is CRITICAL: LOAD CRITICAL - total load average: 6.94, 30.41, 31.19 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban2002&service=check_load [13:33:13] PROBLEM - check_load on frban1002 is CRITICAL: LOAD CRITICAL - total load average: 2.68, 21.81, 30.98 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [13:36:40] (03CR) 10Jgleeson: [C:03+2] "I was able to get a payment through on sandbox with this change https://sandbox.wikimedia.gr4vy.app/merchants/default/transactions/cebcfc4" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180117 (https://phabricator.wikimedia.org/T400413) (owner: 10Damilare Adedoyin) [13:37:23] (03Merged) 10jenkins-bot: Unset buyer field in venmo request [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180117 (https://phabricator.wikimedia.org/T400413) (owner: 10Damilare Adedoyin) [13:37:33] thanks for the cr jgleeson, is your venmo test app still working? [13:38:21] it does load up but it doesnt complete. I used the second flow, something like "fake app" on the gravy redirect [13:39:16] ohh ok, mine doesn't load up. Says app is no longer available [13:39:40] ah [13:40:01] yeah I was confused on how gravy were pulling the venmojoe username via their "fake app" option [13:40:16] but I see this in the braintree response in the API logs [13:40:17] "paymentMethodSnapshot": { [13:40:17] "username": "venmojoe", [13:40:17] "venmoUserId": "1234567891234567891" [13:40:17] } [13:40:53] ohh that makes sense [13:55:17] RECOVERY - check_load on frban2002 is OK: LOAD OK - total load average: 2.55, 2.98, 8.43 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban2002&service=check_load [13:58:09] RECOVERY - check_load on frban1002 is OK: LOAD OK - total load average: 2.30, 2.80, 8.56 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frban1002&service=check_load [14:04:51] hi fr-tech [14:05:39] o/ [15:01:50] (03PS1) 10Jgleeson: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180581 (https://phabricator.wikimedia.org/T395340) [15:02:25] (03CR) 10Jgleeson: [C:03+2] "self-merging maintenance script update. tested with live data locally and working as expected." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180581 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:02:54] (03Merged) 10jenkins-bot: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180581 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:03:01] (03PS1) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:05:32] this patch fixes the error I saw in the last deployment ^ [15:05:47] (03CR) 10CI reject: [V:04-1] Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 (owner: 10Damilare Adedoyin) [15:07:05] (03PS1) 10Jgleeson: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180583 (https://phabricator.wikimedia.org/T395340) [15:07:45] (03PS2) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:07:59] (03CR) 10Jgleeson: [C:03+2] "self-merging maintenance script update." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180583 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:08:30] (03Merged) 10jenkins-bot: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180583 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:08:57] damilare: I'm just pushing out an update to the gravy donation importer so close off a chaos task. I'm gonna deploy smashpig standalone (not tagged) and then I can take a peek at your fix [15:09:19] (03CR) 10CI reject: [V:04-1] Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 (owner: 10Damilare Adedoyin) [15:09:25] TY! [15:15:58] (03PS3) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:17:40] (03PS4) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:18:26] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180586 [15:20:23] (03CR) 10Jgleeson: [C:04-1] "some weird config changes in the merge." [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180586 (owner: 10Jgleeson) [15:29:02] (03PS5) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:31:50] (03CR) 10CI reject: [V:04-1] Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 (owner: 10Damilare Adedoyin) [15:32:50] hmm I'm getting some strange indent differences in smashpig/config/main.yaml between master and deployment [15:34:31] so it looks like deployment has 4 spaces [15:34:34] (03PS6) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:34:34] and master has 2 spaces [15:37:17] (03CR) 10CI reject: [V:04-1] Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 (owner: 10Damilare Adedoyin) [15:39:04] (03PS7) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 [15:43:20] (03PS1) 10Jgleeson: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180592 (https://phabricator.wikimedia.org/T395340) [15:43:21] (03PS1) 10Jgleeson: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180593 (https://phabricator.wikimedia.org/T395340) [15:45:06] (03CR) 10Jgleeson: [C:04-1] "hmm I'm decline this. it looks like we have some config indentation from another commit to figure out. I'll cherry pick for now" [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180586 (owner: 10Jgleeson) [15:45:21] (03Abandoned) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180586 (owner: 10Jgleeson) [15:45:35] (03CR) 10Jgleeson: [C:03+2] Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180592 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:45:40] (03CR) 10Jgleeson: [C:03+2] Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180593 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:45:59] (03Merged) 10jenkins-bot: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180592 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:46:04] (03Merged) 10jenkins-bot: Update BuildGravyDonationFromLogs [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180593 (https://phabricator.wikimedia.org/T395340) (owner: 10Jgleeson) [15:49:10] !log SmashPig standalone upgraded from ebb6e309 to ab4b9cd1 [15:49:12] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [16:03:18] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 13Patch-For-Review, 07payments-orchestration: Settled Adyen Donation - Settled Gravy Donation - Not Appearing in Civi - https://phabricator.wikimedia.org/T395340#11102938 (10jgleeson) @SHust, I've reviewed this ticket again toda... [16:05:24] (03PS1) 10Ejegg: WIP pagination for donations list tables [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180596 (https://phabricator.wikimedia.org/T400934) [16:07:54] (03CR) 10CI reject: [V:04-1] WIP pagination for donations list tables [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180596 (https://phabricator.wikimedia.org/T400934) (owner: 10Ejegg) [16:18:09] (03CR) 10Jgleeson: [V:03+2 C:03+2] Add Gravy / SEPA links to Main_Page [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1179761 (https://phabricator.wikimedia.org/T392065) (owner: 10Ejegg) [16:18:41] cleaning up previous review [16:36:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Settled Adyen Donation - Settled Gravy Donation - Not Appearing in Civi - https://phabricator.wikimedia.org/T395340#11103041 (10SHust) @jgleeson, perfect timing, I found one today, CID 27836669. The donat... [16:38:25] 10fundraising-tech-ops, 06Infrastructure-Foundations, 10netops: Move pfw1b-codfw to rack F5 - https://phabricator.wikimedia.org/T401297#11103043 (10cmooney) @papaul just FYI the diagram you did is now not accessible for other users for some reason [16:56:28] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Two recurring donations missing from CiviCRM. - https://phabricator.wikimedia.org/T399471#11103123 (10SHust) I found one today, CID 27836669. The donation was made on Aug 14th, via Google Pay. We can see... [16:57:11] ejegg I was able to get the buttons working by setting the state variable directly and not the value property in the setpage method [16:58:19] I'd add a comment to the patch [17:02:28] (03CR) 10Damilare Adedoyin: "Hi ejegg, I've been trying to find a documentation for this but there are some weirdness when mixing both the Vue Composition and Options " [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180596 (https://phabricator.wikimedia.org/T400934) (owner: 10Ejegg) [17:07:19] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Two recurring donations missing from CiviCRM. - https://phabricator.wikimedia.org/T399471#11103160 (10jgleeson) Thanks @SHust. I just manually pulled that one in here [[ https://civicrm.wikimedia.org/civi... [17:09:11] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Settled Adyen Donation - Settled Gravy Donation - Not Appearing in Civi - https://phabricator.wikimedia.org/T395340#11103163 (10SHust) Sounds good, thanks @jgleeson. [17:12:50] 03Fundraising Sprint: Pirate's Booty, 06Fundraising-Backlog, 07fr-current-sprint, 13Patch-For-Review, 07payments-orchestration: Enable SEPA via Gr4vy for EU countries. - https://phabricator.wikimedia.org/T392065#11103195 (10Ejegg) Current status: we have the connector correctly configured in the Gravy si... [17:15:24] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Two recurring donations missing from CiviCRM. - https://phabricator.wikimedia.org/T399471#11103205 (10SHust) Thanks, @jgleeson. I merged the records and can also see that a ty email was already sent. [17:15:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Two recurring donations missing from CiviCRM. - https://phabricator.wikimedia.org/T399471#11103206 (10jgleeson) Awesome! Thanks [17:23:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07payments-orchestration: Missing contributions in civicrm due to Gravy - Dlocal Transaction status mismatch - https://phabricator.wikimedia.org/T400479#11103243 (10Damilare) [17:24:09] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations, 07Transaction-missing-in-CiviCRM: Gravy to Civi missing contribs, March - June - https://phabricator.wikimedia.org/T398887#11103250 (10Cstone) [17:33:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: dLocal missing tax ID + status mismatch errors - https://phabricator.wikimedia.org/T401428#11103287 (10AnnWF) 05Resolved→03Open [17:34:40] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: dLocal missing tax ID + status mismatch errors - https://phabricator.wikimedia.org/T401428#11103303 (10AnnWF) p:05Triage→03Medium [17:36:43] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: dLocal missing tax ID + status mismatch errors - https://phabricator.wikimedia.org/T401428#11103309 (10AnnWF) since gravy did not update waiver on their end, so failed, and before they fix that we have the waiver reverted for now ht... [17:43:02] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Gravy-PayPal duplicate donation in Civi: what to do - https://phabricator.wikimedia.org/T401866#11103319 (10Cstone) Might be related to {T394298} [17:54:09] (03PS43) 10Jgleeson: Add Gravy payment error threshold monitoring & alerting system. [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1167604 (https://phabricator.wikimedia.org/T387411) [18:25:03] jgleeson: I found a few other differences between SmashPig deployment and master beyond the yaml WS [18:25:16] I'll put up a deployment patch to align them [18:25:24] well, let me first put up a regular merge patch [18:25:30] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180617 [18:26:12] thanks for digging into that ejegg [18:26:15] I was confused earier [18:26:19] earlier* [18:26:53] yeah, i'm not sure how they got there, maybe incomplete merge conflict resolution on earlier merges [18:27:35] ejegg: I think those two BuildGravyDonationFromLogs commits should already be on deployment [18:27:39] I pushed those out earlier [18:29:20] ok, they just show up on the merge list because the SHA changed from the cherry-picked version [18:29:59] yep I was just checking the changes across the parent commits and I see that file isn't included [18:30:18] i could remove it from the comment if you want [18:30:31] (03PS2) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180617 [18:31:19] oh yeah, there are spacing changes in most of the yaml files as compared to master branch [18:31:23] I'll pull them all over [18:31:41] I could have sworn all that got pushed out months ago [18:31:49] when we did the yaml spacing updates [18:32:04] somehow didn't merge to deploy [18:38:33] (03PS1) 10Ejegg: Align deployment with master [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180620 [18:38:48] ok jgleeson, that gets them back in sync ^^^ [18:39:09] hopefully that adyen currency thing hasn't caused any issues [18:39:56] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180617 (owner: 10Ejegg) [18:40:21] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180617 (owner: 10Ejegg) [18:40:38] (03CR) 10Jgleeson: [C:03+2] "LGTM. thanks for tidying this up" [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180620 (owner: 10Ejegg) [18:40:45] thanks jgleeson [18:41:03] (03Merged) 10jenkins-bot: Align deployment with master [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1180620 (owner: 10Ejegg) [18:41:30] let me just check on the DonationInterface deployment branch [18:42:55] ok, DonationInterface is identical except for test deletion [18:43:32] cool [18:44:13] I'll just deploy those SmashPig changes now [18:45:09] !log standalone (IPN listener) SmashPig upgraded from ab4b9cd1 to 77dc08bd [18:45:11] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:50:47] 06Fundraising-Backlog, 06FR-donorrelations: Civi: enable log search by merchant reference # for failed donations - https://phabricator.wikimedia.org/T402425 (10MBeat33) 03NEW [19:03:42] 10fundraising-tech-ops: Fundraising access request for Lars Sander-Green - https://phabricator.wikimedia.org/T402062#11103566 (10Jgreen) [19:08:47] 10fundraising-tech-ops: Fundraising access request for Lars Sander-Green - https://phabricator.wikimedia.org/T402062#11103598 (10Jgreen) [19:19:15] (03PS2) 10Ejegg: Pagination for donation list tables [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180596 (https://phabricator.wikimedia.org/T400934) [19:20:14] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1180628 [19:20:51] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1180628 (owner: 10Ejegg) [19:21:39] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1180628 (owner: 10Ejegg) [19:21:40] (03CR) 10CI reject: [V:04-1] Pagination for donation list tables [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180596 (https://phabricator.wikimedia.org/T400934) (owner: 10Ejegg) [19:31:59] (03CR) 10Jgleeson: [C:03+2] Enable venmo recurring [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180120 (https://phabricator.wikimedia.org/T400413) (owner: 10Damilare Adedoyin) [19:32:42] (03PS1) 10Ejegg: Ensure SmashPig setup before instantiating adapter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180631 [19:34:57] (03CR) 10CI reject: [V:04-1] Ensure SmashPig setup before instantiating adapter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180631 (owner: 10Ejegg) [19:36:09] (03PS2) 10Ejegg: Ensure SmashPig setup before instantiating adapter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180631 [19:36:12] 10fundraising-tech-ops: Fundraising access request for Lars Sander-Green - https://phabricator.wikimedia.org/T402062#11103894 (10Jgreen) [19:39:06] (03PS1) 10Wfan: Dlocal is fine with no payer.document for waiver countries [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180632 (https://phabricator.wikimedia.org/T401428) [19:40:02] 10fundraising-tech-ops: Fundraising access request for Lars Sander-Green - https://phabricator.wikimedia.org/T402062#11103925 (10Jgreen) a:03Jgreen [19:41:24] (03CR) 10Damilare Adedoyin: [C:03+2] Ensure SmashPig setup before instantiating adapter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180631 (owner: 10Ejegg) [19:41:26] 10fundraising-tech-ops: Fundraising access request for Stacy Burnett - https://phabricator.wikimedia.org/T402061#11103934 (10Jgreen) a:03Jgreen [19:42:00] (03Abandoned) 10Damilare Adedoyin: Ensure the SmashPig PaymentProvider is gotten safely in getCurrencies. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180582 (owner: 10Damilare Adedoyin) [19:44:02] (03Merged) 10jenkins-bot: Enable venmo recurring [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180120 (https://phabricator.wikimedia.org/T400413) (owner: 10Damilare Adedoyin) [19:44:03] (03Merged) 10jenkins-bot: Ensure SmashPig setup before instantiating adapter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1180631 (owner: 10Ejegg) [19:45:37] (03CR) 10Wfan: Show pending message for reject by fraud in fail page (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1174470 (https://phabricator.wikimedia.org/T394098) (owner: 10Wfan) [19:46:01] 10fundraising-tech-ops, 06DC-Ops, 10ops-codfw, 06SRE: Q1:rack/setup/install frdata2002 & frmx2002 - https://phabricator.wikimedia.org/T400275#11103963 (10Jgreen) LGTM! [19:46:57] 06Fundraising-Backlog, 10fundraising-tech-ops: Email send delays for yahoo and aol domains - https://phabricator.wikimedia.org/T399670#11103965 (10Jgreen) 05Open→03Resolved a:03Jgreen [19:49:07] 06Fundraising-Backlog, 10fundraising-tech-ops, 10Infrastructure Security, 06Infrastructure-Foundations, and 2 others: Re-opening our DMarcian Trial - https://phabricator.wikimedia.org/T394788#11103984 (10Jgreen) 05Open→03Resolved a:03Jgreen trial is in progress [19:51:47] (03CR) 10Wfan: "Or we can rename it to isFraudRejection? since we do not have the ValidationAction preserved for this new reload page." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1174470 (https://phabricator.wikimedia.org/T394098) (owner: 10Wfan) [19:56:49] (03PS1) 10Wfan: Remove tax id for dlocal CO CL and UY from url [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1180640 (https://phabricator.wikimedia.org/T401428) [21:20:36] (03PS3) 10Ejegg: Align SEPA country list with request [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1179760 (https://phabricator.wikimedia.org/T392065) [21:21:01] (03PS2) 10Ejegg: SEPA for Gravy [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1180153 (https://phabricator.wikimedia.org/T392065) [21:21:07] (03CR) 10CI reject: [V:04-1] Align SEPA country list with request [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1179760 (https://phabricator.wikimedia.org/T392065) (owner: 10Ejegg) [21:22:01] fr-tech those SEPA patches should be ready to review - they work as intended to create the payment and redirect the donor to SEPA, it's just the donor return processing is waiting on a bugfix from Gravy [21:27:29] !log fundraising civicrm upgraded from b6f2ff27 to 279b3993 [21:27:31] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [21:29:57] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180654 [21:30:29] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180654 (owner: 10Ejegg) [21:31:53] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1180654 (owner: 10Ejegg) [22:01:04] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10MediaWiki-extensions-DonationInterface: Stop exporting permanent contact hash, use expiring checksum instead - https://phabricator.wikimedia.org/T402462 (10Ejegg) 03NEW [22:11:08] 06Fundraising-Backlog, 06FR-donorrelations: Snooze status flapping - https://phabricator.wikimedia.org/T402464 (10SHust) 03NEW [22:21:35] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: URGENT: Merge Behavior Changed — Causing RML Emails to Be Sent in Error! - https://phabricator.wikimedia.org/T401752#11104608 (10SHust) @Eileenmcnaughton, below are a few CIDs I spot checked, and the secondary email is still not opt... [22:27:18] (03PS1) 10Eileen: Add db prefix to option values [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180673 [22:27:34] ejegg: ^^ should fix the error we just hit [22:58:36] (03CR) 10Cstone: [C:03+2] Add db prefix to option values [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180673 (owner: 10Eileen) [22:59:50] (03Merged) 10jenkins-bot: Add db prefix to option values [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/1180673 (owner: 10Eileen) [23:12:29] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180677 [23:12:44] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180677 (owner: 10Eileen) [23:13:22] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/tools into deploy [wikimedia/fundraising/tools] (deploy) - 10https://gerrit.wikimedia.org/r/1180677 (owner: 10Eileen) [23:14:39] !log tools upgraded from 0d6df34a to 284579a9 [23:14:41] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log