All of lore.kernel.org
 help / color / mirror / Atom feed
From: Josh Durgin <jdurgin@redhat.com>
To: Ken Dreyer <kdreyer@redhat.com>, ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: scoping daemon-helper replacement effort
Date: Fri, 29 Jul 2016 09:55:18 -0700	[thread overview]
Message-ID: <579B8A76.7020606@redhat.com> (raw)
In-Reply-To: <CALqRxCxebmt1Anng1JxT-EU0RH8vFBvHaYs-5dFkgzg-5aWpEA@mail.gmail.com>

On 07/29/2016 09:40 AM, Ken Dreyer wrote:
> daemon-helper predates a lot of things in Ceph, and the further we go
> into systemd-land with things like unprivilged daemons, SELinux, and
> cgroups, the further Teuthology diverges from what our users do. To
> remedy this, I want to retire daemon-helper and have Teuthology tests
> use the normal init system, particularly now that our main supported
> distros are unified around systemd.
>
>  From what I understand, we use daemon-helper in Teuthology to:
>
>    1) start a daemon and eventually stop it with either SIGTERM or
> SIGKILL, depending on whether the Teuthology task has enabled the
> coverage or valgrind options,
>
>    2) send data via STDIN
>
>    3) print some messages when the child crashes
>
> I think we could run the services using the systemd unit files and
> still accomplish #1 and #3.
>
> For #2 (communicating to the daemons via STDIN), how could we
> accomplish this? What sort of things are we writing to the daemons'
> STDIN? I'm having trouble finding examples in ceph-qa-suite.git.

We're not using it to write data to the daemons, but as a way to kill 
them automatically if our ssh connection dies.

With fast reimaging in the works, this will be irrelevant. Even now,
it's not really useful for the usual scheduled jobs where the nodes are
rebooted on failure. So I wouldn't worry about (2).

Josh

  reply	other threads:[~2016-07-29 16:55 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-29 16:40 scoping daemon-helper replacement effort Ken Dreyer
2016-07-29 16:55 ` Josh Durgin [this message]
2016-07-29 16:57   ` Ken Dreyer
2016-07-29 17:22   ` Vasu Kulkarni
2016-07-29 17:25     ` Vasu Kulkarni
2016-07-29 18:39     ` Ken Dreyer
2016-07-29 18:40   ` Gregory Farnum

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=579B8A76.7020606@redhat.com \
    --to=jdurgin@redhat.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=kdreyer@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.