Hello Caiman-ites,

Could I please get a code review for the changes for bugs 6431 and 7077.

Huge thanks! Joe



6431 ssh or rlogin service should be enable in install image for 
automated testing

http://defect.opensolaris.org/bz/show_bug.cgi?id=6431


7077 ICT - remove GRUB entry "text boot

http://defect.opensolaris.org/bz/show_bug.cgi?id=7077

Note regarding 6431:
--------------------
This fix is for the GUI installer. The required changes for the AI 
installer will be tracked in bug 4443. See the bug for more info.

Description of change for 7077:
-------------------------------

The additional "text boot" GRUB menu entry went in to the gate at:

changeset:   350:cbcc5f485007
user:        Joseph J Vlcek <Joseph.Vlcek at Sun.COM>
date:        Thu Nov 13 22:13:04 2008 -0700
summary:     4673 Grub text only mode is required

I reverted the section of the affected code to the way it looked in the 
previous rev.

modules affected:
-----------------

* Distro Construction (grub_setup.py)
* LiveCD Image

testing done:
-------------

Using DC, I built an image with the updated grub_setup.py

I then used lofiadm to add the updated live-fs-root and set_lang to the 
image.

Booted the live image and confirmed:
- A new "ssh enabled" GRUB entry is available.
- ssh is enabled on the live desktop
- the default language settings are made.

To test the changes to ict.py for bug 7077:

On the live desktop I used mount -F lofs... to introduce the updated 
ict.py file, performed a full GUI install and confirmed the "text boot" 
entry was no longer presented by GRUB.

Reply via email to