From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.comp.sysutils.supervision.general/2861 Path: news.gmane.io!.POSTED.ciao.gmane.io!not-for-mail From: Steve Litt Newsgroups: gmane.comp.sysutils.supervision.general Subject: Re: runit SIGPWR support Date: Fri, 14 Feb 2020 07:38:13 -0500 Message-ID: <20200214073813.765f362a@mydesk.domain.cxm> References: <1beb6e35-d4be-60b8-fc52-af666c4fffe3@gmx.com> <20200212215427.GG12551@cathexis.xen.prgmr.com> <6843181581673118@sas1-ffdbcd5f1d77.qloud-c.yandex.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="ciao.gmane.io:159.69.161.202"; logging-data="15395"; mail-complaints-to="usenet@ciao.gmane.io" To: supervision@list.skarnet.org Original-X-From: supervision-return-2450-gcsg-supervision=m.gmane-mx.org@list.skarnet.org Fri Feb 14 13:38:21 2020 Return-path: Envelope-to: gcsg-supervision@m.gmane-mx.org Original-Received: from alyss.skarnet.org ([95.142.172.232]) by ciao.gmane.io with smtp (Exim 4.92) (envelope-from ) id 1j2aEd-0003sI-Tm for gcsg-supervision@m.gmane-mx.org; Fri, 14 Feb 2020 13:38:20 +0100 Original-Received: (qmail 12336 invoked by uid 89); 14 Feb 2020 12:38:45 -0000 Mailing-List: contact supervision-help@list.skarnet.org; run by ezmlm Original-Sender: Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Original-Received: (qmail 12329 invoked from network); 14 Feb 2020 12:38:45 -0000 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; h=X-Originating-IP:Date:From:To:Subject:Message-ID:In-Reply-To:References:MIME-Version:Content-Type:Content-Transfer-Encoding; s=default; d=troubleshooters.com; b=R3sFuR2lmO5cBldspphMnDFw9O/SncDqvQ8ZdcIJn3MOkf8TDwcOe+9gg2nQQBi3E6tZqCb0ELryh59A2m4NcA0krf5ZKuHrhPeiIqh2wdIPLjJbw2MdO9aFBxd84AKDlXhHQWHaiMSoiqAeV2eOs1YIkfbmM9mNGGmv0olNKSI=; DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/simple; d=troubleshooters.com; s=default; t=1581683894; bh=/DUgQQ9dY1uMVjDBNBasxrRRKpM=; l=2880; h=X-Originating-IP:Date:From:To:Subject:Message-ID:In-Reply-To: References:MIME-Version:Content-Type:Content-Transfer-Encoding; b=dH6F/O4gSBffcfWDhU8iVqxLyAjeFfo13sboMgXPEX2cF+C2CD/vlqSzJ6Hx8b4Uq J8letW4Z6eK9DTDc+2yjb2e19cI9UpuamlrR4jnA4DQE9Tr3r3wGn2mGUD/IfUEOx/ 3CeIb0mwBZrX8KrndXg/H8MjG6OVTQMMKhbZSQnM= X-Originating-IP: [72.188.224.222] In-Reply-To: <6843181581673118@sas1-ffdbcd5f1d77.qloud-c.yandex.net> Xref: news.gmane.io gmane.comp.sysutils.supervision.general:2861 Archived-At: On Fri, 14 Feb 2020 10:38:38 +0100 Jeff wrote: > 12.02.2020, 22:54, "Colin Booth" : > > On Wed, Feb 12, 2020 at 05:25:56PM +0300, innerspacepilot wrote: =20 > >> =C2=A0Why not just make runit systems run inside containers out of the > >> box? We are talking about one/two lines of code. =20 >=20 > you should patch the code, runit is dead anyway. Oh really. No chance that Gerrit has simply taken a year or two off? Why are you using software you consider dead? Why not move to s6 if runit is dead? > try something along this lines in the source: >=20 > #ifdef SIGPWR > /* handle that one */ > ... > #endif If it's that simple, why don't YOU do it? Someone already mentioned that BSD doesn't have this signal, so incorporating it in the wider project could be objected to by some people as non-portable. So why don't YOU make this change on YOUR copy? >=20 > i can't see the problem, you have to patch the runit sources to > fulfil your requirements since that project is dead and the code > is not maintained anymore. >=20 > >> =C2=A0Why can't we be just a little bit more friendly to each other? = =20 >=20 > that would be indeed helpful. And you can start by not calling a project "dead". >=20 > > I wasn't trying to be hostile, apologies if it came across that > > way. As far as I know SIGPWR is a Linux-specific signal so services > > that are aiming for portability will either need to have special > > handling for that in the linux case or need to ignore it. Ergo, > > runit (and all other POSIX-compliant inits) currently have no > > special handling around SIGPWR as they don't understand what it is. > > > > Is this the right behavior? I don't know. Something like SIGPWR as > > an alerting mechanism when you're switched to UPS battery is pretty > > nice in a general case but using that as your container shutdown > > solution isolates you into a very SysV-specific world. Overriding > > the default via lxc.signal.halt will allow you to modify what you > > send to something that is within the POSIX spec and allow you to > > trigger shutdowns the "right" way. It's a little lame but it is > > portable, and LXC using a non-portable signal is a little bit of a > > bummer. =20 >=20 > just BS. adding a bit of handler code for SIGPWR is no big deal, > please stop your lamento, it's so boring. I guess your resolution to be a little bit more friendly to each other didn't last too long. You're one guy who wants this change. You have many alternatives: * s6 * lxc.signal.halt * Patch runit yourself Everyone else: Just to express my opinion: I use runit every day and personally have no need to have it react to SIGPWR. SteveT Steve Litt=20 February 2020 featured book: Thriving in Tough Times http://www.troubleshooters.com/thrive