supervision - discussion about system services, daemon supervision, init, runlevel management, and tools such as s6 and runit
 help / color / mirror / Atom feed
From: innerspacepilot <innerspacepilot@gmx.com>
To: Laurent Bercot <ska-supervision@skarnet.org>,
	supervision@list.skarnet.org
Subject: Re: runit SIGPWR support
Date: Mon, 17 Feb 2020 13:00:12 +0300	[thread overview]
Message-ID: <cf562a5e-3431-a95c-62ec-0638cba81950@gmx.com> (raw)
In-Reply-To: <emefeddee4-7f81-43ed-977e-82b44b41c43a@elzian>

Great, your point I wanted to hear especially.
But, well, I am disillusioned with my hops for s6.

My fault about SIGPWR, RTMIN+3 should be used instead, please, treat
SIGPWR as a template for any other signal name, that doesn't matter.

Not only me who want this "lxd simplicity", e.g.
https://github.com/skarnet/s6/issues/5, where you fully described your
position.

Just as a thought: You have implemented signal diversion, but limited to
known signals. Why not just pass unknown signals as numbers or something
like (S6SIG55011), so they can be diverted by user? You wouldn't have to
catalogue them.

We need good, flexible and user-friendly init alternatives for linux
(other systems have their own).

So, I just want to make user experience of new runit or s6 users who use
LXD more comfortable, apologies if this hurts someone.

Have a good day!







