Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update installation.md - add ubuntu install instructions #59

Conversation

RationalAnarchist
Copy link

Installation instructions for Ubuntu. I can't be sure that the markdown is correct - can't find a way to preview it, but hopefully it's ok. Aiming to spin up some other distros at the weekend to confirm if the same sort of thing will work for them.

@RationalAnarchist
Copy link
Author

Links to #57

@RationalAnarchist RationalAnarchist changed the title Update installation.md Update installation.md - add ubuntu install instructions Jun 14, 2023
@Berserkir-Wolf
Copy link
Collaborator

Any particular reason your user is defined as 'pi' on a Ubuntu guide? I know that user definitely doesn't exist on at least a dozen of the ubuntu servers I administer.

@ajurna
Copy link
Contributor

ajurna commented Jun 15, 2023

a new service user called kapowarr should be added. usually it's best practice to run as service as a service account and isolate it's permissions.

@RationalAnarchist
Copy link
Author

You're both right - sorry! That was my default user - will update when I get the chance (may not be until the weekend)

@Casvt
Copy link
Owner

Casvt commented Sep 24, 2023

How's it going?

@Casvt
Copy link
Owner

Casvt commented Feb 7, 2024

I'm going to close the PR, seeing that it does not appear to be worked on anymore. But the work that you've done up to now will for sure be used, so thank you for that.

EDIT: So please don't delete your fork yet, because I will still check it and use certain aspects from it.

@Casvt Casvt closed this Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants