Re: [zones-discuss] quick bug fix webrev...

2009-11-20 Thread Edward Pilatowicz
On Thu, Nov 19, 2009 at 09:16:45PM -0800, Edward Pilatowicz wrote: > hey all, > > i need a review for the following bugfix: > > http://cr.opensolaris.org/~edp/onnv-zmount3/ > 6901952 zoneadm fails with "unable to determine default brand" > thanks for all the reviews. also, for the morbidly curiou

Re: [zones-discuss] quick bug fix webrev...

2009-11-20 Thread Edward Pilatowicz
On Thu, Nov 19, 2009 at 11:12:48PM -0800, Dan Price wrote: > On Thu 19 Nov 2009 at 09:16PM, Edward Pilatowicz wrote: > > hey all, > > > > i need a review for the following bugfix: > > > > http://cr.opensolaris.org/~edp/onnv-zmount3/ > > 6901952 zoneadm fails with "unable to determine default brand"

Re: [zones-discuss] quick bug fix webrev...

2009-11-20 Thread Jerry Jelinek
Edward Pilatowicz wrote: hey all, i need a review for the following bugfix: http://cr.opensolaris.org/~edp/onnv-zmount3/ 6901952 zoneadm fails with "unable to determine default brand" Ed, This looks fine to me. Jerry ___ zones-discuss mailing list

Re: [zones-discuss] quick bug fix webrev...

2009-11-19 Thread Dan Price
On Thu 19 Nov 2009 at 09:16PM, Edward Pilatowicz wrote: > hey all, > > i need a review for the following bugfix: > > http://cr.opensolaris.org/~edp/onnv-zmount3/ > 6901952 zoneadm fails with "unable to determine default brand" In the comment: zones -> zone's. Is "default brand" actually a mea

Re: [zones-discuss] quick bug fix webrev...

2009-11-19 Thread Zoram Thanga
Hi Ed, lgtm. Thanks, Zoram On 11/20/09 10:46, Edward Pilatowicz wrote: hey all, i need a review for the following bugfix: http://cr.opensolaris.org/~edp/onnv-zmount3/ 6901952 zoneadm fails with "unable to determine default brand" thanks ed ___ zon

Re: [zones-discuss] quick bug fix webrev...

2009-11-19 Thread Jordan Vaughan
On 11/19/09 09:16 PM, Edward Pilatowicz wrote: hey all, i need a review for the following bugfix: http://cr.opensolaris.org/~edp/onnv-zmount3/ 6901952 zoneadm fails with "unable to determine default brand" thanks ed ___ zones-discuss mailing list zon

[zones-discuss] quick bug fix webrev...

2009-11-19 Thread Edward Pilatowicz
hey all, i need a review for the following bugfix: http://cr.opensolaris.org/~edp/onnv-zmount3/ 6901952 zoneadm fails with "unable to determine default brand" thanks ed ___ zones-discuss mailing list zones-discuss@opensolaris.org