[07:20:35] (CR) jenkins-bot: Localisation updates from https://translatewiki.net. [extensions/FundraisingEmailUnsubscribe] - https://gerrit.wikimedia.org/r/502118 (owner: L10n-bot) [11:33:12] fr-tech, fr-tech-ops, attack of the failmail this morning? [12:48:18] (CR) Jgleeson: Create queue consumer for contribution tracking messages and add drush command to call it. (3 comments) [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 (owner: Jgleeson) [13:55:49] Fundraising-Backlog, MediaWiki-extensions-CentralNotice: Have a way to import a list of languages or a list of wikis to target in CentralNotice - https://phabricator.wikimedia.org/T220386 (Trizek-WMF) [14:22:07] (CR) Ejegg: [C: -1] "Let's just add the contribution_id field back in, to support the case of adding that when populating the civicrm_contribution table. We ca" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 (owner: Jgleeson) [14:25:55] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Weird language code crashes dq consumer - https://phabricator.wikimedia.org/T190467 (Ejegg) Seeing this again in April 2019 with kn_IN [14:35:13] hmm rebuilding vagrant with no power supply close wasn't a good idea [14:35:27] pre-build, 100% battery. post-build, 28% battery [14:36:27] ha, wow jgleeson! [14:36:36] yup! lol [14:40:04] oh jeez [14:48:59] Fundraising-Backlog, MediaWiki-Vagrant: Add php7.2 xdebug support - https://phabricator.wikimedia.org/T220406 (jgleeson) [14:49:20] Fundraising-Backlog, MediaWiki-Vagrant: Add mediawiki-vagrant php7.2 xdebug support - https://phabricator.wikimedia.org/T220406 (jgleeson) [15:00:39] Fundraising-Backlog, MediaWiki-Vagrant, Documentation: Documentation: Review/Enhance fr-tech vagrant docs - https://phabricator.wikimedia.org/T220408 (jgleeson) [15:06:43] hi mepps! [15:06:49] hey ejegg! [15:06:57] oh hey, sorry [15:06:58] I'm in the meet chat if you want to check in [15:07:06] i got my meeting times confused [15:10:29] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, MediaWiki-Vagrant: Figure out why CiviCRM tests do not like mediawiki-vagrant and fix them. - https://phabricator.wikimedia.org/T220411 (jgleeson) [15:10:40] Fundraising-Backlog, fundraising-tech-ops, Mail, Operations: Identify appropriate SPF record for domain wikimediafoundation.org - https://phabricator.wikimedia.org/T220412 (herron) p:Triage→Normal [15:11:47] (CR) Thiemo Kreuz (WMDE): [C: +1] "Consider this basically a +2." (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/501902 (owner: D3r1ck01) [15:12:47] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, MediaWiki-Vagrant: Figure out why CiviCRM tests do not like mediawiki-vagrant and fix them. - https://phabricator.wikimedia.org/T220411 (jgleeson) This was an issue last week as a patch I added introduced a regression elsewhere which I didn't pick u... [15:17:53] (PS3) D3r1ck01: Banner: Fix regex in non-unicode mode and class reference [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/501902 [15:18:28] (CR) D3r1ck01: Banner: Fix regex in non-unicode mode and class reference (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/501902 (owner: D3r1ck01) [15:21:01] Fundraising-Backlog, Analytics: CentralNoticeImpression refined impressionEventSampleRate is int instead of double - https://phabricator.wikimedia.org/T217109 (Milimetric) The easiest thing to do is to delete the old data and change the schema going forward. Let me know if this is ok to do, @AndyRussG.... [15:29:23] (PS27) Jgleeson: Create queue consumer for contribution tracking messages and add drush command to call it. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 [15:30:02] back soon, gonna find somewhere with power as I'm down to 9% [16:53:21] fundraising-tech-ops: upgrade fundraising puppetmasters to Debian Buster - https://phabricator.wikimedia.org/T219315 (Jgreen) Open→Resolved frpm1001=buster too! [16:53:23] fundraising-tech-ops: EPIC: migrate fundraising hosts to Debian Stretch - https://phabricator.wikimedia.org/T185013 (Jgreen) [16:53:40] fundraising-tech-ops: migrate pay-lvs servers to Debian Stretch - https://phabricator.wikimedia.org/T185014 (Jgreen) a:Jgreen pay-lvs1001 is done [17:08:45] Fundraising-Backlog: Dynamic 3ds for Sweden? - https://phabricator.wikimedia.org/T220427 (DStrine) [17:11:32] fundraising-tech-ops: migrate pay-lvs servers to Debian Stretch - https://phabricator.wikimedia.org/T185014 (Jgreen) pay-lvs2002 is done [17:16:20] Fundraising-Backlog: 1-month update for privacy policy - https://phabricator.wikimedia.org/T220428 (DStrine) [17:42:29] (CR) Krinkle: Banner: Fix regex in non-unicode mode and class reference (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/501902 (owner: D3r1ck01) [19:05:52] (CR) Thiemo Kreuz (WMDE): [C: +1] Banner: Fix regex in non-unicode mode and class reference (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/501902 (owner: D3r1ck01) [20:08:58] (PS1) Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/502333 [20:10:19] Fundraising-Backlog, fundraising-tech-ops, Mail, Operations: Identify appropriate SPF record for domain wikimediafoundation.org - https://phabricator.wikimedia.org/T220412 (Jgreen) As far as I know, fundraising does not send mail using this domain but only from wikimedia.org, so I don't think our... [20:10:24] (CR) Eileen: [C: +2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/502333 (owner: Eileen) [20:11:24] (Merged) jenkins-bot: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/502333 (owner: Eileen) [20:12:48] Fundraising-Backlog: Inaccurate placement for the currency symbol (SEK) in Sweden Thank you letter - https://phabricator.wikimedia.org/T220192 (CCogdill_WMF) Can we just remove the amount from the subject line altogether? I talked to @MBeat33 and he is okay with it. I'm happy to update the task description. [20:14:26] (PS1) Eileen: Add test processor on install (already added to live site [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/502334 [20:19:50] Fundraising-Backlog: Inaccurate placement for the currency symbol (SEK) in Sweden Thank you letter - https://phabricator.wikimedia.org/T220192 (Ejegg) Hi @JRobell, the amount formatting seems to choose the placement based on the language. For Swedish language letters, it puts the currency symbol after the am... [20:28:08] Fundraising-Backlog: updates to opt-ins on failed payments - https://phabricator.wikimedia.org/T220446 (DStrine) [20:34:14] ejegg: https://gerrit.wikimedia.org/r/#/c/wikimedia/fundraising/crm/+/502334/ might be needed for tests [20:35:07] eileen: aha, that would have caught the issue in the SmashPig processor [20:35:22] (CR) Ejegg: [C: +2] Add test processor on install (already added to live site [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/502334 (owner: Eileen) [20:35:40] or rather the issue with the recurring payment token import [20:36:07] is that an issue on current code or only the next civi drop? [20:36:38] current code - i put up a fix on Friday [20:36:53] and ran all the broken messages back through [20:37:29] I had just been using getsingle with only name as a filter, to get the processor ID to create a token records [20:37:40] the friday patch was just to add is_test=>0 to the filter [20:41:44] (Merged) jenkins-bot: Add test processor on install (already added to live site [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/502334 (owner: Eileen) [20:52:06] Fundraising-Backlog, MediaWiki-extensions-CentralNotice: Have a way to import a list of languages or a list of wikis to target in CentralNotice - https://phabricator.wikimedia.org/T220386 (Jseddon) p:Triage→Lowest [21:10:03] (PS3) Eileen: Update to civicrm 5.13 stock [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/500646 (https://phabricator.wikimedia.org/T218754) [21:13:59] (CR) jerkins-bot: [V: -1] Update to civicrm 5.13 stock [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/500646 (https://phabricator.wikimedia.org/T218754) (owner: Eileen) [21:15:15] ejegg: dang I still got a test failing :-( - will figure it out [21:21:08] fundraising-tech-ops: EPIC: migrate fundraising off of Debian Jessie - https://phabricator.wikimedia.org/T185013 (Jgreen) [21:22:12] fundraising-tech-ops: upgrade frdev1001 to Debian Buster - https://phabricator.wikimedia.org/T220452 (Jgreen) [21:22:34] Fundraising-Backlog, fundraising-tech-ops: Update SciPy on frdev1001 - https://phabricator.wikimedia.org/T213948 (Jgreen) [21:23:26] fundraising-tech-ops: upgrade frdev1001 to Debian Buster - https://phabricator.wikimedia.org/T220452 (Jgreen) [21:23:28] Fundraising-Backlog, fundraising-tech-ops: Re-image frdev1001 to stretch - https://phabricator.wikimedia.org/T214115 (Jgreen) [21:25:21] fundraising-tech-ops: upgrade frdev1001 and mintaka to Stretch, possibly Buster - https://phabricator.wikimedia.org/T220453 (Jgreen) [21:49:56] eileen: looks like that 'missing' PaymentProcessor 'pay' api method that you shimmed in your omnipay ext and I copied for my SmashPig ext [21:49:59] is in core now? [21:50:12] ejegg: yep I added it [21:50:17] cool, cool [21:50:48] also PaymentProcessor.refund although it’s imature - someone else was working on that [21:51:04] but I kinda rejected most of their code & added the bare basics of it [21:51:43] (I don’t know the code had to be rejected but I wasn’t prepared to accept it without more discussion so I wanted to separate the ‘obvious yes’ stuff from the ‘hmm maybe’ stuff) [21:51:46] ok, looks like we can just get rid of the copy of it in SmashPig then? [21:51:52] yeah probably [21:51:55] that one was pretty bare bones [21:52:08] I haven’t checked whether omnipay will be upset by it yet but will check [21:53:47] fundraising-tech-ops, Operations, netops, Patch-For-Review: Revoke production prometheus fundraising access - https://phabricator.wikimedia.org/T217355 (cwdent) [21:54:46] fundraising-tech-ops, Operations, netops, Patch-For-Review: Revoke production prometheus fundraising access - https://phabricator.wikimedia.org/T217355 (cwdent) @ayounsi there is a new config at -1554758904, removing prod prometheus and grafana access to pay-lvs servers. [23:37:22] (PS28) Ejegg: Create queue consumer for contribution tracking messages and add drush command to call it. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 (owner: Jgleeson) [23:37:56] (CR) Ejegg: [C: +2] "Good stuff!" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 (owner: Jgleeson) [23:42:08] (Merged) jenkins-bot: Create queue consumer for contribution tracking messages and add drush command to call it. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/498413 (owner: Jgleeson)