-
-
Notifications
You must be signed in to change notification settings - Fork 196
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
Convert the daemons to native systemd units #8024
Conversation
1537787
to
fd9444d
Compare
62769a6
to
4faec08
Compare
cc51ee9
to
20b414b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed to enable ERB trim mode else the rake task didn't work. Have also update the xapian compact script to call rails restart
.
Commits should be reordered and messages updated. I'll do this on merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Turns out when and install use rbenv
these weren't working. This was down to how the paths were defined. I have moved this logic into the rake task instead of the service templates and fixed - IE single PATH
env no SET_PATH
env needed.
Allows `-%>` ending ERB sequence.
This isn't nesscasary and causes systemd services to write logs to a new file instead of the old one used by init.d scripts.
Saves conditionals in the daemons services.
This replaces the old init script for the alert tracks daemon with a systemd unit file.
This replaces the old init script for the send notifications daemon with a systemd unit file.
This replaces the old init script for the mail poller daemon with a systemd unit file.
Rather than calling out to a script in `/etc/init.d`, call `rails restart` to directly touch `restart.txt` (which is essentially what the old init script does anyway). This removes a dependency on having to have this init script in place when using this script.
f98d152
to
6d51507
Compare
This should be good to merge next week. |
This replaces the old init scripts for the daemons with
systemd
unit files.Relevant issue(s)
#2523
What does this do?
Replaces the Alert Tracks, Send Notifications and Poll Incoming daemon sysvinit scripts with systemd units
Why was this needed?
To modernise the daemons