Hi Joe, the fix looks good.
Thank you, Jan Joseph J VLcek wrote: > UPDATE To last post: > -------------------- > > This was not tested with AI as AI does not invoke ICT > remove_install_specific_packages. > > I had confirmed with Sanjay that not testing with AI was OK since this > code path is not taken for AI. > > - - - END UPDATE: - - - > > Hello, > > Can two people please do a small review for the fix for ICT bug: > > 3958 ICT remove_install_specific_packages needs to skip indexing > http://defect.opensolaris.org/bz/show_bug.cgi?id=3958 > > The webrev is available at: > http://cr.opensolaris.org/~joev/bug3958/ > > > NOTE: This fix involves a new pkg uninstall flag,--no-index, which > became available in build 100. Once this fix is applied build 100 will > be required as this ICT will fail if run on pre build 100. > > > Thank you very much, > Joe > > - - - > > > * The modules affected and tested: > > ICT - ict.py > > * Testing done for AI > > Using a build 100 image provide by Sanjay I observed the install to hang > for over an hour on ICT remove_install_specific_packages. > > I rebooted the live Image, applied the updated ict.py using mount -F > lofs, ran the full install. > > * Results: > > The install completed in a reasonable time. I don't know exactly how > long but when I checked back in @30->40 minutes it had finished. This > indicated it did not hang on remove_install_specific_packages for over > an hour. > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >
