[00:04:04] (PS3) Cstone: Update 2021 thank you emails. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/676681 [02:17:56] (PS2) Ejegg: composer: Relax monolog requirement to allow 2.x, per MW 1.36 [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/642028 (https://phabricator.wikimedia.org/T268207) (owner: Jforrester) [02:19:22] (CR) Ejegg: [C: +2] "Ack, and per MW 1.35.2!" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/642028 (https://phabricator.wikimedia.org/T268207) (owner: Jforrester) [02:20:01] (CR) Ejegg: [C: +2] "Thanks Jdforrester!" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/642028 (https://phabricator.wikimedia.org/T268207) (owner: Jforrester) [02:22:23] (Merged) jenkins-bot: composer: Relax monolog requirement to allow 2.x, per MW 1.36 [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/642028 (https://phabricator.wikimedia.org/T268207) (owner: Jforrester) [02:22:29] (CR) Jforrester: "> Patch Set 2:" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/642028 (https://phabricator.wikimedia.org/T268207) (owner: Jforrester) [02:41:00] ejegg: hey that deduper one - it wasn't that you needed to truncate civicrm_cache was it? [02:50:26] oh no I see [02:56:24] (PS3) Eileen: 5.37 rc upgrade [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/675616 [03:02:36] (PS4) Eileen: 5.37 rc upgrade [wikimedia/fundraising/crm/civicrm] - https://gerrit.wikimedia.org/r/675616 [03:04:29] hi eileen [03:04:34] hey [03:04:45] I realised what you were saying - use name not id [03:06:21] ah yeah [03:06:43] with the setting stored as you create it in the mgd file it would have worked [03:06:51] but the UI-setting wasn't working [03:06:59] oh, I meant to test with that setting empty as well [03:07:44] I think maybe the getSkipped fields should check if that setting is empty and just return [] without doing its own api call if so [03:10:10] yeah - I'm just having some issues with my site now - I blew it away & now it won't re-install - looking at https://www.drupal.org/project/phpconfig/issues/1826652 [03:10:39] but gonna run setup.sh instead [03:23:26] (CR) Jforrester: [C: +2] Remove php entry point [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/669152 (https://phabricator.wikimedia.org/T140850) (owner: Umherirrender) [03:25:25] aha, jgleeson was running into that same sendmail missing thing [03:26:24] nice fix there [03:26:48] so we could set that sendmail_path in the ini I guess [03:27:57] (Merged) jenkins-bot: Remove php entry point [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/669152 (https://phabricator.wikimedia.org/T140850) (owner: Umherirrender) [03:29:23] (CR) Eileen: [C: +2] Update civicrm image tag for image with new packages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/677939 (https://phabricator.wikimedia.org/T279669) (owner: AndyRussG) [03:30:42] (CR) Eileen: [V: +2 C: +2] Update civicrm image tag for image with new packages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/677939 (https://phabricator.wikimedia.org/T279669) (owner: AndyRussG) [03:31:30] (PS2) Eileen: Update civicrm image tag for image with new packages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/677939 (https://phabricator.wikimedia.org/T279669) (owner: AndyRussG) [03:31:39] (CR) Eileen: [V: +2 C: +2] Update civicrm image tag for image with new packages [wikimedia/fundraising/dev] - https://gerrit.wikimedia.org/r/677939 (https://phabricator.wikimedia.org/T279669) (owner: AndyRussG) [03:33:27] ejegg: yeah it has changed - I didn't have this problem before but it is consistent now [03:33:35] trying Andy's latest image [03:38:00] Fundraising-Backlog, FR-Docker, Patch-For-Review: fundraising-dev CiviCRM set up failures (with fixes and workarounds) - https://phabricator.wikimedia.org/T279669 (Eileenmcnaughton) Something has changed bcase that send mail is a bit constant for me now - testing Andy's latest The gotcha that keeps... [03:45:13] Fundraising-Backlog, FR-Docker, Patch-For-Review: fundraising-dev CiviCRM set up failures (with fixes and workarounds) - https://phabricator.wikimedia.org/T279669 (Eileenmcnaughton) Andy's image update only marginally changed the message Installation complete.... [03:49:27] eileen: ok, so witih sendmail available but maybe not fully configured it's got a different failure mode [03:49:44] yeah - just trying the apt get approach [03:51:20] hmm - no still broke [03:54:42] eileen ejegg hi! yeah with that latest image, I still get this: [03:54:44] Starting Drupal installation. This takes a while. Consider using the --notify global option. [ok] [03:54:46] Who are you? [03:54:48] Installation complete. [ok] [03:54:50] Unable to send e-mail. Contact the site administrator if the problem persists. [03:54:58] [error] [03:55:11] and then setup.sh exists and doesn't complete [03:55:32] AndyRussG: yeah - not sure when this changed [03:55:48] also we were wondering why it wants to send e-mail in the first place [03:55:52] This wasn't a blocker in the past [03:55:57] yeah - ! [03:56:03] eileen: yeah it was working for sure not long ago! [03:56:26] like less than a week ago, with the latest master of buildkig and wmff, it worked fine [03:56:46] though what I did differently this time, on Jack's suggestion, was to remove the persistent storage [03:56:54] (which I think I hadn't done in a while) [03:57:11] Yeah - I did that too - but I don't think it's that long since I last did it [03:57:24] ah hmmm [03:57:43] eileen: think this will stop those auto-update-checks? https://github.com/civicrm/civicrm-buildkit/pull/611 [03:58:16] eileen: I am around to dig in if you wish, though my brain is also kinda fried, and I was planning to work more on Saturday and Sunday (kids are going to their mom's early this week, for their uncle's birthday) [03:58:17] (just to make things faster, not gonna solve the sendmail bit) [03:58:49] I imagine you won't be around again until your Monday morning? or your Tuesday morning? [03:59:03] AndyRussG / eileen maybe easier to do the sendmail_path=/bin/true workaround? [03:59:25] since we don't actually want it sending any email during the setup? [03:59:47] I probably will be around a bit - but not for much longer tonight (multi tasking with baking now) [03:59:50] yeah I was thinking of something like that too [04:00:57] eileen: ah okok cool... I guess if you find something you could ping here or by e-mail, and if you're around on your Monday we might be able to sync up again at that time, if u like... [04:01:15] I'm still at the keyboard-ish for a little while this evening [04:05:11] AndyRussG: if we do the sendmail_path one - where would we put that? [04:06:39] eileen: not exactly sure how that works... is it an environment variable somewhere? [04:06:47] it's a PHP ini var [04:07:04] oh nice [04:07:06] but you can set it for one specific command-line [04:07:13] oh nicer [04:07:33] lessee, I think https://www.drupal.org/project/phpconfig/issues/1826652 had some suggestions [04:07:45] (dropped in IRC by eileen above) [04:08:51] welp we probably don't want to mess with the dmaster.sh script [04:09:08] ah right [04:09:16] we could make whatever ini file it goes in writable by any user (that bit would go in the image) [04:09:43] I guess /etc/php/7.3/cli/php.ini [04:09:45] and then edit it from the command line via sed before and after the setup steps [04:09:53] sure, that could work [04:10:21] either that, or, if we're fine with it always going to /bin/true, we could just set it as such in a static way in the image [04:10:38] or will that prevent the sending of ty e-mails? [04:10:41] looks like it exists commented out in the default ini file [04:10:42] ;sendmail_path = [04:11:02] AndyRussG: good question, lemme take a peep at that [04:11:19] the TY emails use phpmailer, not sendmail directly [04:11:47] so not sure if that'll affect phpmailer too [04:13:47] K lemme try at least to see if that fixes it when I make the change locally [04:14:08] since we don't have ty mail stuff set up in Docker yet, that could at least be a temporary solution [04:14:45] ok, sendmail_path is used by the php builtin mail() fn [04:15:04] and phpmailer can use that, smtp, or sendmail directly [04:15:18] lessee, I think we may be using the 'sendmail directly' option [04:17:33] another option is to somehow change sendmail settings so it can run as a non-user [04:18:38] hmm, nope, it looks like phpmailer defaults to mail() [04:18:46] and I don't see us changing that [04:19:41] but anyway, if sendmail is not working as installed, it's slightly better to not get an error for now [04:21:41] hmmm... [04:21:55] also now I've gotten some weird permission errors early on in setup.sh [04:23:31] just trying now with re-cloning wmff src [04:28:26] ejegg: eileen: k seems to be working now after adding that line to the ini file and re-cloning the source code! [04:28:37] nice! [04:29:15] yeah! [04:29:28] though I also do wonder what the expected fix is for ty mails then [04:30:14] staging? [04:30:16] :P [04:30:30] mmm dunno heheh [04:30:43] I think folks have found local e-mail send testing quite useful [04:31:03] yah, I guess especially with that mailcatcher [04:31:10] yeah [04:31:18] lemme search for a different sendmail workaround [04:31:26] so i wonder why apt install sendmail didn't create a user under docker [04:31:51] ah, I guess it doesn't normally [04:31:59] postfix does tho [04:32:00] ah also now I got bit by "Symfony\Component\Filesystem\Filesystem" [04:32:14] hmmm [04:32:14] oh jeez [04:33:50] hmmm ejegg can we just set the php config then to use postfix instead? [04:34:34] erm, i think maybe postfix is a different part of the mail system than sendmail? [04:34:58] I was just trying to remember which mail-related thing ended up with a /etc/passwd entry [04:35:36] hmm, no, I guess they are the same sort of component [04:35:49] a 'Mail Transfer Agent' [04:35:51] https://serverfault.com/questions/244263/is-postfix-the-same-thing-as-sendmail [04:36:40] oho, so postfix provides a sendmail-compatible thingy [04:36:47] okok, worth a shot [04:43:15] yeee [04:45:46] ejegg: welp apt installing postfix seems to be borking something with apache and causing the whole container to die [04:46:28] ejegg eileen I'm gonna sleep, brain is really fried now, but this is definitely progress! :) [04:46:35] hah, that's extreme AndyRussG [04:46:40] yah, I should sleep too [04:46:43] don't hesitate to write any updates here or elsewhere [04:46:45] see y'all later! [04:47:00] ejegg: nos vemos [07:08:01] (CR) jerkins-bot: [V: -1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/678171 (owner: L10n-bot) [13:30:25] (CR) Umherirrender: "This seems to revert parts of If28af01d2e5ff18880098fadfe80e2f6503ae790 and I046809aaa52131e6a1d8760c9963968a08b46a68" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/678171 (owner: L10n-bot) [13:32:28] (CR) Raimond Spekking: "> Patch Set 1:" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/678171 (owner: L10n-bot) [14:02:14] (Abandoned) Raimond Spekking: Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/678171 (owner: L10n-bot) [16:38:55] RECOVERY - check_ssl on frpig1001 is OK: SSL OK - Certificate payments-listener.wikimedia.org valid until 2022-05-11 14:21:07 +0000 (expires in 396 days) [16:40:51] RECOVERY - check_ssl on civi1001 is OK: SSL OK - Certificate civicrm.wikimedia.org valid until 2022-05-11 14:21:03 +0000 (expires in 396 days) [16:47:01] RECOVERY - check_ssl on frpig2001 is OK: SSL OK - Certificate payments-listener.wikimedia.org valid until 2022-05-11 14:21:07 +0000 (expires in 396 days) [16:47:45] RECOVERY - check_ssl on civi2001 is OK: SSL OK - Certificate civicrm.wikimedia.org valid until 2022-05-11 14:21:03 +0000 (expires in 396 days) [17:26:48] I got my water bottle! [17:42:10] nice ejegg|away