Re: [zones-discuss] review needed for scratch zone mount fix

2009-10-21 Thread Jerry Jelinek
Edward Pilatowicz wrote: so now i've got a fix for that breakage: http://cr.opensolaris.org/~edp/onnv-zmount/ 6889379 zoneadm mount fails on opensolaris Ed, This looks fine to me. Thanks, Jerry ___ zones-discuss mailing list zones-d

Re: [zones-discuss] review needed for scratch zone mount fix

2009-10-19 Thread Edward Pilatowicz
nice catch. i put it there because of it's similarity with zonecfg_default_privset(), but it doesn't sync with the comments above. i've moved it to the "higher-level routines" section as you suggested. thanks ed On Mon, Oct 19, 2009 at 02:02:13PM -0700, Jordan Vaughan wrote: > Hi Ed, > > usr/src

Re: [zones-discuss] review needed for scratch zone mount fix

2009-10-19 Thread Jordan Vaughan
Hi Ed, usr/src/head/libzonecfg.h This isn't critical, but shouldn't zonecfg_default_brand() be declared somewhere other than the group of "privilege-related functions"? Perhaps it should go under "higher-level routines". Other than that, this looks good to me. Thanks, Jordan On 10/16/09 0

Re: [zones-discuss] review needed for scratch zone mount fix

2009-10-16 Thread Steve Lawrence
This looks ok to me. Sorry for all the conflicts! -Steve On Fri, Oct 16, 2009 at 05:12:06PM -0700, Edward Pilatowicz wrote: > hey all, > > so it seems that in opensolaris b120 i broke scratch zones with the > following fix: > > 9392 native zones should fail to install on opensolaris > >

[zones-discuss] review needed for scratch zone mount fix

2009-10-16 Thread Edward Pilatowicz
hey all, so it seems that in opensolaris b120 i broke scratch zones with the following fix: 9392 native zones should fail to install on opensolaris so now i've got a fix for that breakage: http://cr.opensolaris.org/~edp/onnv-zmount/ 6889379 zoneadm mount fails on opensol