-
Notifications
You must be signed in to change notification settings - Fork 116
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
Create xlxd.service and improve systemd integration #243
base: master
Are you sure you want to change the base?
Conversation
run xlxd as xlxd user (needed to be created manually)
… ambed.service is fully started up.
I suggest the unit files for xlxd.service and ambed.service have "Requires=network-online.target" otherwise xlxd will fail to start because the network interface is not configured with an ip address. |
@DcnPat It already contains After=network.target - I guess that this should be enough. I would also appreciate it, if the project maintainers would react on this at some point. |
I found
After=network.target
Results in failure to start because the interface was not up so xlxd could not open ports to listen on. There needs to be an actual IP address associated with the interface and it has to be up, hence network-online.target is needed.
…On Wed, Sep 11, 2024, 4:21 PM Fabian Franz BSc ***@***.***> wrote:
@DcnPat <https://github.com/DcnPat> It already contains
After=network.target - I guess that this should be enough.
—
Reply to this email directly, view it on GitHub
<#243 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASCA6OOAFESEW5JOI5J3YRLZWCQ4JAVCNFSM6AAAAABHW2WCTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBUGYZDSNZXGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Checked that the target exists, so we can add this directive to the server. You comment sounds like you deployed my changes on your server. How are they doing so far? |
starts xlxd after interface has its ip
start ambed after we have an IP address
run xlxd as xlxd user (needed to be created manually)
also add a task to keep the node alive (xlxapi)
alternative to #195