[15:10:25] fr-tech there's a failmail an hour old from a bad iDEAL payments-wiki message (damaged/257525) [15:10:35] I'd really like more info on how those are getting sent [15:10:45] can anyone review the debug logging patch for me? [15:10:56] ejegg: sure which one is it again? [15:11:12] https://gerrit.wikimedia.org/r/594306 [15:11:29] ejegg: okok thanks! [15:11:34] oh, i'll smoke test it locally too [15:11:49] with a hack to unset that key [15:14:18] ejegg: okok :) [15:17:19] Looks like it works, but I'll just tuck that RequestContext::getMan() bit inside the condition where we need it [15:18:00] and rebase [15:18:17] (PS2) Ejegg: Debug logging for missing payment_submethod [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/594306 (https://phabricator.wikimedia.org/T251025) [15:18:19] oh right, there's another patch under that one [15:18:25] just adding a unit test [15:19:24] if you don't mind peeking at https://gerrit.wikimedia.org/r/594213 too, that'd be genial [15:20:07] I had to look up genial then ejegg [15:20:28] heh, it's also useful in spanish [15:21:52] :) [15:25:56] fr-tech anything you want me to put in the scrum of scrums update? [15:26:28] ejegg: nothing here, thanks!! [15:27:17] ejegg: no news from the riddermark here [15:30:18] ok, cool [15:54:50] Fundraising-Backlog, FR-Adyen, FR-Q2-FY2019-20-cleanup-list: Process Adyen IPN messages in parallel (improve TY delivery times) - https://phabricator.wikimedia.org/T233251 (DStrine) [15:56:52] Fundraising-Backlog, FR-Adyen, FR-Q2-FY2019-20-cleanup-list: Process Adyen IPN messages in parallel (improve TY delivery times) - https://phabricator.wikimedia.org/T233251 (Ejegg) It would be nice to have a generalized way to multiplex any of the queues defined in our config yaml files. [15:57:12] Fundraising-Backlog, FR-Adyen, FR-Q2-FY2019-20-cleanup-list: Process Adyen IPN messages in parallel (improve TY delivery times) - https://phabricator.wikimedia.org/T233251 (MBeat33) I'm concerned that with scaling up transaction volume (if we go w/Adyen for EN6C cards for example) we might see an inc... [15:58:00] Fundraising-Backlog, fundraising-tech-ops: Superset user bug: password - https://phabricator.wikimedia.org/T252034 (EYener) [15:59:10] Fundraising-Backlog, fundraising-tech-ops: Superset user certificate - https://phabricator.wikimedia.org/T252035 (EYener) [16:16:34] (PS3) Ejegg: Debug logging for missing payment_submethod [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/594306 (https://phabricator.wikimedia.org/T251025) [16:27:49] (PS4) AndyRussG: Debug logging for missing payment_submethod [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/594306 (https://phabricator.wikimedia.org/T251025) (owner: Ejegg) [16:29:16] (CR) AndyRussG: [C: +2] "Cool beans! Thanks also for the explanation and smoke test! :)" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/594306 (https://phabricator.wikimedia.org/T251025) (owner: Ejegg) [16:34:26] (Merged) jenkins-bot: Debug logging for missing payment_submethod [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/594306 (https://phabricator.wikimedia.org/T251025) (owner: Ejegg) [16:40:48] Fundraising Sprint Ice cream cures everything, Fundraising-Backlog: Spelling error in Dutch TY email - https://phabricator.wikimedia.org/T251782 (DStrine) [16:55:28] Fundraising-Backlog, fundraising-tech-ops: puppet framework for Advancement Analytics staff and server roles - https://phabricator.wikimedia.org/T244859 (Jgreen) Open→Resolved [16:55:30] Fundraising-Backlog, fundraising-tech-ops: Fundraising Analytics Infrastructure and Setup - https://phabricator.wikimedia.org/T238395 (Jgreen) [16:58:08] Fundraising-Backlog, fundraising-tech-ops: Set up database dumps and archiving for fundraising analytics mariadb server. - https://phabricator.wikimedia.org/T251921 (Jgreen) Open→Resolved This is done and tested for frdb1003 and frpm1001 (remote dump from payments wiki db). The script changes are... [16:58:10] Fundraising-Backlog, fundraising-tech-ops: Fundraising Analytics Infrastructure and Setup - https://phabricator.wikimedia.org/T238395 (Jgreen) [16:58:54] Fundraising-Backlog, fundraising-tech-ops: Investigate the rebrand url changes - https://phabricator.wikimedia.org/T248700 (Jgreen) [17:13:27] fr-tech, did yall see how you're meant to pronounce PECL... it's "pickle" apparently https://pear.php.net/manual/en/about.pear.php ⚆ _ ⚆ [17:32:47] i'll be there in a couple min fr-tech [17:39:24] Fundraising-Backlog: Investigate pulling in page view data to our version of superset - https://phabricator.wikimedia.org/T252049 (DStrine) [17:40:53] Fundraising-Backlog, fundraising-tech-ops: Investigate pulling in page view data to our version of superset - https://phabricator.wikimedia.org/T252049 (DStrine) [17:53:26] Fundraising-Backlog, fundraising-tech-ops: Investigate pulling in page view data to the fr-tech version of superset - https://phabricator.wikimedia.org/T252049 (DStrine) [18:22:20] (PS1) Cstone: Maintenance script to show the top message on the queue. [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/594775 [18:22:43] (CR) jerkins-bot: [V: -1] Maintenance script to show the top message on the queue. [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/594775 (owner: Cstone) [18:28:57] (PS2) Cstone: Maintenance script to show the top message on the queue. [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/594775 [19:07:17] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Figure out how to check regularly on screen scrape data coming in - https://phabricator.wikimedia.org/T251833 (CCogdill_WMF) This seems like a great idea! Monthly makes sense to me unless @EYener sees a reason to do it more frequently. [19:23:19] Fundraising-Backlog, fundraising-tech-ops: Superset user bug: password - https://phabricator.wikimedia.org/T252034 (Jgreen) Based on this https://github.com/apache/incubator-superset/issues/2731 I created a new role "ChangeOwnPassword" and added it to the roles of each of the Alpha users. Seems to work,... [20:02:30] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Figure out how to check regularly on screen scrape data coming in - https://phabricator.wikimedia.org/T251833 (EYener) That cadence works for me - thanks! [20:05:03] Fundraising-Backlog, fundraising-tech-ops: Superset user bug: password - https://phabricator.wikimedia.org/T252034 (EYener) @Jgreen what do you think of adding the new ChangeOwnPassword role to Beta and Gamma as well? I can see all new users needing to change their own passwords - it might be good to do... [20:06:37] (PS1) Ejegg: Fix NL email spelling error [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/594789 (https://phabricator.wikimedia.org/T251782) [20:08:13] (CR) Ejegg: [C: +2] Fix NL email spelling error [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/594789 (https://phabricator.wikimedia.org/T251782) (owner: Ejegg) [20:15:01] (Merged) jenkins-bot: Fix NL email spelling error [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/594789 (https://phabricator.wikimedia.org/T251782) (owner: Ejegg) [20:21:40] Fundraising-Backlog, fundraising-tech-ops: Investigate pulling in page view data to the fr-tech version of superset - https://phabricator.wikimedia.org/T252049 (EYener) @Jgreen pointed out that we should be able to connect fr-superset to Hive without pulling data into Fundraising from the data lake itsel... [20:28:57] (CR) Mepps: "recheck" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/510757 (https://phabricator.wikimedia.org/T215463) (owner: Ejegg) [20:31:33] (PS1) Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/594795 [20:31:53] (CR) Ejegg: [C: +2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/594795 (owner: Ejegg) [20:35:36] !log updated Fundraising CiviCRM from b15b2cfbb5 to cfb6101e39 [20:35:37] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:37:40] Fundraising Sprint Ice cream cures everything, Fundraising-Backlog, Patch-For-Review: Spelling error in Dutch TY email - https://phabricator.wikimedia.org/T251782 (Ejegg) Open→Resolved a:Ejegg All set. I deployed this directly to production since it was a one character change. [20:57:05] Fundraising Sprint Ice cream cures everything, Fundraising-Backlog, Patch-For-Review: Spelling error in Dutch TY email - https://phabricator.wikimedia.org/T251782 (CDenes_WMF) Thank you very much Elliott! [21:30:12] Fundraising-Backlog, MediaWiki-Vagrant: /var/www/w linking to /vagrant/mediawiki for all wikis regardless of MWMultiVersion set up - https://phabricator.wikimedia.org/T252075 (jgleeson)