Looks good to me Matt. Thanks,
Darren. On 16/08/2011 14:48, Matt Keenan wrote: > Hi, > > Can I get two pairs of eyes to review relatively simple change for bug : > > 7077923 - http://monaco.sfbay.sun.com/detail.jsf?cr=7077923 > > Webrev: > https://cr.opensolaris.org/action/browse/caiman/mattman/7077923/ > > When creating a number of slices and the slice sizes are not given, > Target Selection will attempt to determine the size of each slice based > on the largest gap available. > > If the gap available is less than the min install size an exception was > being thrown, this is fine if the slice is going to be used for a > solaris install, which in quite a few cases will not be the case. > > Remove the min install size check for both slices and partitions as this > particular check is actually done later in when the root pool is > validated to be large enough to install to in __validate_logical(). > > Added unit test to capture this, and also fixed up other unit tests that > had started to fail post the swap/dump putback changes done recently. > > cheers > > Matt > _______________________________________________ > caiman-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
