[03:33:21] (03CR) 10Cstone: [C:03+2] "Looks good, one times get to donations queue, recurrings get to pendings. Still not able to get a test IPN from adyen but it was the same " [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037848 (owner: 10Ejegg) [03:33:48] (03Merged) 10jenkins-bot: Set initial SEPA/iDEAL recurrings to 'pending' [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037848 (owner: 10Ejegg) [07:33:36] (03PS1) 10Eileen: Upgrade drupal, admin_menu, ctools [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1039168 [07:53:39] (03CR) 10CI reject: [V:04-1] Upgrade drupal, admin_menu, ctools [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1039168 (owner: 10Eileen) [12:51:15] (03PS12) 10Damilare Adedoyin: Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) [12:51:33] (03CR) 10CI reject: [V:04-1] Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) (owner: 10Damilare Adedoyin) [13:21:08] (03PS13) 10Damilare Adedoyin: Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) [13:21:27] (03CR) 10CI reject: [V:04-1] Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) (owner: 10Damilare Adedoyin) [13:34:18] (03PS14) 10Damilare Adedoyin: Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) [13:34:35] (03CR) 10CI reject: [V:04-1] Gravy create payment [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1037776 (https://phabricator.wikimedia.org/T364506) (owner: 10Damilare Adedoyin) [13:53:14] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 6642 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [13:58:14] RECOVERY - check_mysql on frdb1004 is OK: Uptime: 1711764 Threads: 5 Questions: 70965148 Slow queries: 794 Opens: 4768 Open tables: 1149 Queries per second avg: 41.457 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:22:29] damilare|brb: cstone: so those adyen failmails seem to be from RecordCaptureJob [14:23:27] let's make sure that job doesn't send a message when it's got no token [14:23:51] and then maybe the RecurringContractJob could add a token to the pending row for non-ideal / non-sepa [14:31:07] hmm, so what's the simplest flow here [14:49:07] (03PS1) 10Ejegg: Better missing token flow for Adyen [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1039230 (https://phabricator.wikimedia.org/T365884) [14:49:22] so ^^^ seems like the minimum change from current flow [14:49:31] (03CR) 10CI reject: [V:04-1] Better missing token flow for Adyen [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1039230 (https://phabricator.wikimedia.org/T365884) (owner: 10Ejegg) [14:49:57] but I'm wondering if we should unify the ideal / sepa handling [14:50:33] if they always get the same order of RecurringContractJob first, then RecordCaptureJob [14:51:12] (03PS2) 10Ejegg: Better missing token flow for Adyen [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1039230 (https://phabricator.wikimedia.org/T365884) [15:18:23] (03PS1) 10Jgleeson: WIP: Add CiviCRM Standalone [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1039235 [15:23:00] fr-tech that last WIP patch gets CiviCRM Standalone running on the fun-dev stack. It's WIP as it still needs a good click through and there's some odd network error alert that I need to dig into but if folks want a sneak peek at the UI on standalone, that should do it. [15:23:29] hey - as long as there's no contact-form or any kinda fedback - posting here:  It's sad & unfortunate, but reality - as long as wiki doesn't accept cprypto - that long we won't donate. By 'we' I mean huge community which understand, that wiki is reasonable project, but on the other hand - as long as it doesn't support crypto - it doesn't deserve [15:23:29] to get any benefits out of it. Thus - feel free repetitively beg for donations over shtty Paypal/ another Google sht or stoneage institutions...  have a nice one! [15:45:40] bummer of a hit-n-run comment there... [15:46:37] also, nice one jgleeson|civi ! [15:46:43] I'll definitely take that for a spin [15:50:22] interesting that it still has a separate dsn for the cms [16:06:12] ejegg: I'm pretty sure we can get rid of that cms db. it doesn't get populated with anything [16:37:04] XenoRyet: I forgot at planning yesterday - shust asked that we consider https://phabricator.wikimedia.org/T341247 for doing [16:37:24] I'd like to get better data in that field too [16:37:38] mind if I pull it into sprint? I don't think it'll be too huge [18:13:49] cstone: oh interesting, no RecurringContract IPN for Roan's SEPA [18:13:56] zgrep 206223888 payments-listener-adyen-2024052* [18:14:21] just shows the AUTHORIZATION and then the REFUND 3 days later [18:15:11] but the authorization has a sepadirectdebit.mandateId [18:17:08] gonna test recurring SEPA in dev with the IPN proxy on [18:19:55] Feel free to do more test transactions with my account too (if you're able to reuse one of my tokenized transactions) [18:20:37] I'm not gonna be in Europe for the next 9 months or so and I won't miss a couple dozen euros [18:21:10] ah thanks Roan! I think we didn't stash enough details to start a new one ourselves, but we may hit you up once we have a new revision on production. [18:21:32] (The bit we would want to test would be starting a new one rather than charging an existing one) [18:25:14] hmm, no processorContactID on that response either [18:27:00] oh, so that usually is on recurring.shopperReference [18:28:38] hmm, docs suggest we SHOULD get a RECURRING_CONTRACT IPN: https://docs.adyen.com/payment-methods/sepa-direct-debit/web-component/ [18:43:46] ejegg: i couldnt get one locally [18:44:15] sorry ejegg the one on the account wasnt the recurring one [18:44:21] oh i see [18:44:24] did you find that recurring one from the email though I can pull it out of adyen [18:44:39] oh i can look [18:44:42] oh its the phab [18:44:45] for the broken sepa [18:44:47] thanks! [18:45:12] https://phabricator.wikimedia.org/T366231 [18:45:17] sorry forgot I had put it there [18:45:26] it says name missing but the issue was no pending msg [18:45:32] ill deploy that fix now and we can retest [18:45:56] ah yeah i just tagged smashpig [18:46:01] need to update in packagist still [18:46:29] ok, updated in packagist to v0.8.10.10 [18:46:34] ok thanks [18:53:06] (03PS1) 10Cstone: Update SmashPig to v0.8.10.10 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039277 [18:56:28] ok, just checked again on the Carte Bancaire (non-visa) test data, and it's got a submethod in the front-end (mapped from the JS component) as well as in the getHostedPaymentDetails result [18:56:57] also i seem to not be getting ANY ipns even with forwarding on [18:57:07] my endpoint is marked as 'failing' [18:57:08] hmm [18:57:59] oh hah, here they all are [18:58:08] was just waiting for me to complain [18:59:18] you need to do the troubleshoot i think before it trigers the giant dump [18:59:25] i was testing a lot last night so probably more than usual too [18:59:52] (03CR) 10Cstone: [C:03+2] Update SmashPig to v0.8.10.10 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039277 (owner: 10Cstone) [19:00:16] oh hmmm all the tokens lost in the last hour?? [19:01:00] ohohoo french banner test [19:01:10] ahh [19:01:24] 10fundraising-tech-ops, 06Infrastructure-Foundations, 10Mail: Update fundraising mail settings to use new production mx hosts - https://phabricator.wikimedia.org/T366740 (10Dwisehaupt) 03NEW [19:02:50] ooooh is everyting being routed through cartebaincare?? [19:03:31] oh it just ended [19:03:34] well [19:17:05] (03Merged) 10jenkins-bot: Update SmashPig to v0.8.10.10 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039277 (owner: 10Cstone) [19:42:53] (03PS1) 10Ejegg: Don't overwrite submethod='' with null [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039283 [19:47:44] (03PS2) 10Ejegg: Don't overwrite submethod='' with null [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039283 [19:49:31] (03CR) 10Damilare Adedoyin: [C:03+2] Don't overwrite submethod='' with null [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039283 (owner: 10Ejegg) [19:50:05] (03PS1) 10Ejegg: Fix a bit of outdated documentation [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039286 [19:51:24] (03Merged) 10jenkins-bot: Don't overwrite submethod='' with null [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039283 (owner: 10Ejegg) [19:59:40] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1039290 [19:59:42] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1039290 (owner: 10Ejegg) [20:02:03] (03PS1) 10Ejegg: Update composer.lock for smashpig upgrade [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1039291 [20:02:05] (03CR) 10Ejegg: [C:03+2] Update composer.lock for smashpig upgrade [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1039291 (owner: 10Ejegg) [20:03:05] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1039290 (owner: 10Ejegg) [20:06:22] !log payments-wiki upgraded from c255fda8 to 82a5e588 [20:06:23] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:09:51] (03CR) 10CI reject: [V:04-1] Update composer.lock for smashpig upgrade [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1039291 (owner: 10Ejegg) [20:16:20] (03PS2) 10Ejegg: Update composer.lock for smashpig upgrade [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/1039291 [20:17:42] (03PS1) 10Ejegg: Another fallback for CC submethod under Adyen [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1039296 [20:30:04] 10fundraising-tech-ops, 06Infrastructure-Foundations, 10Mail: Update fundraising mail settings to use new production mx hosts - https://phabricator.wikimedia.org/T366740#9865675 (10Dwisehaupt) a:03Dwisehaupt PFW and iptables changes pushed. Awaiting pfw rollout (T366753) before we can test. [20:56:57] fr-tech very small SmashPig fix: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1038888 [20:58:22] oh right i wanted to look at that DR View bug [21:05:37] well, I can replicate the in_array() expects parameter 2 to be array bug [21:06:06] assistant block is there twice??