[systemd-devel] [PATCH] service: don't create extra cgroup for control process when reloading SysV service

["Jóhann B. Guðmundsson"]

On 03/11/2014 09:05 AM, Michael Biebl wrote:
> 2014-03-10 15:25 GMT+01:00 Lukas Nykryn <lnykryn at redhat.com>:
>> Unfortunately common practice in initscripts is to have reload as an
>> alias for restart (https://fedoraproject.org/wiki/Packaging:SysVInitScript).
>> In that case the newly started process will be killed immediately after
>> the reload process ends and its cgroup is destroyed.
> Interesting. The Debian/Ubuntu packaging policies are quite contrary
> in that regard.
> LSB/SysV init scripts only have the reload option, if the daemon
> actually supports it.
> There is a force-reload action though, which is mapped to reload, if
> the daemon supports reload and to restart, if not.
>
> Given that, I wonder if this should be maintained as a Fedora specific patch?

Debian is getting it right in this regard since LSB defines the behavior 
of reload to be [1]

"*reload* cause the configuration of the service to be reloaded without 
actually stopping and restarting the service"

In anycase systemd should follow LSB in this regard leaving any 
downstream distribution having to patch their own common practices in 
initscripts by themselves.

JBG

1. 
http://refspecs.linuxbase.org/LSB_4.1.0/LSB-Core-generic/LSB-Core-generic/iniscrptact.html
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/systemd-devel/attachments/20140311/f32329ed/attachment.html>