Bug 1301297 - Switch to systemd unit files
Summary: Switch to systemd unit files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: sigul
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Miloslav Trmač
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-23 20:41 UTC by Kevin Fenzi
Modified: 2016-01-25 17:58 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-01-25 17:50:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Convert sigul to using systemd units (4.53 KB, application/mbox)
2016-01-23 20:41 UTC, Kevin Fenzi
no flags Details

Description Kevin Fenzi 2016-01-23 20:41:27 UTC
Created attachment 1117485 [details]
Convert sigul to using systemd units

Soon FESCo has decided to drop all packages that still use sysvinit scripts. 

I've attached a patch that moves sigul to use systemd units instead of sysvinit scripts. Consider applying, or if you prefer I can go ahead and apply and build.

Comment 1 Miloslav Trmač 2016-01-25 17:50:34 UTC
Thanks for the patch.

AFAICS those units couldn’t work:
* Starting services as the sigul user means they don't have enough privileges to open the log file, or create the pid file (nor to implement the unix-{user,group} directives from the config files).
* Type=simple along with -d means the service terminates as soon as the parent launches the daemon child.

I have now built sigul-0.102-2.fc24 with units that seem to work for me, though testing would be appreciated.

(systemctl stop takes 90 seconds to time out for each service, mostly due to #707382; but eventually systemd does terminate the daemons. The default timeout could very likely be tuned further, but this is good enough as a minimal conversion.)

Comment 2 Kevin Fenzi 2016-01-25 17:58:59 UTC
Yeah, I meant to test, but ran out of time. ;) Sorry about that and thanks for fixing things up.


Note You need to log in before you can comment on or make changes to this bug.