LGTM Mary... Thanks,
Darren. On 17/01/2012 18:13, Mary Ding wrote: > Hi: > > I will like to ask for a code review for: > > 7099216 - Need to remove workaround from net-assembly/media-assembly for > 4729220 after bug is fixed in s11u1. > > http://monaco.us.oracle.com/detail.jsf?cr=7099216 > > > Webrev: > > https://cr.opensolaris.org/action/browse/caiman/md5645/7099216 > > > Testing: > > I used DC to build all 5 images with my packages. > > 1. I ran the slim install unit tests and there is only one failure and > it is not related to my fix. It is a regression cause by bugid 7129922. > > 2. Install-pit had use my packages to do AI builds and install and the > results was pass. > > The build is a PASS, you can check the results here: > http://ipit.ie.oracle.com:8080/electron/servlet/ElectronWrapper?option=menu_results_index&Proceed=YES&State_REGEXP=BATCHID:10106 > > And the DC logs and AI images here: > http://ipit-ai.ie.oracle.com/dc_images/s11u1_06-7099216/ > > 3. SST also tested my AI and boot with kmdb many times and there were > no issues: > > So we have tested both AI and text installs using the provided AI image > on almost all platforms (including both x64 and sparc) in our lab and > haven't found any issues related to the mentioned CR. > > 4. I had done install testing with both Livecd and TI iso on sparc and > x86. I booted with and without kmdb and they all work fine. > > > _______________________________________________ > 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
