Karen,
Thanks for looking. I looked again at summary.py and agree. The 2
changes in the file don't really need to be there so I'll fix the file.
-Drew
On 4/4/12 2:37 PM, Karen Tung wrote:
Hi Drew,
I reviewed all the files related to text installer. I only have 1
comment:
usr/src/cmd/text-install/summary.py:
I don't think there's any change in this file. Is it included by
accident?
Thanks,
--Karen
On 04/ 4/12 09:52 AM, Drew Fisher wrote:
Good morning!
Dermot and I would like to get a second-round code review for the
iSCSI project.
https://cr.opensolaris.org/action/browse/caiman/drewfish/iSCSI2/
Due to the large amount of time between code review cycles and the
large amount of code pushed for the UEFI project, an incremental
webrev is difficult to cobble together. If people really need one we
can try to get one created but since the majority of the first-round
comments were nits or small requests (except for Jesse's ...), I'm
not sure how much value there would be for an incremental.
Jesse's request (for those not interested in scrolling back in the
email history) was to make the Target LUN optional. If the LUN is
specified, only show the one LUN. If the LUN was omitted show all of
the LUNs on the Target.
These bits are current in a second round of QE testing but I'm not
expecting any large issues to come out of the test cycle.
I'd like to get the review by Wednesday, April 11th. I'd like to ask
the same people to look again at specific sections:
Karen: Could I get you to look specifically at the text-installer
changes?
Niall/John: Could I get you both to look specifically at the GUI
changes?
Jesse: Can you look at code in $SRC/lib/install_target as that's
where all the calls to iscsiadm and libima.so exist
If anybody needs screenshots or ISOs, let me know.
Thanks!
-Drew and Dermot
On 2/15/12 10:30 AM, Drew Fisher wrote:
Good morning!
Dermot and I would like to get a first-round code review for the
following PSARC/CRs:
PSARC/2012/023
<http://psarc.us.oracle.com/Archives/CaseLog/arc/PSARC/2012/023> Interactive
Installation to iSCSI
6974246 <http://monaco.us.oracle.com/detail.jsf?cr=6974246> Automated Install
should provide mechanism for setting iSCSI initiator-id-node
7004719 <http://monaco.us.oracle.com/detail.jsf?cr=7004719> Opensolaris
LiveCD installation should give a GUI for installing Solaris onto iSCSI LUN
7004720 <http://monaco.us.oracle.com/detail.jsf?cr=7004720> Opensolaris text
installer should give a screen for installing solaris onto iscsi lun
7114789 <http://monaco.us.oracle.com/detail.jsf?cr=7114789> unlabelled iSCSI
drives and setting an iSCSI boot-device require special handling on SPARC
7121245 <http://monaco.us.oracle.com/detail.jsf?cr=7121245> iscsi paths don't
translate to bootable OBP paths/strings
7132111 <http://monaco.us.oracle.com/detail.jsf?cr=7132111> sample ai
manifest file does not mention setting whole_disk in disk target section
7132457 <http://monaco.us.oracle.com/detail.jsf?cr=7132457> Race condition in
AI involving the target discovery and the check for the new OS Device Name
7145512 <http://monaco.us.oracle.com/detail.jsf?cr=7145512> pulling iSCSI
information from the DHCP server does not support iSCSI boot
https://cr.opensolaris.org/action/browse/caiman/drewfish/iSCSI/
Karen: Could I get you to look specifically at the text-installer
changes?
Niall/John: Could I get you both to look specifically at the GUI
changes?
Jesse: Can you look at code in $SRC/lib/install_target as that's
where all the calls to iscsiadm and libima.so exist
Outstanding issues:
- QE has not yet started a test cycle. I've been working with
Angela Li on this. I'm hoping to start a test cycle soon
- ai_manifest.4 is still being edited. I've emailed proposed
changes to Alta who will translate them into actual English
- The fix for 7132457 is slated to be tested by PIT soon for
confirmation.
ISO / USB access can be found at Hudson:
http://indiana-build.us.oracle.com/view/iSCSI/ Choose the job for
the installer you want to look at and you'll see links along the
job's page pointing to the latest ISO / USB image
Screenshots of the GUI and Text-Installer (for those not interested
in firing up an entire ISO) can be found here:
http://mox.us.oracle.com/code/screenshots/gui/
http://mox.us.oracle.com/code/screenshots/ti/
If people want to try out the ISOs, let me know and I can provide
target IP/LUN information to use.
Thanks!
-Drew and Dermot
_______________________________________________
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
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