On 14.02.2020 21:30, Laurent Bercot wrote:
>> You mean that adding few lines of code in one place is worse than many
>> users of many distros must configure their containers?
>> I can configure that myself, but I don't want every user of runit driven
>> container to walk this path. Is it necessary?
>
> As counterintuitive as it may seem at first glance, the answers to your
> questions are yes and yes. Patching software is always more complex than
> configuring it.
>
> Configuring software is using an API that has been especially thought
> out to accommodate the needs of various users; if a piece of software
> does what you want but requires you to tweak a configuration lever,
> then it does what you want period. Because your needing to tweak the
> configuration lever is *the exact reason why that lever is there*.
> If you refuse to use it, you are basically putting your needs in
> front of everyone else's, and demanding that the author of the software
> adapt to you at the exclusion of others, instead of using the mechanism
> that has been prepared for you.
>
> Patching software:
> - requires communication with upstream, so, takes support resources
> - requires new deployment, which is significant effort
> - is dangerous: it may introduce bugs that you haven't thought of
> - may change the workflow of other users
>
> It is, of course, a supplementary order of magnitude more difficult with
> software that has no well-defined upstream, as is the case with runit
> these days.
> But even if your containers were using s6, which has a well-defined
> upstream (me) and which does not understand SIGPWR either, I would not
> apply your patch suggestion. Why? Because SIGPWR is not standardized,
> and s6 aims to be portable, it works ootb on other systems than Linux
> and making it use SIGPWR would endanger that. It's the exact kind of
> problems you haven't thought of but run into when you want to patch
> software, and makes patching always more complex than it seems from the
> outside.
>
> Explaining to users how to configure lxc to send the correct signal
> to the init system running in the container is a matter of one line in
> the documentation. It's extremely manageable.
>
>
>> Also there is a huge lack of documentation about it on the net,
>> especially on signals that runit accepts.
>
> You are talking about patching the code, and you're not going to
> look at runit's code to see what signals it accepts? ;)
>
>
>> It adds complexity to users, and that means users will choose other
>> distros which just work.
>
> If your definition of "just working" is "everything is working with
> the default configuration", then I don't think you'll find a single
> Linux distribution that "just works".
>
> Your runit distro is working just as well as any other. You just
> need to set one variable in the lxc configuration. It's certainly not
> the only variable you need to set; it's certainly not even the only
> variable you need to set conditionally depending on the guest distro.
> So, there's really no reason to get hung up on this.
>
>
>>>> Why can't we be just a little bit more friendly to each other?
>
>  Most participants in this thread would benefit from taking this advice
> to heart. And in your case, the friendliness will consist in using
> the configuration levers that were made for you instead of demanding
> that upstream change its defaults to adapt to you.
>
> --
>  Laurent
>



  reply	other threads:[~2020-02-17 10:00 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1beb6e35-d4be-60b8-fc52-af666c4fffe3@gmx.com>
2020-02-12 14:25 ` innerspacepilot
2020-02-12 21:54   ` Colin Booth
2020-02-12 22:16     ` Dewayne Geraghty
2020-02-14  9:38     ` Jeff
2020-02-14 12:38       ` Steve Litt
2020-02-15 10:47       ` fungal-net
2020-02-14 10:08     ` Jeff
2020-02-14 10:46     ` Jeff
2020-02-14 12:29       ` innerspacepilot
2020-02-14 12:45         ` Steve Litt
     [not found]           ` <CALZWFRLvtofWfP4kzxJ8_8_K3nzebPjCR-NsJ2MU22cSuaOLng@mail.gmail.com>
     [not found]             ` <20200214182241.15614126@mydesk.domain.cxm>
2020-02-17 19:46               ` Cameron Nemo
2020-02-23 16:11                 ` Jeff
2020-02-17 14:39         ` Jeff
2020-02-14 14:02       ` Casper Ti. Vector
2020-02-17 14:45     ` Jeff
2020-02-17 14:50       ` Jeff
2020-02-14 13:15   ` Casper Ti. Vector
2020-02-14 13:39     ` innerspacepilot
2020-02-14 13:57       ` Casper Ti. Vector
2020-02-14 14:06         ` innerspacepilot
2020-02-14 14:25           ` Casper Ti. Vector
2020-02-14 18:30       ` Laurent Bercot
2020-02-17 10:00         ` innerspacepilot [this message]
2020-02-17 15:13           ` Jeff
2020-02-18  9:39             ` Laurent Bercot
2020-02-20 20:39               ` Serge E. Hallyn
2020-02-23 16:51               ` Jeff
2020-02-23 23:53                 ` Laurent Bercot
2020-02-24  6:31                   ` innerspacepilot
2020-02-24 10:23                     ` Laurent Bercot
2020-02-24 13:00                       ` Jeff
2020-02-24 19:53                         ` Laurent Bercot
2020-02-24 13:12                       ` innerspacepilot
2020-02-24 15:26                         ` Serge E. Hallyn
2020-02-26  8:07                           ` innerspacepilot
2020-02-28  6:39                             ` Jan Braun
2020-02-28  9:45                               ` Alex Suykov
2020-02-28 23:50                                 ` fungal-net
2020-02-29 13:44                                 ` Jonathan de Boyne Pollard
2020-02-29 18:20                             ` Guillermo
2020-03-06 20:07                               ` innerspacepilot
2020-03-06 20:09                               ` innerspacepilot
2020-02-25  8:39                       ` Jonathan de Boyne Pollard
2020-02-24 21:13                   ` Guillermo
2020-02-24 22:25                     ` Laurent Bercot
2020-02-24 22:49                       ` Laurent Bercot
2020-02-24 23:08                         ` Guillermo
2020-02-25  1:48                           ` Laurent Bercot
2020-02-25  9:08                             ` Jonathan de Boyne Pollard
2020-02-25 18:38                               ` Guillermo
2020-03-16 12:49                               ` Jeff
2020-03-16 17:13                               ` Jeff
2020-02-24 23:03                       ` Guillermo
2020-03-16 12:31                       ` Jeff
2020-03-16 18:03                         ` Laurent Bercot
2020-02-23 17:31               ` Jeff
2020-02-24  0:33                 ` Laurent Bercot
2020-02-14 19:08   ` John W Higgins
2020-02-14 23:18     ` Laurent Bercot
2020-02-14 23:38       ` John W Higgins
2020-02-15  2:15         ` Laurent Bercot
2020-04-14 16:57 Maxim Vetsalo
  -- strict thread matches above, loose matches on Subject: below --
2020-01-23 20:44 innerspacepilot
2020-01-31  4:39 ` Colin Booth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cf562a5e-3431-a95c-62ec-0638cba81950@gmx.com \
    --to=innerspacepilot@gmx.com \
    --cc=ska-supervision@skarnet.org \
    --cc=supervision@list.skarnet.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).