Drew,
LGTM
- Dermot
On 8/9/2011 9:05 PM, Drew Fisher wrote:
Good afternoon!
Could I please get a code review for:
https://cr.opensolaris.org/action/browse/caiman/drewfish/ips66/webrev/
If you looked at yesterday's preemptive review, very little has
changed with the exception of moving the IPS version in
install_common/__init__.py to 66 and the small rewrite of the
get_pkg_list functionality in installadm/image.py based on Shawn's
comments.
I ran all the unittests for installadm, js2ai, install_common,
install_ict, and install_transfer. I'm not seeing anything that's not
already being picked up by Hudson (I hope...)
I also confirmed the fix by:
- constructing an AI ISO and installing it completely to a VM guest.
(in progress, but it's past where the code would fail for a version
mismatch).
- creating a zone with the same bits
- testing Sue's image-from-a-package flags to installadm
Once I get a couple of reviews and Mary's blessing to push, I will do so.
Thanks!
-Drew
_______________________________________________
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
