All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ed Bartosh <ed.bartosh@linux.intel.com>
To: toaster@yoctoproject.org
Subject: [PATCH 0/4] improved process management
Date: Tue, 23 Aug 2016 14:44:40 +0300	[thread overview]
Message-ID: <cover.1471951968.git.ed.bartosh@linux.intel.com> (raw)

Hi,

This is a set of improvements for process management in toaster script.
It includes a bit of cleanup (first 2 patches) and quite serious changes in
the way toaster script starts and stops runbuild and runserver processes:
 - usage of process groups (PGID) instead of PID
 - usage of pid namespaces http://man7.org/linux/man-pages/man7/pid_namespaces.7.html

Both changes should hopefully make toaster to stop running processes more reliably.

One side effect and a partial reason of this work:
 it should fix #7973: stopping toaster kills ALL django apps running on the machine.

Testing instructions:
 - run toaster
 - run build in Toaster UI
 - wait until you see a lot of build-related processes in 'ps x' output
 - stop toaster. expected outcome: no toaster or build-related processes in 'ps x' output

The following changes since commit 34ccad16d40ec0685f405b8d7116250fb58e323d:

  buildinfohelper: discover kernel artifacts correctly (2016-08-23 14:29:32 +0300)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ed/toaster/kill-toaster-pid-namespace-7973
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/kill-toaster-pid-namespace-7973

Ed Bartosh (4):
  toaster: don't kill toaster on start
  toaster: remove handling of .toasterui.pid
  toaster: use process group id to kill processes
  toaster: run builds in pid namespace

 bitbake/bin/toaster | 48 ++++++++++++++++--------------------------------
 1 file changed, 16 insertions(+), 32 deletions(-)

--
Regards,
Ed



             reply	other threads:[~2016-08-23 11:45 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-23 11:44 Ed Bartosh [this message]
2016-08-23 11:44 ` [PATCH 1/4] toaster: don't kill toaster on start Ed Bartosh
2016-08-23 11:44 ` [PATCH 2/4] toaster: remove handling of .toasterui.pid Ed Bartosh
2016-08-23 11:44 ` [PATCH 3/4] toaster: use process group id to kill processes Ed Bartosh
2016-08-23 11:44 ` [PATCH 4/4] toaster: run builds in pid namespace Ed Bartosh

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=cover.1471951968.git.ed.bartosh@linux.intel.com \
    --to=ed.bartosh@linux.intel.com \
    --cc=toaster@yoctoproject.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 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.