The reason behind this is to make these functions available for using
them in rescue mode too.
Note to reviewers, the code of addDriveDialog() and friends has not
changed a single letter it was only moved to the new textw/add_drive_text.py
file (and some boilerplate was added on the top to make it into a new class).
---
textw/add_drive_text.py | 180 +++++++++++++++++++++++++++++++++++++++++++++++
textw/partition_text.py | 165 +------------------------------------------
2 files changed, 183 insertions(+), 162 deletions(-)
create mode 100644 textw/add_drive_text.py
diff --git a/textw/add_drive_text.py b/textw/add_drive_text.py
new file mode 100644
index 0000000..b8edc29
--- /dev/null
+++ b/textw/add_drive_text.py
@@ -0,0 +1,180 @@
+import iutil
+import network
+import storage.iscsi
+import storage.fcoe
+import storage.zfcp
+from snack import *
+from constants_text import *
+from constants import *
+
+import gettext
+_ = lambda x: gettext.ldgettext("anaconda", x)
+
+import logging
+log = logging.getLogger("anaconda")
+
+class addDriveDialog(object):
+ def __init__(self, anaconda):
+ self.anaconda = anaconda
+
+ def addDriveDialog(self, screen):
+ newdrv = []
+ if storage.iscsi.has_iscsi():
+ newdrv.append("Add iSCSI target")
+ if iutil.isS390():
+ newdrv.append( "Add zFCP LUN" )
+ if storage.fcoe.has_fcoe():
+ newdrv.append("Add FCoE SAN")
+
+ if len(newdrv) == 0:
+ return INSTALL_BACK
+
+ (button, choice) = ListboxChoiceWindow(screen,
+ _("Advanced Storage Options"),
+ _("How would you like to modify "
+ "your drive configuration?"),
+ newdrv,
+ [ TEXT_OK_BUTTON, TEXT_BACK_BUTTON],
+ width=55, height=3)
+
+ if button == TEXT_BACK_CHECK:
+ return INSTALL_BACK
+ if newdrv[choice] == "Add zFCP LUN":
+ try:
+ return self.addZFCPDriveDialog(screen)
+ except ValueError, e:
+ ButtonChoiceWindow(screen, _("Error"), str(e))
+ return INSTALL_BACK
+ elif newdrv[choice] == "Add FCoE SAN":
+ try:
+ return self.addFcoeDriveDialog(screen)
+ except ValueError, e:
+ ButtonChoiceWindow(screen, _("Error"), str(e))
+ return INSTALL_BACK
+ else:
+ try:
+ return self.addIscsiDriveDialog(screen)
+ except (ValueError, IOError), e:
+ ButtonChoiceWindow(screen, _("Error"), str(e))
+ return INSTALL_BACK
+
+ def addZFCPDriveDialog(self, screen):
+ (button, entries) = EntryWindow(screen,
+ _("Add FCP Device"),
+ _("zSeries machines can access industry-standard SCSI devices via Fibre Channel (FCP). You need to provide a 16 bit device number, a 64 bit World Wide Port Name (WWPN), and a 64 bit FCP LUN for each device."),
+ prompts = [ "Device number",
+ "WWPN",
+ "FCP LUN" ] )
+ if button == TEXT_CANCEL_CHECK:
+ return INSTALL_BACK
+
+ devnum = entries[0].strip()
+ wwpn = entries[1].strip()
+ fcplun = entries[2].strip()
+ try:
+ storage.zfcp.ZFCP().addFCP(devnum, wwpn, fcplun)
+ except ValueError, e:
+ log.warn(str(e)) # alternatively popup error dialog instead
+
+ return INSTALL_OK
+
+ def addFcoeDriveDialog(self, screen):
+ netdevs = self.anaconda.network.available()
+ devs = netdevs.keys()
+ devs.sort()
+
+ if not devs:
+ ButtonChoiceWindow(screen, _("Error"),
+ _("No network cards present."))
+ return INSTALL_BACK
+
+ grid = GridFormHelp(screen, _("Add FCoE SAN"), "fcoeconfig",
+ 1, 4)
+
+ tb = TextboxReflowed(60,
+ _("Select which NIC is connected to the FCoE SAN."))
+ grid.add(tb, 0, 0, anchorLeft = 1, padding = (0, 0, 0, 1))
+
+ interfaceList = Listbox(height=len(devs), scroll=1)
+ for dev in devs:
+ hwaddr = netdevs[dev].get("HWADDR")
+ if hwaddr:
+ desc = "%s - %.50s" % (dev, hwaddr)
+ else:
+ desc = dev
+
+ interfaceList.append(desc, dev)
+
+ interfaceList.setCurrent(devs[0])
+ grid.add(interfaceList, 0, 1, padding = (0, 1, 0, 0))
+
+ dcbCheckbox = Checkbox(_("Use DCB"), 1)
+ grid.add(dcbCheckbox, 0, 2, anchorLeft = 1)
+
+ buttons = ButtonBar(screen, [TEXT_OK_BUTTON, TEXT_BACK_BUTTON] )
+ grid.add(buttons, 0, 3, anchorLeft = 1, growx = 1)
+
+ result = grid.run()
+ if buttons.buttonPressed(result) == TEXT_BACK_CHECK:
+ screen.popWindow()
+ return INSTALL_BACK
+
+ nic = interfaceList.current()
+ dcb = dcbCheckbox.selected()
+
+ storage.fcoe.fcoe().addSan(nic=nic, dcb=dcb,
+ intf=self.anaconda.intf)
+
+ screen.popWindow()
+ return INSTALL_OK
+
+ def addIscsiDriveDialog(self, screen):
+ if not network.hasActiveNetDev():
+ ButtonChoiceWindow(screen, _("Error"),
+ "Must have a network configuration set up "
+ "for iSCSI config. Please boot with "
+ "'linux asknetwork'")
+ return INSTALL_BACK
+
+ (button, entries) = EntryWindow(screen,
+ _("Configure iSCSI Parameters"),
+ _("To use iSCSI disks, you must provide the address of your iSCSI target and the iSCSI initiator name you've configured for your host."),
+ prompts = [ _("Target IP Address"),
+ _("iSCSI Initiator Name"),
+ _("CHAP username"),
+ _("CHAP password"),
+ _("Reverse CHAP username"),
+ _("Reverse CHAP password") ])
+ if button == TEXT_CANCEL_CHECK:
+ return INSTALL_BACK
+
+ (user, pw, user_in, pw_in) = entries[2:]
+
+ target = entries[0].strip()
+ try:
+ count = len(target.split(":"))
+ idx = target.rfind("]:")
+ # Check for IPV6 [IPV6-ip]:port
+ if idx != -1:
+ ip = target[1:idx]
+ port = target[idx+2:]
+ # Check for IPV4 aaa.bbb.ccc.ddd:port
+ elif count == 2:
+ idx = target.rfind(":")
+ ip = target[:idx]
+ port = target[idx+1:]
+ else:
+ ip = target
+ port = "3260"
+ network.sanityCheckIPString(ip)
+ except network.IPMissing, msg:
+ raise ValueError, msg
+ except network.IPError, msg:
+ raise ValueError, msg
+
+ iname = entries[1].strip()
+ if not storage.iscsi.iscsi().initiatorSet:
+ storage.iscsi.iscsi().initiator = iname
+ storage.iscsi.iscsi().addTarget(ip, port, user, pw, user_in, pw_in)
+
+ return INSTALL_OK
diff --git a/textw/partition_text.py b/textw/partition_text.py
index 92036be..193ff21 100644
--- a/textw/partition_text.py
+++ b/textw/partition_text.py
@@ -34,6 +34,7 @@ from partIntfHelpers import *
from snack import *
from constants_text import *
from constants import *
+from add_drive_text import addDriveDialog
import gettext
_ = lambda x: gettext.ldgettext("anaconda", x)
@@ -131,7 +132,8 @@ class PartitionTypeWindow:
screen.popWindow()
if rc == "F2":
- if self.addDriveDialog(screen) != INSTALL_BACK:
+ addDialog = addDriveDialog(anaconda)
+ if addDialog.addDriveDialog(screen) != INSTALL_BACK:
anaconda.storage.reset()
anaconda.bootloader.updateDriveList()
continue
@@ -158,164 +160,3 @@ class PartitionTypeWindow:
return INSTALL_OK
- def addDriveDialog(self, screen):
- newdrv = []
- if storage.iscsi.has_iscsi():
- newdrv.append("Add iSCSI target")
- if iutil.isS390():
- newdrv.append( "Add zFCP LUN" )
- if storage.fcoe.has_fcoe():
- newdrv.append("Add FCoE SAN")
-
- if len(newdrv) == 0:
- return INSTALL_BACK
-
- (button, choice) = ListboxChoiceWindow(screen,
- _("Advanced Storage Options"),
- _("How would you like to modify "
- "your drive configuration?"),
- newdrv,
- [ TEXT_OK_BUTTON, TEXT_BACK_BUTTON],
- width=55, height=3)
-
- if button == TEXT_BACK_CHECK:
- return INSTALL_BACK
- if newdrv[choice] == "Add zFCP LUN":
- try:
- return self.addZFCPDriveDialog(screen)
- except ValueError, e:
- ButtonChoiceWindow(screen, _("Error"), str(e))
- return INSTALL_BACK
- elif newdrv[choice] == "Add FCoE SAN":
- try:
- return self.addFcoeDriveDialog(screen)
- except ValueError, e:
- ButtonChoiceWindow(screen, _("Error"), str(e))
- return INSTALL_BACK
- else:
- try:
- return self.addIscsiDriveDialog(screen)
- except (ValueError, IOError), e:
- ButtonChoiceWindow(screen, _("Error"), str(e))
- return INSTALL_BACK
-
- def addZFCPDriveDialog(self, screen):
- (button, entries) = EntryWindow(screen,
- _("Add FCP Device"),
- _("zSeries machines can access industry-standard SCSI devices via Fibre Channel (FCP). You need to provide a 16 bit device number, a 64 bit World Wide Port Name (WWPN), and a 64 bit FCP LUN for each device."),
- prompts = [ "Device number",
- "WWPN",
- "FCP LUN" ] )
- if button == TEXT_CANCEL_CHECK:
- return INSTALL_BACK
-
- devnum = entries[0].strip()
- wwpn = entries[1].strip()
- fcplun = entries[2].strip()
- try:
- storage.zfcp.ZFCP().addFCP(devnum, wwpn, fcplun)
- except ValueError, e:
- log.warn(str(e)) # alternatively popup error dialog instead
-
- return INSTALL_OK
-
- def addFcoeDriveDialog(self, screen):
- netdevs = self.anaconda.network.available()
- devs = netdevs.keys()
- devs.sort()
-
- if not devs:
- ButtonChoiceWindow(screen, _("Error"),
- _("No network cards present."))
- return INSTALL_BACK
-
- grid = GridFormHelp(screen, _("Add FCoE SAN"), "fcoeconfig",
- 1, 4)
-
- tb = TextboxReflowed(60,
- _("Select which NIC is connected to the FCoE SAN."))
- grid.add(tb, 0, 0, anchorLeft = 1, padding = (0, 0, 0, 1))
-
- interfaceList = Listbox(height=len(devs), scroll=1)
- for dev in devs:
- hwaddr = netdevs[dev].get("HWADDR")
- if hwaddr:
- desc = "%s - %.50s" % (dev, hwaddr)
- else:
- desc = dev
-
- interfaceList.append(desc, dev)
-
- interfaceList.setCurrent(devs[0])
- grid.add(interfaceList, 0, 1, padding = (0, 1, 0, 0))
-
- dcbCheckbox = Checkbox(_("Use DCB"), 1)
- grid.add(dcbCheckbox, 0, 2, anchorLeft = 1)
-
- buttons = ButtonBar(screen, [TEXT_OK_BUTTON, TEXT_BACK_BUTTON] )
- grid.add(buttons, 0, 3, anchorLeft = 1, growx = 1)
-
- result = grid.run()
- if buttons.buttonPressed(result) == TEXT_BACK_CHECK:
- screen.popWindow()
- return INSTALL_BACK
-
- nic = interfaceList.current()
- dcb = dcbCheckbox.selected()
-
- storage.fcoe.fcoe().addSan(nic=nic, dcb=dcb,
- intf=self.anaconda.intf)
-
- screen.popWindow()
- return INSTALL_OK
-
- def addIscsiDriveDialog(self, screen):
- if not network.hasActiveNetDev():
- ButtonChoiceWindow(screen, _("Error"),
- "Must have a network configuration set up "
- "for iSCSI config. Please boot with "
- "'linux asknetwork'")
- return INSTALL_BACK
-
- (button, entries) = EntryWindow(screen,
- _("Configure iSCSI Parameters"),
- _("To use iSCSI disks, you must provide the address of your iSCSI target and the iSCSI initiator name you've configured for your host."),
- prompts = [ _("Target IP Address"),
- _("iSCSI Initiator Name"),
- _("CHAP username"),
- _("CHAP password"),
- _("Reverse CHAP username"),
- _("Reverse CHAP password") ])
- if button == TEXT_CANCEL_CHECK:
- return INSTALL_BACK
-
- (user, pw, user_in, pw_in) = entries[2:]
-
- target = entries[0].strip()
- try:
- count = len(target.split(":"))
- idx = target.rfind("]:")
- # Check for IPV6 [IPV6-ip]:port
- if idx != -1:
- ip = target[1:idx]
- port = target[idx+2:]
- # Check for IPV4 aaa.bbb.ccc.ddd:port
- elif count == 2:
- idx = target.rfind(":")
- ip = target[:idx]
- port = target[idx+1:]
- else:
- ip = target
- port = "3260"
- network.sanityCheckIPString(ip)
- except network.IPMissing, msg:
- raise ValueError, msg
- except network.IPError, msg:
- raise ValueError, msg
-
- iname = entries[1].strip()
- if not storage.iscsi.iscsi().initiatorSet:
- storage.iscsi.iscsi().initiator = iname
- storage.iscsi.iscsi().addTarget(ip, port, user, pw, user_in, pw_in)
-
- return INSTALL_OK
--
1.7.0.1
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
