Update from_scratch.md (LEMMY_UI variables, and added note about failing postgresql config) #221

mtekman · 2023-06-19T08:22:11Z

Updated out of date LEMMY_UI environment variables, and added caution with failing postgresql configuration

Nutomic · 2023-06-20T13:11:07Z

src/en/administration/from_scratch.md

-Note that currently Lemmy only supports non-SSL connections to databases. More info [here](https://github.com/LemmyNet/lemmy/issues/3007).
+Note:  
+* Lemmy currently only supports non-SSL connections to databases. More info [here](https://github.com/LemmyNet/lemmy/issues/3007).
+* Your postgres config might need to be edited to allow password authentication instead of peer authentication. More info [here](https://github.com/LemmyNet/lemmy-docs/issues/220).


Linking to an issue is not great in this case. Better link to an external page which explains this, so that the issue can be closed.

Agreed.... but the issue page gives an instant fix and solution to people who are not familiar with postgres. It took me some time too to understand from the various different search queries of "Failed Authentication postgres" what exactly was required to get things running.

Do you have a good reference with an example solution we can link directly to? All I'm seeing is various stackoverflow links

I added the fix inline and removed the link to the issue.
I've also just seen that there is a Troubleshooting page, but it looks like it's more docker specific. Should I add the postgres note there instead?

Nutomic · 2023-06-20T13:11:57Z

src/en/administration/from_scratch.md

@@ -118,7 +120,7 @@ yarn build:prod
 exit
 ```

-Add another systemd unit file, this time for lemmy-ui. You need to replace example.com with your actual domain. Put the file in `/etc/systemd/system/lemmy-ui.service`, then run `systemctl enable lemmy-ui` and `systemctl start lemmy-ui`.
+Add another systemd unit file, this time for lemmy-ui. You need to replace example.com with your actual domain. Put the file in `/etc/systemd/system/lemmy-ui.service`, then run `systemctl enable lemmy-ui` and `systemctl start lemmy-ui`. Note that some of the environment variables `LEMMY_*` are subject to change, and you should use the most up to date variable names, as [found here](https://github.com/LemmyNet/lemmy-ui#configuration)


I dont think there are any plans to change the env vars again, so remove this line.

Ah good. I can get rid of the "Note that the..." part, but should I still keep the link to the config page?

I kept the config link, but removed the part where I said the envs might change. I also added a rust log.

Nutomic · 2023-06-21T10:10:57Z

Looks good like this, but you need to format the file with prettier.

mtekman · 2023-06-21T10:24:36Z

prettier -c from_scratch.md
Checking formatting...
All matched files use Prettier code style!

Not sure what I'm doing wrong...

mtekman · 2023-06-21T10:29:49Z

One other issue that is slightly outside the scope of this PR, is how does the update of pictrs fit?

When we do cargo install lemmy_server --features embed_pictrs, I see no pictrs binary appear in my installation target (~/.cargo/bin/), so I assume that pictrs is embedded within lemmy_server.

But later in the document we talk about building in /var/lib/pictrs and copying over to /usr/bin/.

I guess I'm not really clear on what the --features embed_pictrs flag does.

Related: MichaIng/DietPi#6434 (comment)

Nutomic · 2023-06-21T10:44:16Z

The CI has a problem where it sometimes runs on an old version of the code, thats probably why its failing here.

Yes with --features embed_pictrs it is embedded inside the Lemmy binary, running on a different port. Its worth mentioning the possiblity to compile without this option and compiling pict-rs manually. This is necessary to use a custom config or other customizations. It looks like the pictrs section under upgrading was forgotten to be removed when that feature was added, could you get rid of it?

mtekman · 2023-06-21T12:03:12Z

Oh I see! So when lemmy_server is updated, so is the embedded pictrs. Okay, I will kick it out

Edit: I kept it in because I think it's useful, but I added a note about it not being needed if --features embed_picts is used.
Let me know if I should kick it out entirely, and I will happily do so.

Nutomic · 2023-06-21T14:23:21Z

src/en/administration/from_scratch.md

@@ -207,14 +217,17 @@ systemctl restart lemmy-ui

 ### Pict-rs

+If you did **not** use the `--features embed-pictrs` flag, then this script below is necessary for installing/updating Pict-rs as a standalone server.
+Otherwise, pict-rs should update with lemmy_server.


This only makes sense if you mention in the beginning that pictrs can be installed as standalone.

I will add, but another point of confusion for me:

Pict-rs requires the magick command which comes with Imagemagick version 7, but Ubuntu 20.04 only comes with Imagemagick 6. So you need to install that command manually, eg from the official website.

This is still true, regardless of whether pict-rs is standalone or embedded, right?

Yes dependencies would be the same either way.

Nutomic · 2023-06-22T07:55:11Z

Looks good but there is a conflict.

Updated out of date LEMMY_UI environment variables, and added caution with failing postgresql configuration

mtekman · 2023-06-22T09:20:06Z

force pushed after rebase, let me know if anything looks weird. Cheers

Nutomic reviewed Jun 20, 2023

View reviewed changes

Nutomic reviewed Jun 21, 2023

View reviewed changes

Nutomic approved these changes Jun 22, 2023

View reviewed changes

mtekman and others added 5 commits June 22, 2023 11:05

Update from_scratch.md

dfccfa1

Updated out of date LEMMY_UI environment variables, and added caution with failing postgresql configuration

Feedback from @Nutomic and @RocketDerp

a816fca

ran through prettier

8673f77

resolve merge conflict

bbfd5f3

Added note about pict-rs being standalone and embeddable

a501d22

mtekman force-pushed the main branch from 6ee93d2 to a501d22 Compare June 22, 2023 09:10

Nutomic merged commit 17588ef into LemmyNet:main Jun 22, 2023

mtekman mentioned this pull request Jun 24, 2023

Under the "From Scratch" installation, it is not clear where Pict-rs is initially installed, so the update doesn't work #219

Open

Nutomic mentioned this pull request Jun 26, 2023

"Peer authentication failed for user lemmy" #220

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update from_scratch.md (LEMMY_UI variables, and added note about failing postgresql config) #221

Update from_scratch.md (LEMMY_UI variables, and added note about failing postgresql config) #221

mtekman commented Jun 19, 2023

Nutomic Jun 20, 2023 •

edited

Loading

mtekman Jun 20, 2023

mtekman Jun 21, 2023

Nutomic Jun 20, 2023

mtekman Jun 20, 2023

This comment was marked as abuse.

mtekman Jun 21, 2023

Nutomic commented Jun 21, 2023

mtekman commented Jun 21, 2023

mtekman commented Jun 21, 2023

Nutomic commented Jun 21, 2023

mtekman commented Jun 21, 2023 •

edited

Loading

Nutomic Jun 21, 2023

mtekman Jun 21, 2023

Nutomic Jun 22, 2023

Nutomic commented Jun 22, 2023

mtekman commented Jun 22, 2023

Update from_scratch.md (LEMMY_UI variables, and added note about failing postgresql config) #221

Update from_scratch.md (LEMMY_UI variables, and added note about failing postgresql config) #221

Conversation

mtekman commented Jun 19, 2023

Nutomic Jun 20, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This comment was marked as abuse.

Choose a reason for hiding this comment

Nutomic commented Jun 21, 2023

mtekman commented Jun 21, 2023

mtekman commented Jun 21, 2023

Nutomic commented Jun 21, 2023

mtekman commented Jun 21, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Nutomic commented Jun 22, 2023

mtekman commented Jun 22, 2023

Nutomic Jun 20, 2023 •

edited

Loading

mtekman commented Jun 21, 2023 •

edited

Loading