Skip to content

Add systemd service#79

Closed
akirilov wants to merge 18 commits into
nightlyfrom
feat-systemd-service
Closed

Add systemd service#79
akirilov wants to merge 18 commits into
nightlyfrom
feat-systemd-service

Conversation

@akirilov

@akirilov akirilov commented Sep 15, 2025

Copy link
Copy Markdown
Contributor

Description

Improvements

  • Added a autorun.sh script on Linux. Running this will install a system service to run SSUI
  • This will fail if run as root or run on a non-systemd system

Copilot AI review requested due to automatic review settings September 15, 2025 14:16

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@akirilov akirilov requested a review from Copilot September 15, 2025 14:22

This comment was marked as outdated.

@akirilov akirilov requested a review from Copilot September 15, 2025 14:25
@JacksonTheMaster JacksonTheMaster requested review from JacksonTheMaster and removed request for Copilot September 15, 2025 14:25

@JacksonTheMaster JacksonTheMaster left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok - this was a test of the new rule and permission system in place

@akirilov akirilov closed this Sep 15, 2025
@akirilov

Copy link
Copy Markdown
Contributor Author

Closing this for now since I realized updates will break the service, will re-open when it's fixed

This comment was marked as outdated.

@akirilov akirilov requested a review from Copilot September 15, 2025 15:35

This comment was marked as outdated.

@akirilov akirilov requested a review from Copilot September 15, 2025 15:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh
Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh Outdated
Comment thread UIMod/onboard_bundled/scripts/autostart.sh
@mitoskalandiel

Copy link
Copy Markdown
Contributor

Something more important occured to me; when we are in a docker environment we cannot allow a user to use this function at all.

mitoskalandiel
mitoskalandiel previously approved these changes Sep 17, 2025

@mitoskalandiel mitoskalandiel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good to me

@mitoskalandiel

Copy link
Copy Markdown
Contributor

Something more important occured to me; when we are in a docker environment we cannot allow a user to use this function at all.

Along the same line, lets in future differentiate between "service autostart" (like ssui starting up during HOST start) and "game autostart" which just laucnhes the game server, once ssui is up.

Comment thread src/setup/autostartscripts.go
Comment thread src/setup/autostart/autostartscripts.go
@JacksonTheMaster

Copy link
Copy Markdown
Member

Updated nightly to be on sync with main, didnt realize this would close this PR

JacksonTheMaster and others added 2 commits September 18, 2025 02:51
- Trying to not run into merge issues later on
- because im working on go and localization too here
…ostart scripts and service. SSUI now can be run with --setupautostart to add a systemd service.

- revised autostart.sh to only create the service file
- moved SetupAutostartScripts to autostart package, handles script creation for both Windows and Linux.
- Created service-lin.go for Linux service setup, including checks for existing service installation.
- Added symlink-linux.go to manage symlink creation of a StationeersServerUI.lnk
- Implemented container detection to skip symlink creation if running inside a Docker container.
- Placeholder for Windows service setup in service-win.go.
…ostart scripts and service. SSUI now can be run with --setupautostart to add a systemd service.

- revised autostart.sh to only create the service file
- moved SetupAutostartScripts to autostart package, handles script creation for both Windows and Linux.
- Created service-lin.go for Linux service setup, including checks for existing service installation.
- Added symlink-linux.go to manage symlink creation of a StationeersServerUI.lnk
- Implemented container detection to skip symlink creation if running inside a Docker container.
- Placeholder for Windows service setup in service-win.go.
- Added instructions to first time setup message
- moved init of VirtFS earlier in server.go
@JacksonTheMaster

Copy link
Copy Markdown
Member

@akirilov please test my additions from 5a1eeb4

@JacksonTheMaster JacksonTheMaster removed the request for review from mitoskalandiel September 21, 2025 02:47
@mitoskalandiel

Copy link
Copy Markdown
Contributor

This will need to be re-implemented from current #94

@mitoskalandiel mitoskalandiel deleted the branch nightly September 22, 2025 18:03
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