Hello,

Can two people please do a code review for a fix for bug:

4871 ICT failures are not reported as a failed installation
http://defect.opensolaris.org/bz/show_bug.cgi?id=4871


The webrev is available at:
http://cr.opensolaris.org/~joev/bug4871/


* The modules affected and tested:

liborchestrator
install-finish

* Testing done for GUI Install

I booted a 101 live Image and applied the updated library using 
LD_LIBRARY_PATH, I copied the updated install-finish script onto /sbin

I fabricated a failure in ict.py with a hard coded error to have one of 
the ICT return a failure.

I used mount -F lofs so the ict.py containing the error would be used.

I then repeated the test using the version of ict.py we deliver to 
ensure the installation succeeded.

* Results:

When using the version of ict.py with a hard coded error the 
installation reported errors.

When using the version of ict.py we deliver, without a hard coded error, 
the installation succeeded.


* Testing done for AI

No AI testing was performed.


Thank you,
Joe


Reply via email to