Re: [PATCH 04/11] services: Rename %do-not-kill-file.

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> Ludovic Courtès (2016-01-28 13:25 +0300) wrote:
>
>> Alex Kost <address@hidden> skribis:
>>
>>> * gnu/services/base.scm (%do-not-kill-file): Rename to
>>>   "/etc/shepherd/do-not-kill".
>>
>> Shouldn’t it be merged with patch 03?  Otherwise patch 03 alone would
>> break things AIUI, because it would create a different file than the one
>> actually used.
>
> OK, you know better (I actually don't understand what these files are
> used for).
>
> An interesting thing is that ‘mark-as-not-killable’ from (gnu build
> linux-boot) uses "/root/etc/dmd/do-not-kill", while ‘%do-not-kill-file’
> is "/etc/dmd/do-not-kill".  These files have always been different
> (since commit 7d57cfd) and it didn't break things.  So IIUC it should be
> the same file name in both places, right?

I replied on IRC, but for posterity ;-), replacing “/dmd” with
“/shepherd” in both places is fine, and it’s also purely cosmetic since
these are the only two places referring to this file.

This file tells the shepherd not to kill the unionfs process that
provides the root file system.

The unionfs on / is used in images produced by ‘guix system vm’.  If you
do:

  kill $(cat /etc/shepherd/do-not-kill)

in such a VM, you’ll notice that things start failing.  :-)

Ludo’.