Thank you all for the comments. We have updated all of your concerns that you mentioned into our new webrev: http://cr.opensolaris.org/~chris_n/netboot_text_install_v2/
and the updated parts located at: http://cr.opensolaris.org/~chris_n/netboot_text_install_part/ To address specific concerns: - ict.py: The check was added to give additional debugging statements. Install finish was also now modified so that the packages to remove are the exact same for text and auto install. This check will prevent an error later down from trying to uninstall a package that is not there. - We made modifications to installadm and to the finalizer script that generates .image_info so that if it is a net-text image installadm would generate a new GRUB menu to represent that the default choice is now text install. We were looking for feedback if this is the right way to go. If we change the GRUB menu some users may get confused when they boot it up for the sole purpose of getting dropped into a console and see <release> Text Install. However a user who wants to use it to do a text install may also get confused if it doesn't say anything except <release> boot image. Does anybody have an alternative or feel strongly one way or the other? We would appreciate any additional feedback. Thanks, Chris & Andrew On Tue, Aug 31, 2010 at 1:35 PM, Chris Navrides <[email protected]> wrote: > Hi all, > > Could we please get a code review of the modifications made to > enable 6971585 Network Text Install > > This will enable a client over the network to either do an Automated > Install or a Text Install. The default in the grub menu will lead to the > text install menu. For sparc if no "-install" flag is set then it will also > default to the text-install menu. > > Webrev: > http://cr.opensolaris.org/~chris_n/netboot_text_instal<http://cr.opensolaris.org/~chris_n/netboot_text_install/> > l/ <http://cr.opensolaris.org/~chris_n/netboot_text_install/> > > We have done testing with both x86 and sparc using the test bed on the > 10.10.45 subnet. There are no changes done to installadm on the server side. > > Thanks, > Chris & Andrew >
_______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
