[00:00:38] Fundraising-Backlog: es419 Thank You Email Edits for the March 9th Pretest - https://phabricator.wikimedia.org/T274319 (CDenes_WMF) [00:13:17] (Merged) jenkins-bot: Add notes about ci-create-dbs [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/662812 (owner: Eileen) [00:18:29] Fundraising-Backlog, fundraising-tech-ops: Issue new SSL Client Certificate for nisrael - https://phabricator.wikimedia.org/T274320 (Dwisehaupt) [00:18:44] cstone that passed [00:47:46] so still failing - but I feel more confident the fail is meaningful now [01:00:26] (PS1) Eileen: Update data_strings to not specify engine [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663089 (https://phabricator.wikimedia.org/T273704) [01:14:33] eileen: i'm about to head out for the night. if there is any thing i need to work on tomorrow during my day to prep for the upgrade, just send me a note. otherwise, i'll see you tomorrow. [01:14:47] cool - thanks [01:14:58] have a good one. [01:38:34] (Abandoned) Eileen: Try removing utf8 declaration from pre-upgrade code [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663047 (https://phabricator.wikimedia.org/T273704) (owner: Eileen) [01:38:49] (Abandoned) Eileen: Remove charset declarations & reply on db default [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663045 (https://phabricator.wikimedia.org/T273704) (owner: Eileen) [01:51:05] (PS1) Eileen: A bit of a hack to get more info (do not merge) [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663094 [02:11:12] (CR) jerkins-bot: [V: -1] A bit of a hack to get more info (do not merge) [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663094 (owner: Eileen) [02:16:47] (PS2) Eileen: A bit of a hack to get more info (do not merge) [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663094 [02:19:20] cstone: can you +2 this one https://gerrit.wikimedia.org/r/663089 [02:20:26] (CR) Cstone: [C: +2] Update data_strings to not specify engine [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663089 (https://phabricator.wikimedia.org/T273704) (owner: Eileen) [02:20:31] thanks! [02:20:33] np [02:22:43] (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/663097 [02:23:39] eileen: Im half afk for the next 1.5 hours but then I can finish up the engage stuff [02:23:56] thanks - hopefully I'll have figured out this last test issue too [02:25:33] cstone: ohh I think I got it [02:25:39] !! [02:27:28] (PS2) Eileen: Update data_strings to not specify engine [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663089 (https://phabricator.wikimedia.org/T273704) [02:27:30] (PS1) Eileen: Aha - it's unicode not general in the collate [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663098 [02:33:21] (CR) jerkins-bot: [V: -1] A bit of a hack to get more info (do not merge) [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663094 (owner: Eileen) [02:38:55] (PS2) Eileen: Aha - it's unicode not general in the collate [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663098 (https://phabricator.wikimedia.org/T273704) [02:43:31] Fundraising Sprint Corrugated super slide, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Deal with utf8 issues for new install - https://phabricator.wikimedia.org/T273704 (Eileenmcnaughton) @Dwisehaupt working through this on our CI enviroment we learnt 2 things 1) the fo... [03:23:46] cstone: if you can review https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/663098 when you are back I think that's it [03:23:53] eileen: hah i was just going to ask i have a short break [03:24:03] lol [03:24:07] (CR) Cstone: [C: +2] Aha - it's unicode not general in the collate [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663098 (https://phabricator.wikimedia.org/T273704) (owner: Eileen) [03:24:19] I'm heading out soon (to get a passport photo so I can go .... nowhere) [03:25:57] but good to be prepared to go nowhere [03:41:46] (Merged) jenkins-bot: Aha - it's unicode not general in the collate [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663098 (https://phabricator.wikimedia.org/T273704) (owner: Eileen) [03:45:51] (CR) Eileen: "recheck" [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/660041 (https://phabricator.wikimedia.org/T272061) (owner: Eileen) [03:46:00] :-) [03:46:32] I have to apply in Auckland - for my AU passport (I have 3) - & my parents are going down so I figured now is the time [03:48:39] how long of a drive is that for you? [04:04:58] woo woo it passed!! [05:22:25] (CR) Cstone: [C: +2] Add Partner as a field on Individual Engage import [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/660988 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [05:33:44] (CR) Cstone: [C: +2] Add extra organization custom fields to reflect production [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/661277 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [05:39:37] (Merged) jenkins-bot: Add Partner as a field on Individual Engage import [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/660988 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [05:46:56] (CR) Cstone: [C: +2] Add support for extra organization fields to be imported [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/661279 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [05:50:06] (Merged) jenkins-bot: Add extra organization custom fields to reflect production [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/661277 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [06:03:19] (Merged) jenkins-bot: Add support for extra organization fields to be imported [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/661279 (https://phabricator.wikimedia.org/T273011) (owner: Eileen) [07:22:36] (Abandoned) Eileen: A bit of a hack to get more info (do not merge) [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663094 (owner: Eileen) [09:06:25] Fundraising-Backlog, FR-Docker: Fix Docker performance on Mac - https://phabricator.wikimedia.org/T274339 (AndyRussG) [09:29:41] Fundraising-Backlog, FR-Docker: Investigate requirements and setup options for self-contained, quick-and-simple-to-run FR applications on Docker - https://phabricator.wikimedia.org/T274341 (AndyRussG) [09:49:44] (PS7) AndyRussG: WIP Initial Civi setup [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/644884 (https://phabricator.wikimedia.org/T262975) [09:52:20] (PS8) AndyRussG: WIP Initial Civi setup [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/644884 (https://phabricator.wikimedia.org/T262975) [12:26:02] Fundraising-Backlog, MediaViewer, MediaWiki-extensions-CentralNotice, Structured Data Engineering, and 3 others: CentralNotice throws exception on malformed URIs - https://phabricator.wikimedia.org/T269948 (SarthakKundra) Hi @Jdlrobson. I can't find the steps to setup the project locally on Githu... [13:24:34] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) [13:28:37] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) It's possible that this could be related to the invoice_id bug. At the moment we can't see the exception attempted to be thrown but once we do it might well be due to the invoice_id problem. [14:02:43] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) @Jgreen just mentioned that the Nagios warning is triggered when the result of the query: `SELECT count(*) FROM civicrm.civicrm_contribution_recur WHERE contribution_status_id = 14;` is >=... [14:15:34] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) I'm guessing that failing recurring transactions are getting stuck. [15:00:22] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) So those five recurring transactions on live stuck at 'Processing' that triggered the Nagios alert don't really have any obvious connection apart from most of them being via Ingenico and thr... [15:09:03] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (jgleeson) [15:29:12] Fundraising-Backlog: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (Ejegg) Ah yeah, in our first version of the SmashPig code the Approve response inherited from the Create response. We changed it so both now inherit from the PaymentProcessorResponse, and that class d... [15:29:21] jgleeson are you doing that one-line fix? ^^^ [15:29:36] if not I'll just push it up [15:29:49] I was just getting unit tests to run ejegg back over on vagrant [15:30:01] to test! [15:30:02] :( [15:33:42] oh fun. [15:33:52] ok they're working now [15:34:16] cool cool. I'll make sure I've got em running locally too [15:37:52] yeah that change breaks two tests for me [15:40:29] (PS1) Jgleeson: Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663225 (https://phabricator.wikimedia.org/T274361) [15:40:45] let's see if it's really breaking them or it's just my environment data [15:41:55] ejegg: I've gotta go start cooking now but if that patch passes on CI it should be +2'able. back later [15:56:04] (PS1) Ejegg: Allow longer order IDs for Ingenico [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/663229 (https://phabricator.wikimedia.org/T274301) [15:56:43] cstone: that truncation you caught might be a simple fix ^^^ [15:57:27] did it happen to be just the ingenico ones? [15:57:47] thanks jgleeson! [16:33:03] fr-tech it's scrum of scrums time - is anyone planning to go, or should I hop on? [16:33:06] and any news? [16:33:27] No news that I can think of. [16:56:33] fr-tech is anyone interested in going to the imagination session that starts at the same time as our daily standup? [16:56:42] maybe we could push the standup back an hour [16:57:07] Moving the standup is totally find if folks are interested. [16:57:45] I'll go ahead and move it. [16:58:38] I was kind of interested in lightning talks that were in that hour but def less important [16:59:28] thanks XenoRyet [16:59:39] cstone: ooh but always fun [17:00:14] hopefully itll be recorded [17:04:02] (CR) Ejegg: [C: +2] Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663225 (https://phabricator.wikimedia.org/T274361) (owner: Jgleeson) [17:04:41] Fundraising Sprint Corrugated super slide, Fundraising-Backlog, Patch-For-Review: SmashPig Charge Recurring Donations failed - https://phabricator.wikimedia.org/T274361 (Ejegg) p:Triage→High a:jgleeson [17:09:56] recurring processor nooo but it looks like not too many go stuck? [17:16:36] Fundraising-Backlog, fundraising-tech-ops: Issue new SSL Client Certificate for rkhumar - https://phabricator.wikimedia.org/T273130 (Dwisehaupt) Certificate renewed and delivered via email. Password sent via SMS. [17:17:07] Fundraising-Backlog, fundraising-tech-ops: Issue new SSL Client Certificate for rkumar - https://phabricator.wikimedia.org/T273130 (Dwisehaupt) [17:23:12] (Merged) jenkins-bot: Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/663225 (https://phabricator.wikimedia.org/T274361) (owner: Jgleeson) [17:43:47] Fundraising-Backlog, fundraising-tech-ops: Issue new SSL Client Certificate for rstearns - https://phabricator.wikimedia.org/T274162 (Dwisehaupt) Open→Resolved [17:48:56] only 5 cstone [17:49:30] I jotted down my findings here https://phabricator.wikimedia.org/T274361 [17:53:06] nice jgleeson [17:53:19] sorry, only two from today I think! [17:53:23] five in total [17:55:51] hmm interesting that it broke in a way that didnt prevent the job from continuing on [17:58:03] cstone: I think it did break for that run [17:58:13] but the next run picked up from the following contribution [17:58:27] since the one it broke on was left in 'currently charging' status [17:58:35] or whatever we call status number 14 [18:05:17] fr-tech standup? [18:26:10] fr-tech I'm not gonna make tech-talk [18:26:50] jgleeson stew! probably more enjoyable eh [18:27:21] jgleeson if you like we can sync up afterwards, if you'll still be working, and if that works for you :) [18:30:17] i'm going to ack the check_recurring_contrib_processing warning since it is being worked. [18:32:12] thanks dwisehaupt [18:33:30] hmm, the 'edit smart group criteria' screen is hanging the heck out of my browser [18:33:44] let's see if chrome handles it better [18:56:54] Fundraising-Backlog, FR-Civi-Prospect, Fr-endowment: Need a very special group created - https://phabricator.wikimedia.org/T271333 (Ejegg) Looks like @LeanneS has created one that should work (group ID 320) - is there any more work to do on this ticket? [19:13:04] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [19:43:39] fr-tech I'm gonna try and deploy that fix to smashpig that got pushed up today if it's not already done an then maybe ejegg we could reset those transactions in the console like you said? [19:45:17] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [19:46:03] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [19:53:37] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [19:54:57] (PS1) Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663290 [19:55:31] cstone: eileen there are tons of changes between master & deployment on civicrm according to that patch https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/663290 [19:55:37] I'm guessing we're holding off on some? [19:57:48] ok I'm just gonna cherry pick that smashpig ext fix and push that out in the chance there's some stuff there that can't go until eileen or cstone can confirm [19:57:58] AndyRussG: want to push up a WIP for the dev-images patch so I can work with the latest to get that text on payments? [19:58:11] jgleeson: cherry-pick sounds good for this deploy! [19:59:07] ejegg: sure, though what we were just looking at is likely partly broken still [19:59:11] (PS1) Jgleeson: Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663292 (https://phabricator.wikimedia.org/T274361) [19:59:46] ejegg if you want something currently working I'd recommend the last patch sets linked in the last EOD e-mail I sent? [20:00:28] that's probably a better starting point for working on elements not super related to the Docker aspects [20:00:47] I'll still push up what I have so far for the separation of Payments and Civi image config we were just looking at [20:01:37] ooohhh the images did build! [20:03:18] (CR) Jgleeson: [C: +2] Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663292 (https://phabricator.wikimedia.org/T274361) (owner: Jgleeson) [20:09:07] cstone ejegg jgleeson fr-tech so here is the WIP dev-images patch with initial work to separate out Civi and Payments image setups... again, doesn't work with current tip of fundraising-dev changes, for something more working-ish see EOD e-mail (i.e., use latest tip of fundraising-dev changes with the previous dev-images patch, that is https://gerrit.wikimedia.org/r/c/releng/dev-images/+/663159 [20:09:08] ) [20:14:46] (Abandoned) Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663290 (owner: Jgleeson) [20:15:02] (Abandoned) Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/650490 (owner: Jgleeson) [20:15:33] (CR) Jgleeson: [V: +2 C: +2] Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663292 (https://phabricator.wikimedia.org/T274361) (owner: Jgleeson) [20:19:45] !log updated civicrm from 1e9a86dd6e to f161a34266 [20:19:48] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:21:18] (PS1) Jgleeson: Revert "Dockerize process-control" [wikimedia/fundraising/process-control] - https://gerrit.wikimedia.org/r/663077 [20:22:18] oh ejegg cstone jgleeson fr-tech ahh also just noting for some reason it seemed tests with payments broke again in a big way under fundraising-dev Docker [20:22:51] (not sure if it's still the case though) [20:23:09] jgleeson what were the bugs in the process-control patch? [20:23:28] (CR) Jgleeson: [C: +2] "reverting this as it probably won't be shipped with the project and was merged with bugs. thanks for all the review @awight and finding th" [wikimedia/fundraising/process-control] - https://gerrit.wikimedia.org/r/663077 (owner: Jgleeson) [20:23:59] (Merged) jenkins-bot: Revert "Dockerize process-control" [wikimedia/fundraising/process-control] - https://gerrit.wikimedia.org/r/663077 (owner: Jgleeson) [20:25:47] AndyRussG: apparently when failmail was triggered it threw errors for awight and there a kill-job command I hadn't tested from the looks of it [20:31:18] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [20:34:19] (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/663097 (owner: Eileen) [20:35:36] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Dwisehaupt) Just reworked slander to run within firejail. ` [frack::puppet] d0e31bc9 Move slander to using firejail and new log location [frack::puppet] 2dbe000f Shift... [20:35:50] fundraising-tech-ops: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Dwisehaupt) [20:35:56] cstone: so in terms of the upgrade today.... the civi stock patch is passing - YAY [20:35:56] - that leaves 2 patches we had 1) the UI fix on the recurrings & 2) the soft credit form hack [20:35:56] - I can probably move the second to an extension. on the first - I'm not sure if I just want to go easy &port it or finally achieve stock civi [20:36:35] But I def need stock civi patch merged [20:37:32] !log civicrm revision changed from f161a34266 to ae24f87158, config revision is a48a7db0a2 [20:37:35] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [20:41:28] jgleeson: aww hmmm! well good it was possible for awight to test it like that then! :) [20:52:14] AndyRussG: so how long is the image delay with releng ? [20:53:03] (Abandoned) Eileen: Create bash script to run OmnimailJobProgress.check [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/641236 (https://phabricator.wikimedia.org/T222287) (owner: Mepps) [20:56:38] eileen: in what sense? [20:57:11] eileen: for them to put them in the registry? not sure, here's the task for the error they're experiencing [20:58:25] https://phabricator.wikimedia.org/T274306 [20:59:05] eileen: as regards merging our stuff, not sure, but for the most recent things to work, there are new unreviewed and unfinished Gerrit patches in releng's dev-images repo [21:00:07] hmm - yeah -that seems like it's not #1 priority for them [21:01:59] eileen: mmm well also the latest one I marked WIP, I don't think it's ready to ask them for review yet [21:02:16] I was thinking of the image publishing bug [21:02:17] eileen: though if you want a working-ish stack look at the one before the WIP one in dev-images [21:02:30] ahhhh right yeah no idea about that one [21:02:38] but this one is against our repo? https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/644884 [21:03:07] eileen: ah yeah that one works [21:03:45] eileen: and this one, in dev-images, doesn't: https://gerrit.wikimedia.org/r/c/releng/dev-images/+/663294 [21:04:00] could well be that it's different releng folks who look at CI vs dev-images also [21:04:03] ? [21:20:01] Ok - I'll have a go at fixing the last civi things & then see if I can figure out how to fire up your latest [21:22:15] AndyRussG: looks like that changelog issue was indeed the three spaces before the date [21:25:02] fundraising-tech-ops, Patch-For-Review: Reconfigure fundraising public-reporting server role to fundraising-data - https://phabricator.wikimedia.org/T255435 (Jgreen) [21:37:21] ejegg: ahhh creisi heh thx! [21:37:31] 3 spaces ftw then! [21:46:14] (PS1) Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment + 31fdb8627cd6f2ee435d9f2a6078d8876a4262bd Add Partner as a field on Individual Engage import + 633df53720573fd48d028d18c7cc34afb6ec6582 Add extra organization custom fields to reflect production + c6a0f601d342639c8e2b1faacf44d55baf95e5b6 Add support for extra organization fields to be imported + ab4be4a2abd46afd08dd [21:46:14] a247f9b2bc58f0f5158cdef6064e410a86c93a53 Aha - it's unicode not general in the collate - b2af975549dd37ec410fc3db1ca50a82796f4822 Make throwException() accept PaymentProviderResponse [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663308 [21:58:05] hey eileen so +2 on the civi stock then? [21:58:16] cstone: let's do it [21:58:51] (CR) Cstone: [C: +2] "woohoo" [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/660041 (https://phabricator.wikimedia.org/T272061) (owner: Eileen) [22:03:14] fr-tech another docker-y activity might also be to get review and mebbe +2 on the other outstanding fundraising-dev patches, before the WIP civi one, btw.... ;) [22:03:40] cstone: you have your head into which contribution recurs have the wrong status better than I do - should we do another db update or just keep the patch for now & drop it next time? [22:05:33] so the main thing is theres a ton of paypal ones at 1 like 200k [22:06:30] (completed) [22:06:52] cstone: so you lean towards re-patching for now & dropping next time? [22:07:01] yeah just in case there is some paypal weirdness [22:08:15] also to see how 2 operates on live [22:08:20] (pending) [22:14:07] (PS1) Eileen: Re-add wmf hacks: [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663315 (https://phabricator.wikimedia.org/T272061) [22:16:16] cstone: ok it's ^^ - I also left in the soft credit patch since I haven't sorted that yet - this at least gives time to sort that [22:16:24] sounds good eileen [22:19:02] (CR) Eileen: [C: +2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment + 31fdb8627cd6f2ee435d9f2a6078d8876a4262bd [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663308 (owner: Eileen) [22:19:20] Fundraising Sprint Corrugated super slide, Fundraising Sprint Upbeat Upholstery, Fundraising Sprint Vagranty McVagrantface, Fundraising Sprint We all meet again, and 2 others: Set up text that goes next to payments forms on Docker - https://phabricator.wikimedia.org/T266092 (Ejegg) [22:19:32] Fundraising Sprint Corrugated super slide, Fundraising Sprint Upbeat Upholstery, Fundraising Sprint Vagranty McVagrantface, Fundraising Sprint We all meet again, and 2 others: Set up text that goes next to payments forms on Docker - https://phabricator.wikimedia.org/T266092 (Ejegg) a:AndyRussG... [22:20:01] (PS4) Eileen: Remove campaign_id pseudoconstants [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/660073 [22:20:02] eileen: did you want this one in it too? https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/civicrm/+/660073/3 [22:20:10] oh haha you just updated it [22:20:16] :-) [22:20:28] (Merged) jenkins-bot: 5.35-stock [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/660041 (https://phabricator.wikimedia.org/T272061) (owner: Eileen) [22:20:36] yeah - that change was made because when I was testing search kit it was slow in some places [22:20:48] so we might as well patch it now (it's merged upstream) [22:24:43] Fundraising-Backlog, fundraising-tech-ops: Update mariadb GLOBAL defaults for frdbs in preparation for utf8mb4 conversion - https://phabricator.wikimedia.org/T274438 (Dwisehaupt) [22:25:17] Fundraising-Backlog, fundraising-tech-ops: Update mariadb GLOBAL defaults for frdbs in preparation for utf8mb4 conversion - https://phabricator.wikimedia.org/T274438 (Dwisehaupt) [22:25:19] Fundraising-Backlog, fundraising-tech-ops: Fix our database to support emojis - convert tables to utf8mb4 - https://phabricator.wikimedia.org/T270218 (Dwisehaupt) [22:27:39] Fundraising Sprint Corrugated super slide, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Deal with utf8 issues for new install - https://phabricator.wikimedia.org/T273704 (Dwisehaupt) The collation setting has been adjusted on the civicrm and dev_civicrm databases. Innodb... [22:39:20] (CR) jerkins-bot: [V: -1] Remove campaign_id pseudoconstants [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/660073 (owner: Eileen) [22:40:30] looks like I need to dig into ^^ - it's not required for today [22:41:13] Fundraising-Backlog, fundraising-tech-ops: Update mariadb GLOBAL defaults for frdbs in preparation for utf8mb4 conversion - https://phabricator.wikimedia.org/T274438 (Dwisehaupt) https://mariadb.com/kb/en/innodb-system-variables/#innodb_large_prefix Quite a history there. we are on 10.1 on the primary b... [22:43:22] (PS1) Ejegg: WIP text for payments-wiki pages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/663317 (https://phabricator.wikimedia.org/T266092) [22:43:36] (CR) Cstone: [C: +2] Re-add wmf hacks: [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663315 (https://phabricator.wikimedia.org/T272061) (owner: Eileen) [22:53:19] Fundraising-Backlog, fundraising-tech-ops: Update mariadb GLOBAL defaults for frdbs in preparation for utf8mb4 conversion - https://phabricator.wikimedia.org/T274438 (Dwisehaupt) Looking at the three additional variables, it looks like we may be able to avoid this when we upgrade the current origin serve... [22:53:56] (PS2) Ejegg: WIP text for payments-wiki pages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/663317 (https://phabricator.wikimedia.org/T266092) [22:54:21] AndyRussG: ^^^ seems to get the text in to the payments-wiki [22:55:11] I'm just redirecting the wikitext files into the docker-compose command's stdin, so they don't actually need to be accessible inside the machine [22:55:25] in case there's a non-config place that would be more appropriate [23:10:15] (Merged) jenkins-bot: Re-add wmf hacks: [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/663315 (https://phabricator.wikimedia.org/T272061) (owner: Eileen) [23:30:15] (CR) Eileen: [C: +2] "recheck" [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/663308 (owner: Eileen) [23:36:25] !log civicrm revision changed from ae24f87158 to 2ce8194c07, config revision is a48a7db0a2 [23:36:28] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log