From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.io/gmane.comp.sysutils.supervision.general/2528 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: "Casper Ti. Vector" Newsgroups: gmane.comp.sysutils.supervision.general Subject: Re: Update on the progress of slew development Date: Wed, 20 Mar 2019 19:51:34 +0800 Message-ID: <20190320115134.GA14073@CasperVector> References: <20190317132532.GA22622@CasperVector> <20190317153002.52c28cf7@dickeberta> <20190319124239.GA26884@CasperVector> <20190319165853.6bb9f44a@flunder> <20190320051439.GA7636@CasperVector> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="171310"; mail-complaints-to="usenet@blaine.gmane.org" User-Agent: Mutt/1.11.3 (2019-02-01) To: supervision@list.skarnet.org Original-X-From: supervision-return-2118-gcsg-supervision=m.gmane.org@list.skarnet.org Wed Mar 20 12:51:45 2019 Return-path: Envelope-to: gcsg-supervision@m.gmane.org Original-Received: from alyss.skarnet.org ([95.142.172.232]) by blaine.gmane.org with smtp (Exim 4.89) (envelope-from ) id 1h6Zl2-000iTQ-UA for gcsg-supervision@m.gmane.org; Wed, 20 Mar 2019 12:51:45 +0100 Original-Received: (qmail 31602 invoked by uid 89); 20 Mar 2019 11:52:09 -0000 Mailing-List: contact supervision-help@list.skarnet.org; run by ezmlm Original-Sender: Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Original-Received: (qmail 31595 invoked from network); 20 Mar 2019 11:52:08 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:date:to:subject:message-id:mail-followup-to:references :mime-version:content-disposition:in-reply-to:user-agent; bh=pqb9tr11D6D64x0NwQJHczY66zn48zjrDpO8Uw9sxxc=; b=CsZXfriOSQhdzZlc6+4LYf4Z1JuDLcIU1efa0twsS6K9jukIacQQW07QMLIZ8lwgvU cb66jEj8tWsKHCv8pw1eeBraPyzRy2ciUKDlqhfrQh5CaPCXVf01DbIPB4fRIZxu86sW SK8sShwBWqoCV5CuyDn+q4CDa3vH96dTweYaQPhAIpJsxNC5w/N/0l1KO5yROwcdQRLv cTsy2AZexXfZWvLKzotS1NJXNwdGvhKJewsKZKYTItbCTQ0xZWI0/0GXLsNTCEGF6rkZ WudqnpiuXwVYCxM7vQQIaMWDMvyazJqduUFcTGxsnBtcRqtoNN0OJwSWbowDSBJY2baR BZjQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:date:to:subject:message-id:mail-followup-to :references:mime-version:content-disposition:in-reply-to:user-agent; bh=pqb9tr11D6D64x0NwQJHczY66zn48zjrDpO8Uw9sxxc=; b=OfAXMtu/AnYwECCj64ixrLxJeoQH0AY/7DyfVfpw4KeKDGSpN3oDD0l6r+KLaIRXx5 Ip0tY+NWOsbWb4GRsVofaJqqNGkrYZ1eKzTCMa1znNXMrFmkeupVgTX2dTOcWWKFoA1h ared7U9agcWaPnnP5XJD6kxE2fjjD6Y7AgGv44I6LsiXF345f3imA0kTM2dgLhKZtyKh 0dHMF7ZlGwI9/KbeeRE4tUBOrytIjrpcQ6Qph9G61XDaoz9JL8S+BLdFLV8plr+D9rav pk2M1ZPpoRq0VV9dvk5Xm9E6PbQDAov+rnpXPVGuEX9Ro1z3XBbDYqfH5cHLiM8Z3Zf2 O/uQ== X-Gm-Message-State: APjAAAW1Bx/rJdAocVYyE+KsMCuzcvJ5VhQTvoBWdbntV8wwyEDc5R+c 9WuZswx/rpIkB2FOweFUsWR4amt+MB4= X-Google-Smtp-Source: APXvYqyGZFbqG2VcLxlxwArcuUKK5ECKCUI1Cn8LMO53p8K5cioVvnjQCj/4axXiffPZFpiSujTFaw== X-Received: by 2002:a17:906:4f8e:: with SMTP id o14mr16918635eju.198.1553082701350; Wed, 20 Mar 2019 04:51:41 -0700 (PDT) X-Google-Original-From: "Casper Ti. Vector" Mail-Followup-To: supervision@list.skarnet.org Content-Disposition: inline In-Reply-To: <20190320051439.GA7636@CasperVector> Xref: news.gmane.org gmane.comp.sysutils.supervision.general:2528 Archived-At: On Wed, Mar 20, 2019 at 01:14:39PM +0800, Casper Ti. Vector wrote: > Fixed (provided that sysvinit killall(8) is included in the container) > in commit 3f246b20 the day before yesterday :) I forgot to note that sending SIGHUP is unnecessary, and `rc.halt' did this previously because of my misunderstanding of how the catch-all logger was stopped. `s6-svc -X /run/service/s6-svscan.log' lets the corresponding `s6-supervise' process immediately close its fds 0/1/2 and wait to exit after the catch-all logger exits, so the logger's input is only connected to outputs of unlogged services (except for itself) not stopped in `rc.fin' (which should not exist if `s6-rc -d change ...' succeeded) and all stray processes that has inherited fds from parents that directly wrote to the catch-all logger. If no such processes remained at this moment, the SIGHUP (intended to stop the catch-all logger because it is started with the `-b' option of `s6-log') would have no effect; if they existed, sending SIGHUP immediately after SIGTERM/SIGCONT would probably result in lost logs for the final outputs of them, and simply letting the impending SIGKILL kill the logger would be better. Additionally, I have just realised that these final logs were not considered by the current version of `rc.halt', and then pushed commit 5911f892 to fix this. > something like > : /bin/importas PIA PIA > : /usr/bin/env -i PATH=/usr/sbin:/usr/bin:/sbin:/bin PIA=${PIA} Note that here correctness of the environment is ensured whether $PIA is non-empty, empty or unset, because in the last case `importas' would simply delete `PIA=${PIA}' from the command line to be exec()ed. (Kudos to Laurent, of course :) -- My current OpenPGP key: RSA4096/0x227E8CAAB7AA186C (expires: 2020.10.19) 7077 7781 B859 5166 AE07 0286 227E 8CAA B7AA 186C