Date: Fri, 25 May 2012 00:59:31 +0200 From: Mateusz Guzik <mjguzik@gmail.com> To: Jamie Gritton <jamie@FreeBSD.org> Cc: Mike Jakubik <mike.jakubik@intertainservices.com>, "freebsd-stable@freebsd.org" <freebsd-stable@FreeBSD.org> Subject: Re: Jail startup/shutdown broken on latest 9-STABLE Message-ID: <20120524225931.GD17579@dft-labs.eu> In-Reply-To: <4FBEBA5C.7050804@FreeBSD.org> References: <1337887134.1908.20.camel@mike-PC> <20120524212219.GA17579@dft-labs.eu> <1337897210.1908.24.camel@mike-PC> <20120524221353.GB17579@dft-labs.eu> <1337898015.1908.27.camel@mike-PC> <20120524223004.GC17579@dft-labs.eu> <4FBEBA5C.7050804@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, May 24, 2012 at 04:46:52PM -0600, Jamie Gritton wrote: > I'll get the patch to jail(8) in - thanks for catching that. But I > wonder about the patch to /etc/rc.d/jail. It looks correct, but I'm > going to see if it's /etc/rc.d/jail that needs changing, or if my recent > changes to jail(8) have changed the order in which things are written. > Yeah, was not sure whether I should change the order or the script. :) Would not it be better to just create empty persistent jail as first step? Since in this case only one line will be generated (jid), rc.d script will be able to just take the output - this seems much less fragile than the current method. Then of course it would proceed with jexec running /etc/rc and in the end drop persist flag. It looks like rc.d script still uses old syntax so this actually may be less trivial than it sounds. That being said, if this is idea sounds okay, I can try to come up with a patch this weekend. -- Mateusz Guzik <mjguzik gmail.com>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120524225931.GD17579>