Dave Marker has already reviewed this but with the build 17-20 rule I
need to get 2 reviews, so can I get an additional code review for the
following bug:
CR: http://monaco.sfbay/detail.jsf?cr=7147174
There is an additional change that isn't noted in the webrev. I moved
line 194 in target_selection.py left by one tab. The function,
__pretty_print_disk, if called with a list of disks would only create a
string with the first disk in the list. The return statement needed to
be moved out of the loop.
Webrev: https://cr.opensolaris.org/action/browse/caiman/hshaw/7147174_2/
All of the target unit tests were run on both Sparc and x86. All of the
tests passed. Unit tests were added to both the Sparc and x86 target
selection tests to verify the fix.
A number of AI manifests were written to test different sets of disk
specifications in the manifest. They were run using auto-install and
stopping before the target instantiation checkpoint. These manifests
have been sent to QE to be used as a basis for a new test suite that
will be able to test a large set of manifests that describe a range of
disk environments.
Full Installs were run with the default manifest and 2 of the test
manifests to ensure proper layout and installation.
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
