From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51048) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d8XLE-0007iG-CN for guix-patches@gnu.org; Wed, 10 May 2017 15:32:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d8XL9-0003RZ-Ky for guix-patches@gnu.org; Wed, 10 May 2017 15:32:08 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:34430) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1d8XL9-0003RV-Hk for guix-patches@gnu.org; Wed, 10 May 2017 15:32:03 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1d8XL9-0003mk-Bj for guix-patches@gnu.org; Wed, 10 May 2017 15:32:03 -0400 Subject: bug#26830: Allow services to implement a 'reload' action Resent-Message-ID: References: <87d1bjtlpd.fsf@lassieur.org> <86vapa6nyi.fsf@gmail.com> From: =?UTF-8?Q?Cl=C3=A9ment?= Lassieur In-reply-to: <86vapa6nyi.fsf@gmail.com> Date: Wed, 10 May 2017 21:31:58 +0200 Message-ID: <87vap8h5jl.fsf@lassieur.org> MIME-Version: 1.0 Content-Type: text/plain List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Mathieu Othacehe Cc: 26830@debbugs.gnu.org Mathieu Othacehe writes: >> Services do not have to implement 'reload' and if, say, foo-daemon >> doesn't implement it, 'herd reload foo-daemon' will return 1 and display >> a message saying that foo-deamon does not have an action 'reload'. >> That's the reason of the #f default value. >> >> WDYT? > > Your whole serie LGTM for me ! Hi Mathieu, thanks for reviewing :) > I have just one small concern, there is a already a "reload" action on > shepherd root service. > > For instance you can call "herd reload root conf.scm". > > Maybe it will be unclear for users how reload action differs on root > service where it takes an argument and guix services where it does not. They don't differ: 'root' is just another service, as 'nginx' is. Our 'reload' action can handle many arguments as well. The only tiny difference is that the 'root' service is implemented by Shepherd, not by Guix. > You could maybe mention that in the documentation and/or in the code ? Sure, I updated the documentation. I had forgotten the "Shepherd Services" part and I think it helps understanding. But I didn't talk about the 'root' service because it is a Shepherd thing and is already described in the Shepherd manual. WDYT?