[00:02:26] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Epic: improve MG mailing process - https://phabricator.wikimedia.org/T179882#4012425 (Eileenmcnaughton) [00:02:28] Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Epic: Investigate getting output to word option to work bett... - https://phabricator.wikimedia.org/T187251#4012424 [00:03:16] oh hey, OSM nominatum seems to be the thing coding for me locally [00:03:42] we'd want to disable that on prod too, tho... [00:03:50] ah, and the zip db too [00:04:29] I mean, locally both are geocoding for me [00:07:22] eesh, this db-based autoloader.... [00:07:41] no way to tell drupal7 to use apc or something? [00:08:33] ooh: https://www.drupal.org/project/xautoload [00:13:34] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Spike: would drupal's x-autoload help our civi performance? - https://phabricator.wikimedia.org/T188573#4012431 (Ejegg) [00:19:31] ejegg: the wmf install script should alter [00:20:12] which geocoders are enabled [00:20:17] eileen: oops, right, I messed with that while trying to diagnose the issue [00:20:20] lemme fix that [00:20:55] re the d7 - yeah - I've seen that autoloader stuff in debugging - *should* only kick in when drupal is finding? [00:26:52] eileen: the d7 autoloader is still doing a ton of db queries on every page request though, right? [00:27:46] I guess ? [00:28:00] I think it must cache them tho [00:36:10] eileen: well, the geocoder stuff looks good as far as I can see [00:36:43] ejegg: cool - I'll try deploying then [00:36:46] modulo some really trivial stuff like quoting the 0 in the zip str_pad [00:36:54] rockin! [00:37:09] yeah, just will definitely be interesting to see the performance [00:37:21] so, what do we do - stop the q a bit first [00:37:25] yeah, let's do that [00:37:31] & get a metric [00:37:57] yeah, I'm not sure we've gotten any big spikes since we did the civi version bump [00:44:37] ejegg: ok - are you doing that - tbh I never have done it [00:46:13] (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/415490 [00:54:28] (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/415490 (owner: Eileen) [00:55:37] (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/415490 (owner: Eileen) [00:58:51] ejegg: I mean I've never done the process of stopping the q & getting a metric [01:26:01] eileen: oh shoot, sorry! [01:26:36] well, let me see if I can find something from the past week [01:26:44] during some SE high times [01:27:56] ejegg: I haven't deployed yet - only merged [01:29:23] just thinking it's pretty slow times right now, so we would have to wait a while [01:30:27] ejegg: ok - should I deploy& we can just track agains that later? [01:30:34] eileen: sure, go ahead [01:32:00] that geocoder library has a caching plugin which is interesting [01:32:47] !log update civicrm revision changed from 341c734a79 to a819d64d98, config revision is 62631813fc (add geocoder extension) [01:32:59] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [01:33:17] oh right, that stuff's all in prometheus now! [01:35:02] hmm - well I deployed & moved to 90210 & got coded [01:35:16] felt sluggish but that is prob not a good metric [01:35:54] didn't feel sluggish re-locating to NZ :-) [01:37:30] heh, interesting [01:37:34] hmm it's not working in advanced search :-( [01:37:46] not finding co-ords of 90210 [01:37:55] after relocating? [01:38:05] oh, you mean it's not even geocoding the search input [01:38:09] yep [01:38:19] phooey [01:38:38] prox search IS I now have the number of contacts within 2 miles of 90210 [01:39:23] hmm adv search slower this time [01:39:48] so the thing is adv search couldn't find it with no country & then I added country & it still didn't [01:39:59] but now trying country the first time is working [01:41:57] oh weird [01:42:44] I'm gonna have to pop off, but let me know what you find tonight and I can pick that back up tomorrow. [01:43:13] thanks, and see ya later! [01:47:16] Fundraising Sprint Bermuda Rhombus (where things disappear then reappear), Fundraising Sprint Cottage Cheese isn't Made of Cottages, Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never inte... - https://phabricator.wikimedia.org/T185047#4012681 [02:17:45] (PS1) Eileen: Attempt to track file_get_contents [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/415499 [02:17:47] (PS1) Eileen: [not for review or deployment] Attempt to find where bad calls are happening [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/415500 [02:50:53] Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-CiviCRM-dedupe-FY2017/18: Spike: Can the engage import dedupe on import? - https://phabricator.wikimedia.org/T181090#4012722 (Eileenmcnaughton) SO I looked at the s/sheet & analysed... [02:53:11] (PS1) Eileen: Remove accidentally committed test files [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/415502 [04:26:19] Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-CiviCRM-dedupe-FY2017/18: Spike: Can the engage import dedupe on import? - https://phabricator.wikimedia.org/T181090#4012785 (Eileenmcnaughton) So what I'll do here is a first itera... [09:41:09] Wikimedia-Fundraising: Create translated versions of new Thank You page - https://phabricator.wikimedia.org/T185333#4013146 (jrobell) Thank you very much Robin, Peter and all. I have reviewed and made some tweaks to the Swedish version. We shall just remove the endowment info, then the svSE is good to go! [13:07:06] Wikimedia-Fundraising: Create translated versions of new Thank You page - https://phabricator.wikimedia.org/T185333#4013742 (schoenbaechler) Thanks for checking these @Pcoombe and your updates to the templates. Especially like how you got rid of the share message redundancy! Excited to the see the pages in a... [16:17:24] ccccccfhrkhfndjttriecjerbcgejejchtlrnvhiterc [16:17:45] derp, that's like twice in a day [16:17:49] better burn that one.... [16:18:28] easily done [16:18:57] heh, yep, just gotta use one successfully [16:28:41] ejegg so i think to test the paypal refund script, i have to deploy the code to prod [16:29:16] mepps I just deployed DI last night, which I think included your paypal script [16:50:12] learning so much that needs fixing through testing...sigh [16:50:31] mepps ah well, nobody got hurt! [16:56:24] Fundraising Sprint Cottage Cheese isn't Made of Cottages, Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Patch-For-Review: Unintended recu... - https://phabricator.wikimedia.org/T186883#4014474 [16:58:29] Fundraising Sprint Cottage Cheese isn't Made of Cottages, Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Patch-For-Review: Unintended recu... - https://phabricator.wikimedia.org/T186883#4014486 [17:00:47] Fundraising Sprint Cottage Cheese isn't Made of Cottages, Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Patch-For-Review: Unintended recu... - https://phabricator.wikimedia.org/T186883#4014493 [17:25:31] (CR) Ejegg: [C: 2] Remove accidentally committed test files [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/415502 (owner: Eileen) [17:26:30] (Merged) jenkins-bot: Remove accidentally committed test files [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/415502 (owner: Eileen) [17:39:45] Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Fr-Ingenico-integration_2017-18, Patch-For-Review: Ingenico adapter: parse response errors - https://phabricator.wikimedia.org/T176502#4014624 (jgleeson) Ingenico credit card test cases in excel format: https://docs... [18:35:22] Wikimedia-Fundraising: Create translated versions of new Thank You page - https://phabricator.wikimedia.org/T185333#4015078 (Pcoombe) New [Swedish](https://wikimediafoundation.org/wiki/Thank_You/sv) and [German](https://wikimediafoundation.org/wiki/Thank_You/de) pages are now live. Waiting on final proofread... [18:36:39] MBeat: you might be interested in the above, we just updated the Swedish and German thank you pages. Italian should change soon as well, getting some final proofreading [18:37:18] ty pcoombe [18:49:43] Wikimedia-Fundraising: Create translated versions of new Thank You page - https://phabricator.wikimedia.org/T185333#4015137 (Pcoombe) And [Italian](https://wikimediafoundation.org/wiki/Thank_You/it) is now live. [18:56:18] fundraising-tech-ops, Operations, ops-eqiad, Patch-For-Review: Rack/setup frmon1001 - https://phabricator.wikimedia.org/T186073#4015171 (Cmjohnson) management password changed [18:56:31] fundraising-tech-ops, Operations, ops-eqiad: rack frpig1001 - https://phabricator.wikimedia.org/T187365#4015172 (Cmjohnson) management password changed [18:56:53] fundraising-tech-ops, Operations, ops-eqiad: rack frbast1001 - https://phabricator.wikimedia.org/T187363#4015173 (Cmjohnson) management password changed [20:18:59] Fundraising Sprint Dinosaur Cookies co-existed with Gingerbread People, Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog: fill out PCI SAQ A - https://phabricator.wikimedia.org/T187246#4015458 (XenoRyet) Open>Resolved [21:15:07] fundraising-tech-ops: adopt apt.wikimedia.org on frack - https://phabricator.wikimedia.org/T188668#4015659 (cwdent) [21:24:47] Fundraising-Backlog, FR-PayPal-ExpressCheckout, MediaWiki-extensions-DonationInterface: Require PayPal account for EC donations - https://phabricator.wikimedia.org/T188671#4015712 (Ejegg) [21:34:22] (PS1) Ejegg: Send SOLUTIONTYPE=Mark with SetExpressCheckout calls [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415692 (https://phabricator.wikimedia.org/T188671) [21:52:51] (PS1) Mepps: Fixes after testing with data [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) [21:52:59] fr-tech still wip ^^ [21:53:45] actually it might be done--the refunds are failing but the order_ids are from prod and i'm testing on sandbox [21:54:49] looking [21:55:24] mepps so far so good modulo whitespace inconsistency [21:56:02] (PS2) Mepps: Fixes after testing with data [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) [21:58:12] updated whitespace [21:59:00] (PS3) Mepps: Fixes after testing with data [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) [21:59:38] mepps: I'm about to grab a bite to eat, but in the meantime I updated the file on frdev to include amounts. [21:59:52] awesome XenoRyet [22:12:46] (CR) Ejegg: [C: -1] "Looks pretty good, just one problem with putting _ec on the payment_method" (3 comments) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) (owner: Mepps) [22:14:36] oh hey, that requireExtension patch from January is a bit wrong [22:14:45] should be DonationInterface, not Donation Interface [22:16:34] ooh, and more importantly, requireExtension doesn't exist on the 1.27 version of mediawiki [22:26:18] (PS1) Ejegg: Make requireExtension calls conditional [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415757 (https://phabricator.wikimedia.org/T152139) [22:26:31] fr-tech pretty trivial thing ^^^ [22:26:56] but needed to get our maint scripts running again on 1.27 [22:28:35] Fundraising Sprint Elevators were never intended to go down, Fundraising-Backlog, Fr-Ingenico-integration_2017-18: Update Ingenico hosted style to match new payments form CSS - https://phabricator.wikimedia.org/T184288#3878650 (Ejegg) a:Ejegg [22:40:16] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-Smashpig: Create Civi extension to use SmashPig as a payment processor - https://phabricator.wikimedia.org/T188678#4015933 (Ejegg) [22:40:32] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-Smashpig: Create Civi extension to use SmashPig as a payment processor - https://phabricator.wikimedia.org/T188678#4015943 (Ejegg) p:Triage>Low [22:49:43] (CR) XenoRyet: [C: 2] Make requireExtension calls conditional [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415757 (https://phabricator.wikimedia.org/T152139) (owner: Ejegg) [22:51:59] (Merged) jenkins-bot: Make requireExtension calls conditional [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415757 (https://phabricator.wikimedia.org/T152139) (owner: Ejegg) [23:25:17] thanks XenoRyet [23:26:28] mepps: are you still working on the fixes patch? If you're done for the day but want it to go out before friday, I can make the tweak about method and ask XenoRyet to review [23:32:24] (PS4) Ejegg: Fixes after testing with data [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) (owner: Mepps) [23:32:39] Just manually rebased it around the requireExtension fix ^^ [23:59:15] (PS5) Ejegg: Fixes after testing with data [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/415752 (https://phabricator.wikimedia.org/T186883) (owner: Mepps) [23:59:32] XenoRyet: are you still hoping to get that up on prod before friday freezeday? [23:59:52] I just made the one change I thought the patch needed, in case you want to review + deploy