supervision - discussion about system services, daemon supervision, init, runlevel management, and tools such as s6 and runit
 help / color / mirror / Atom feed
From: "Serge E. Hallyn" <serge@hallyn.com>
To: Jonathan de Boyne Pollard <J.deBoynePollard-newsgroups@NTLWorld.COM>
Cc: Supervision <supervision@list.skarnet.org>
Subject: Re: Readiness notification exemplars
Date: Sat, 4 Apr 2020 10:48:51 -0500	[thread overview]
Message-ID: <20200404154851.GC698@mail.hallyn.com> (raw)
In-Reply-To: <2771eb82-9e19-06a7-d15c-28633971d8b2@NTLWorld.COM>

On Fri, Apr 03, 2020 at 07:41:25AM +0000, Jonathan de Boyne Pollard wrote:
> Serge E. Hallyn:
> 
> > If making changes to daemons were going to palatable, [...]
> > 
> 
> Clearly, it *is* palatable, given that a few people have been adding the
> systemd mechanism to their programs for several years, now.   Pierre-Yves

Well both that and the less successfull upstart sigstop had the full force
of popular distros behind them.  Still I hope you're right, and maybe I'll
go ahead and post some patches for a few daemons.

> Ritschard's code and Cameron Norman's code come straight out of actual
> service programs.

> Serge E. Hallyn:
> 
> > you are arguing for parsing stdout for a text message and/or using
> > pidfiles (written to an fd)
> > 
> 
> No, M. Bercot is not.  Quite the opposite, in fact.  The s6 readiness
> mechanism demands *neither* pidfiles *nor* standard output.  It's *any* file
> descriptor (as defined by the service), and *just a linefeed* followed by
> closing the descriptor.
> 
> I'm nodding at almost everything M. Bercot is saying here, by the way.
> 
> * http://jdebp.uk./FGA/unix-daemon-readiness-protocol-problems.html

Yes it sounds like I completely misread the earlier emails, sorry about
that.  Now, looking at http://skarnet.org/software/s6/notifywhenup.html,
I'm probably not reading that quite right, but it seems to tie the
proposal to the 'notifcation-fd' file in the service directory, making
it a bit less general.  s6-log however uses the -d argument to pass the
fd, so I'll go with that.

thanks,
-serge


  reply	other threads:[~2020-04-04 15:48 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-01 13:23 Brett Neumeier
2020-04-01 13:36 ` Casper Ti. Vector
2020-04-01 14:21 ` Serge E. Hallyn
2020-04-01 15:06   ` Laurent Bercot
2020-04-01 15:28     ` Serge E. Hallyn
2020-04-01 15:59       ` Laurent Bercot
2020-04-01 16:26         ` Serge E. Hallyn
2020-04-01 17:13           ` Laurent Bercot
2020-04-04 15:02             ` Serge E. Hallyn
2020-04-04 15:54               ` Laurent Bercot
2020-04-03  7:41           ` Jonathan de Boyne Pollard
2020-04-04 15:48             ` Serge E. Hallyn [this message]
2020-04-04 21:29               ` Laurent Bercot
2020-04-04 22:18                 ` Serge E. Hallyn
2020-04-07 23:03                   ` Brett Neumeier
2020-04-08 11:02                     ` Laurent Bercot
2020-04-09 10:31                     ` Jonathan de Boyne Pollard
2020-04-01 14:55 ` Laurent Bercot

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=20200404154851.GC698@mail.hallyn.com \
    --to=serge@hallyn.com \
    --cc=J.deBoynePollard-newsgroups@NTLWorld.COM \
    --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).