Date: Fri, 21 Aug 2020 19:16:40 +0100
From: Natanael Copa <ncopa@alpinelinux.org>
To: Ariadne Conill <ariadne@dereferenced.org>
Cc: =?UTF-8?B?U8O2cmVu?= Tempel <soeren@soeren-tempel.net>,
 ~alpine/devel@lists.alpinelinux.org
Subject: Re: Use of supervise-daemon in Alpine
Message-ID: <20200821191640.5238557d@ncopa-macbook.copa.dup.pw>
In-Reply-To: <f7b0aab5-4dc6-69f1-048d-f90f59891b45@dereferenced.org>
References: <3LLUI2KOULSYM.359WA6HATX45B@8pit.net>
	<f7b0aab5-4dc6-69f1-048d-f90f59891b45@dereferenced.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Thu, 20 Aug 2020 14:33:53 -0600
Ariadne Conill <ariadne@dereferenced.org> wrote:

> Hello,
>=20
> On 2020-08-20 11:04, S=C3=B6ren Tempel wrote:
> > Hello,
> >=20
> > OpenRC ships a program called supervise-daemon(8) which is capable of
> > starting daemons and restarting them if they crash. Contrary to
> > start-stop-daemon, it does not rely on PID files instead the started
> > daemon is a child process of supervise-daemon.
> >=20
> > Some Alpine OpenRC services already use supervise-daemon(8) (e.g.
> > unbound, xdm, wpa_supplicant, =E2=80=A6). I was recently wondering if w=
e want to
> > migrate busybox-initscripts to using supervise-daemon too and was
> > pointed to some comments in the GitLab issue tracker which critique use
> > of supervise-daemon for busybox-initscripts because of concern over
> > memory usage [0]. Upon further discussion in the IRC some people also
> > expressed discomfort in regards to the automatic restarting of crashed
> > services (=E2=80=9Cyou don't want to mask crashes=E2=80=9D). =20
>=20
> I would rather mask crashes than deal with a 3 AM phone call.  As long=20
> as the crashes are logged (which supervise-daemon does), I don't see the=
=20
> problem.

I think many agree with you, but not everyone. So it should be
configurable.

-nc