[06:18:09] (03CR) 10Thiemo Kreuz (WMDE): [C: 03+1] Fix a couple of undefined var errors (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/676472 (owner: 10Ejegg) [06:23:49] (03CR) 10AndyRussG: "Cool! Enabling ApplePay with this worked, just had to also add 443 to the ports.conf file inside the container." [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/711216 (https://phabricator.wikimedia.org/T288772) (owner: 10Jgleeson) [06:29:52] (03CR) 10AndyRussG: [V: 03+2] Remove a line in config/payments/LocalSettings.php so tests pass [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/703259 (https://phabricator.wikimedia.org/T266088) (owner: 10AndyRussG) [06:51:00] (03CR) 10AndyRussG: WIP Adyen Checkout: Applepay (031 comment) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/713551 (https://phabricator.wikimedia.org/T282596) (owner: 10Jgleeson) [11:21:28] 10Fundraising-Backlog: Campaign Notification list - https://phabricator.wikimedia.org/T289478 (10Peachey88) [11:22:07] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Campaign Notification list - https://phabricator.wikimedia.org/T289478 (10Peachey88) [14:27:14] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising Sprint NULL calorie food cart, 10fundraising sprint Parallel processing roller coaster, and 4 others: Recurring donors segmentation criteria Civi/Acoustic - https://phabricator.wikimedia.org/T283798 (10KHaggard) Hi @Eileenmcnaughton - I'... [15:48:16] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10MW-1.37-notes (1.37.0-wmf.20; 2021-08-23): Non-English descriptor change back to 877 number string - https://phabricator.wikimedia.org/T289171 (10EMartin) Noting here that Vantiv only allows these characters in their descriptor... [15:54:21] 10Fundraising-Backlog, 10FR-AutoTY-Email, 10fr-donorservices: Civi TY email shows two different dates for same contrib - https://phabricator.wikimedia.org/T289165 (10MBeat33) 05Open→03Resolved a:03MBeat33 We looked at this one at the last DR-Fr-Tech checkin, and as it seems like this is most likely a U... [16:05:38] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-codfw: (Need By: TBD) rack/setup/install frdb2003.frack.codfw.wmnet - https://phabricator.wikimedia.org/T281177 (10Papaul) ` papaul@fasw-c-codfw# run show interfaces ge-0/0/5 terse Interface Admin Link Proto Local Remote ge-0/0... [16:13:31] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Link mailing data to contacts imported from RML - https://phabricator.wikimedia.org/T222854 (10Ejegg) @Eileenmcnaughton what do you think of adding the RML contacts to the silverpop exports so we can populate the contact_id and contact_hash... [16:28:28] 10Fundraising-Backlog, 10FR-Adyen: Set up Additional Apple Pay Test servers - https://phabricator.wikimedia.org/T289511 (10jgleeson) [16:40:12] (03CR) 10Ejegg: [C: 03+2] "Looks good!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/713941 (https://phabricator.wikimedia.org/T261689) (owner: 10Jgleeson) [16:49:44] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-codfw: (Need By: TBD) rack/setup/install frdb2003.frack.codfw.wmnet - https://phabricator.wikimedia.org/T281177 (10Papaul) [16:51:49] (03Merged) 10jenkins-bot: Add unit tests for civicrm import stats patch to confirm stats output. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/713941 (https://phabricator.wikimedia.org/T261689) (owner: 10Jgleeson) [17:05:54] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-codfw, 10Patch-For-Review: (Need By: TBD) rack/setup/install frdb2003.frack.codfw.wmnet - https://phabricator.wikimedia.org/T281177 (10Papaul) [17:06:30] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-codfw, 10Patch-For-Review: (Need By: TBD) rack/setup/install frdb2003.frack.codfw.wmnet - https://phabricator.wikimedia.org/T281177 (10Papaul) 05Open→03Resolved @Jgreen this is complete on my end . Let me know if you have any issues. [17:08:42] thanks ejegg ! [17:13:20] hmm lemme see why I'm calling that ctQueueConsume [17:13:27] that might be an oversight [17:17:25] copy-pasted perhaps? [17:19:09] probably! [17:19:19] just gonna remove it and run tests again [17:38:58] (03PS1) 10Jgleeson: Remove unused code from ImportMessage stats unit tests. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714390 (https://phabricator.wikimedia.org/T261689) [17:58:08] 10Fundraising-Backlog, 10FR-Adyen: Set up Additional Apple Pay Test servers - https://phabricator.wikimedia.org/T289511 (10jgleeson) DNS Entries added test1.jackgleeson.co.uk A Simple - 3.230.160.72 test2.jackgleeson.co.uk A Simple - 3.228.22.226 test3.jackgleeson.co.uk A Simple - 3.237.186.242 test4.ja... [17:59:01] 10Fundraising-Backlog, 10FR-Adyen: Set up Additional Apple Pay Test servers - https://phabricator.wikimedia.org/T289511 (10jgleeson) Public keys from [[ https://docs.google.com/document/d/1Vva2ZBxOO-wb9dPu7aiRU6AlljmxpO7N0On9gzailC8/edit | here ]] added to authorized_keys file for root across the five machines. [18:02:35] AndyRussG: cstone fr-tech, sorry for asking you to test the new Apple Pay servers last week. I'd messed up the setup by adding your keys to the wrong user and also forgot to added the DNS entries for the test*.jackgleeson.co.uk subdomains. I did manage to connect into the five machines myself but that was using the DNS hostnames but I should have also tried to the subdomains! sorry about that [18:03:10] you should now be able to run `ssh -T test1.jackgleeson.co.uk date` and it /should/ work [18:03:56] cycling through the test[1-5].jackgleeson.co.uk addresses [18:04:53] looks like when I cloned the initial server in AWS to make the following four, the clone behaviour doesn't copy the filesystem [18:05:26] so I also need to update the sshd config to allow tunnelling in the way we're using it [18:06:30] damilare: I've also added your key so you should have access too [18:07:06] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10FR-Adyen: Set up Additional Apple Pay Test servers - https://phabricator.wikimedia.org/T289511 (10jgleeson) a:03jgleeson [18:09:11] gonna grab food [18:20:14] jgleeson I [18:20:22] 'm getting the DNS resolution fine [18:20:33] but permission denied (publickey) [18:20:38] when I try to connect [18:20:46] (to 1 and 2 at least) [18:21:08] thanks again for setting all those up! [18:30:59] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster: Catch Adyen Checkout API errors on the DI side - https://phabricator.wikimedia.org/T288830 (10AndyRussG) Hi! Quick question: do we have more details on which sorts of errors we wish to catch? The alert following an actual API reque... [18:36:01] (03PS3) 10Ejegg: Fix a couple of undefined var errors [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/676472 [18:36:03] (03PS5) 10Ejegg: Get rid of card_type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/676473 [18:36:35] (03CR) 10Ejegg: "Thanks for the review! PS5 is updated with your suggestions." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/676473 (owner: 10Ejegg) [18:38:29] (03CR) 10Ejegg: "Thanks for the CR AndyRussG! I've renamed that variable in PS3" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/676472 (owner: 10Ejegg) [18:58:33] Thanks jgleeson, although I'm still getting permission deniod [19:04:58] ah shoot so it's still not working for other folks [19:05:01] hmmm [19:08:52] could I grab someone fr-tech to be help test? [19:09:11] sure jgleeson [19:09:22] thanks cstone wanna jump on today's call? [19:09:27] okie [19:09:30] ty! [19:11:23] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fr-matching-gifts: when an employer is not in HEP data or misspelled, save the string in the "employer" custom field - https://phabricator.wikimedia.org/T289083 (10Ejegg) a:03Ejegg [19:12:34] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fr-matching-gifts: when an employer is not in HEP data or misspelled, save the string in the "employer" custom field - https://phabricator.wikimedia.org/T289083 (10Ejegg) OK, this is working when it creates a new contact. When... [19:13:42] ok fr-tech we figured it ut [19:13:45] finally got into the last server on the list jgleeson [19:13:47] out* [19:14:00] ohh, probably why it worked [19:14:02] and it was due to me giving the wrong instructions [19:14:41] so instead of trying `ssh -T test1.jackgleeson.co.uk date` [19:14:49] it should have actually been `ssh -T root@test1.jackgleeson.co.uk date` [19:14:56] (I'd missed the user off) [19:14:57] oh hah, right [19:15:06] my bad [19:15:14] cstone just managed to connect in to all five [19:15:27] so hopefully it'll be the same for y'all [19:15:30] yep, I can get in too [19:16:26] great! [19:18:01] got in too, thanks cstone jgleeson [19:19:01] awesome! [19:19:21] ok so the last bit is to update the sshd config to allow us to tunnel externally [19:19:35] GatewayPorts or something [19:21:40] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10FR-Adyen: Set up Additional Apple Pay Test servers - https://phabricator.wikimedia.org/T289511 (10jgleeson) [19:26:45] 10Fundraising-Backlog, 10FR-Adyen: Adyen Checkout: Handle network error when loading Adyen JS - https://phabricator.wikimedia.org/T289525 (10AndyRussG) [19:27:34] 10Fundraising-Backlog, 10FR-Docker: Docker dev setup: try out and set up stuff for silverpop export - https://phabricator.wikimedia.org/T289526 (10AndyRussG) [20:04:39] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Formatting of Acoustic contact field "Last Native Amount" with decimal point alters how the amount populates in Donatewiki - https://phabricator.wikimedia.org/T289091 (10DStrine) [20:12:35] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Link mailing data to contacts imported from RML - https://phabricator.wikimedia.org/T222854 (10DStrine) [20:34:15] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Parallel processing roller coaster, 10Recurring-Donations: Trickle of damaged recurring donation messages with contact_id but without gateway_txn_id - https://phabricator.wikimedia.org/T285396 (10Cstone) a:03Cstone [20:45:36] AndyRussG: do you wanna quickly talk through the apple pay stuff? [20:45:45] I'm hanging around for a bit longer [20:47:03] in the meantime, Backyard Squirrel Maze [20:47:51] jgleeson: sure, maybe in 5 min? [20:47:55] thanks! [20:48:20] sure! [20:55:19] jgleeson: K anytime is fine now [20:55:33] we could use the same hangout from the last call if u like? [20:55:43] sure heading back now [20:58:39] ^ fr-tech in case anyone else wants to join in https://meet.google.com/mkm-dptq-dqq?authuser=0 [21:06:09] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising Sprint NULL calorie food cart, 10fundraising sprint Parallel processing roller coaster, and 4 others: Recurring donors segmentation criteria Civi/Acoustic - https://phabricator.wikimedia.org/T283798 (10Eileenmcnaughton) @KHaggard that's... [21:13:17] oh drat, docker's not working after a package update [21:14:02] AndyRussG: if you're still on debian experimental, might want to watch out for that [21:31:20] ejegg: ah hmmm thanks for the heads-up [21:31:48] mostly I run a combination of testing and unstable [21:32:20] mmm, apt pick-n-mix [21:32:59] cat /etc/apt/apt.conf.d/10default-release [21:33:01] APT::Default-Release "bullseye" [21:33:14] (used to be testing anyway) [21:33:23] oh interesting, i don't have that file [21:33:25] I have the other distros, including unstable and experimental in sources list [21:33:33] ah yeah I always add that myself [21:33:41] so most packages will be on that distro [21:33:58] and then when I want something newer I can say aptitude install -t experimental firefox [21:34:26] also you can say apt-cache madison firefox (or any package name) [21:34:40] to see what versions of any given package are available in any distribution apt knows about [21:34:42] ahh, ok, so you have sources.list entries for experimental and unstable? [21:34:51] yep! [21:34:55] nice nice [21:34:56] and also respective security upates [21:35:01] right [21:46:58] (03PS1) 10Ejegg: Update Employer custom field on contact match [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) [21:48:27] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fr-matching-gifts, 10MW-1.37-notes (1.37.0-wmf.20; 2021-08-23): add employer drop down to all matching gifts variants - https://phabricator.wikimedia.org/T289082 (10Ejegg) a:03Ejegg [21:50:07] 10Fundraising-Backlog, 10fundraising sprint Parallel processing roller coaster, 10fr-matching-gifts: Enable and extend process-control matching_gifts_sync job to also call civicrm-matching-gifts-employers-check - https://phabricator.wikimedia.org/T273555 (10Ejegg) a:03Ejegg [22:01:09] (03CR) 10Eileen: "ejegg - so looking at this I think the concern I have is that this class has an update process & a create process - for legacy reasons - I" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) (owner: 10Ejegg) [22:27:24] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Campaign Notification list - https://phabricator.wikimedia.org/T289478 (10Eileenmcnaughton) Whoa are you getting this error? {F34618290} [22:56:22] (03PS2) 10Ejegg: Update Employer custom field on contact match [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) [22:58:04] (03CR) 10Ejegg: "Thanks for the CR Eileen!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) (owner: 10Ejegg) [22:59:15] (03CR) 10Eileen: [C: 03+2] "OK - happy to leave the imposition of sanity for later" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) (owner: 10Ejegg) [23:10:39] (03Merged) 10jenkins-bot: Update Employer custom field on contact match [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714423 (https://phabricator.wikimedia.org/T289083) (owner: 10Ejegg) [23:11:15] (03PS4) 10Eileen: Stock CiviCRM 5.41 rc [wikimedia/fundraising/crm/civicrm] - 10https://gerrit.wikimedia.org/r/713378 [23:19:45] (03PS1) 10Ejegg: Use handleUpdate for all updates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714434 [23:21:36] (03PS2) 10Ejegg: Use handleUpdate for all contact updates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714434 [23:22:31] (03CR) 10Eileen: "I think this is probably fine (will dig a bit deeper once tests finish) - but note that the Contact::save() api call can do updates and cr" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714434 (owner: 10Ejegg) [23:22:55] (03CR) 10jerkins-bot: [V: 04-1] Stock CiviCRM 5.41 rc [wikimedia/fundraising/crm/civicrm] - 10https://gerrit.wikimedia.org/r/713378 (owner: 10Eileen) [23:42:10] (03PS1) 10Eileen: Further api fix for 5.41 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714437 [23:43:54] ejegg: I think ^^ is the last fix to stop tests failing once we switch to 5.41 [23:44:10] cool, will look in a sec [23:44:28] yep & I'll check your one :-) [23:44:49] thanks! [23:45:41] (03CR) 10Ejegg: [C: 03+2] Further api fix for 5.41 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/714437 (owner: 10Eileen) [23:55:28] (03CR) 10Eileen: "recheck" [wikimedia/fundraising/crm/civicrm] - 10https://gerrit.wikimedia.org/r/713378 (owner: 10Eileen)