Re: New jail(8) with configuration files, not yet in head

2011-08-07 Thread joris dedieu
2011/8/7 Simon L. B. Nielsen : > > On 19 Jul 2011, at 19:14, Jamie Gritton wrote: > >> On 07/18/11 13:08, Paul Schenkeveld wrote: >> >>> Although I really like this new functionality, there is one issue that >>> I am concerned about.  Should all this functionality be integrated into >>> the jail(8)

Re: New jail(8) with configuration files, not yet in head

2011-08-07 Thread Simon L. B. Nielsen
On 19 Jul 2011, at 19:14, Jamie Gritton wrote: > On 07/18/11 13:08, Paul Schenkeveld wrote: > >> Although I really like this new functionality, there is one issue that >> I am concerned about. Should all this functionality be integrated into >> the jail(8) command? > > This project came from a

Re: New jail(8) with configuration files, not yet in head

2011-07-19 Thread Jamie Gritton
This project came from a desire to improve the jail startup procedure in rc.d/jail, which remains stuck handling the old fixed-parameter jails. Rather that continue to extend an already unwieldy number of rc.conf shell variables, I opted to add a configuration file like other subsystems use (e.

Re: New jail(8) with configuration files, not yet in head

2011-07-18 Thread Paul Schenkeveld
Hi, On Sun, Jul 03, 2011 at 11:24:57PM -0600, Jamie Gritton wrote: > I'm hoping to get the latest version of jail(8) in before the door slams > shut on 9.0. If anyone wants to take a look at the new code and give it > a spin, it may help to ease RE's mind about my tardiness. The included > di

Re: New jail(8) with configuration files, not yet in head

2011-07-07 Thread Brandon Gooch
On Thu, Jul 7, 2011 at 12:42 AM, Jamie Gritton wrote: > This is code that notes error return codes from processes run under the > jail (e.g. the exec.start script). But in this case, it's reporting an > error from a process that was part of the jail's shutdown. The reason > the command is "(null)"

Re: New jail(8) with configuration files, not yet in head

2011-07-06 Thread Jamie Gritton
The delay you're seeing is $rcshutdown_timeout (default 30 seconds) which will kill rc.shutdown if it doesn't successfully stop all the jail's processes. After that, it will forcibly kill all (jailed) processes. jail(8) has a similar thing going where after any stop scripts have run, it will send

Re: New jail(8) with configuration files, not yet in head

2011-07-06 Thread Jamie Gritton
This is code that notes error return codes from processes run under the jail (e.g. the exec.start script). But in this case, it's reporting an error from a process that was part of the jail's shutdown. The reason the command is "(null)" is this wasn't a command started from jail(8) itself. I've f

Re: New jail(8) with configuration files, not yet in head

2011-07-04 Thread Brandon Gooch
On Mon, Jul 4, 2011 at 4:50 PM, Jamie Gritton wrote: > Oh, never good when "(null)" shows up. I'll see what I'm trying to run > that isn't there. > > - Jamie > > > On 07/04/11 15:36, Brandon Gooch wrote: >> >> I did however notice a minor nit in the output when removing a jail. >> Here's the scena

Re: New jail(8) with configuration files, not yet in head

2011-07-04 Thread Brandon Gooch
On Mon, Jul 4, 2011 at 12:24 AM, Jamie Gritton wrote: > I'm hoping to get the latest version of jail(8) in before the door slams > shut on 9.0.  If anyone wants to take a look at the new code and give it a > spin, it may help to ease RE's mind about my tardiness.  The included diff > applies to th

Re: New jail(8) with configuration files, not yet in head

2011-07-04 Thread Jamie Gritton
Oh, never good when "(null)" shows up. I'll see what I'm trying to run that isn't there. - Jamie On 07/04/11 15:36, Brandon Gooch wrote: I did however notice a minor nit in the output when removing a jail. Here's the scenario: ... Now, when I go to remove the jail: # jail -r ports Stopping