Skip to content

Include systemd documentation for run on boot - #3

Merged
dgibbs64 merged 3 commits into
GameServerManagers:masterfrom
Chrzi:patch-1
Jan 4, 2019
Merged

dgibbs64 merged 3 commits into
GameServerManagers:masterfrom
Chrzi:patch-1

Conversation

@Chrzi

@Chrzi Chrzi commented Jan 2, 2019

Copy link
Copy Markdown
Contributor

systemd is the default init system for most modern distros and should be preferred over crontabs and SysVinit

I included a simple service file for a teamspeak 3 server that can be easily adopted to other servers as well

Chrzi added 2 commits January 2, 2019 14:24
systemd is the default init system for most modern distros and should be preferred over crontabs and SysVinit

I included a simple service file for a teamspeak 3 server that can be easily adopted to other servers as well/
Forking type did not work as systemd could not reliably determine the PID of the actual server.
@Chrzi

Chrzi commented Jan 4, 2019 •

Copy link
Copy Markdown
Contributor Author

You could use systemd timers (that work just like cronjobs), but with the unit files itself currently no.

Systemd could detect with the process died and try to restart it then, but this fails because the LGSM script is forking to tmux and exits after that, without a PID file systemd can not reliably determine the main process of the actual server.

https://www.freedesktop.org/software/systemd/man/systemd.service.html

@dgibbs64

dgibbs64 commented Jan 4, 2019

Copy link
Copy Markdown
Member

Thanks for this. system.d is a little more complicated to install than just using cronjobs and achieves the same result.

Having said that it is a very good idea to include this method as it is the current standard. I might be a good idea for LinuxGSM to automate the installation of a system.d service maybe using adding a new command allowing admins to use use the command systemsctl restart csgoserver for example

@dgibbs64
dgibbs64 merged commit 6fd9d55 into GameServerManagers:master Jan 4, 2019
dgibbs64 pushed a commit that referenced this pull request Feb 10, 2019
Merge pull request #11 from AntonnMal/master
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.

3 participants