Dermot,
Looks good. My only comment is on the comments. Please
correct the TargetController at line 644:
644 - slices added by TargetControlerr may be present
No additional review is necessary from my standpoint.
Thanks,
John
On Sep 15, 2011, at 9:22 AM, Dermot McCluskey wrote:
> I'm seeking b175 putback approval and two code reviewers for:
>
> http://monaco.sfbay.sun.com/detail.jsf?cr=7090232
> GUI installer does not create rpool on whole partition
> if previous slices exists in partition
>
> Webrev:
> https://cr.opensolaris.org/action/browse/caiman/dermot/7090232/
>
> Details:
> This is an oversight in gui-install: in certain cases if there's
> an existing Solaris partition, and it contains slices, gui-install/
> TargetController do not wipe those slices and lay down our
> own preferred layout (TextInstall does this explicitly).
> If the user makes any changes to the Solaris partition (type,
> size) the child slice details are lost and gui-install notices this
> and re-instates slice 0 at validation time. This fix takes
> advantage of this by always wiping the slices and creating
> slice 0 at validation time. This results in a consistent layout
> after validation, regardless of whether:
> - the partition has an existing slice layout
> - TargetController created a slice layout
> - the slice layout has previously been removed
>
> Testing done:
> Used format to create an "unfavorable" slice layout on the
> Solaris2 partition of the target disk. Ran gui-install.
> Before fix: if no changes are made to the partition, the
> old slice layout is retained.
> After fix: regardless of whether changes are made, the
> correct slice 0 layout is imposed.
>
>
> - Dermot
>
> _______________________________________________
> 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
