[anaconda] Make anaconda changes necessary for libblockdev s390 plugin.
Samantha N. Bueno
sbueno+anaconda at redhat.com
Wed May 13 18:16:25 UTC 2015
---
pyanaconda/ui/gui/spokes/advstorage/dasd.py | 10 ++++++----
pyanaconda/ui/gui/spokes/advstorage/zfcp.py | 12 +++++++-----
pyanaconda/ui/gui/spokes/lib/dasdfmt.py | 8 ++++----
pyanaconda/ui/gui/spokes/storage.py | 12 ++++++------
pyanaconda/ui/tui/spokes/storage.py | 8 ++++----
5 files changed, 27 insertions(+), 23 deletions(-)
diff --git a/pyanaconda/ui/gui/spokes/advstorage/dasd.py b/pyanaconda/ui/gui/spokes/advstorage/dasd.py
index 88e1a01..38748a8 100644
--- a/pyanaconda/ui/gui/spokes/advstorage/dasd.py
+++ b/pyanaconda/ui/gui/spokes/advstorage/dasd.py
@@ -21,6 +21,8 @@
from pyanaconda.ui.gui import GUIObject
from pyanaconda.ui.gui.utils import gtk_action_nowait
+from gi.repository import BlockDev as blockdev
+from gi.repository import GLib
from blivet.devicelibs.dasd import sanitize_dasd_dev_input, online_dasd
@@ -83,7 +85,7 @@ class DASDDialog(GUIObject):
self._conditionNotebook.set_current_page(1)
try:
- device = sanitize_dasd_dev_input(self._deviceEntry.get_text())
+ device = blockdev.s390.sanitize_dev_input(self._deviceEntry.get_text())
except ValueError as e:
_config_error = str(e)
self.builder.get_object("deviceErrorLabel").set_text(_config_error)
@@ -126,8 +128,8 @@ class DASDDialog(GUIObject):
"""
# attempt to add the device
try:
- online_dasd(device)
+ blockdev.s390.dasd_online(device)
self._update_devicetree = True
- except ValueError as e:
- self._discoveryError = str(e)
+ except GLib.GError as err:
+ self._discoveryError = str(err)
return
diff --git a/pyanaconda/ui/gui/spokes/advstorage/zfcp.py b/pyanaconda/ui/gui/spokes/advstorage/zfcp.py
index 0674b1f..772471e 100644
--- a/pyanaconda/ui/gui/spokes/advstorage/zfcp.py
+++ b/pyanaconda/ui/gui/spokes/advstorage/zfcp.py
@@ -21,6 +21,8 @@
from pyanaconda.ui.gui import GUIObject
from pyanaconda.ui.gui.utils import gtk_action_nowait
+from gi.repository import BlockDev as blockdev
+from gi.repository import GLib
from blivet.zfcp import ZFCPDevice
@@ -96,11 +98,11 @@ class ZFCPDialog(GUIObject):
# below really, really is ugly and needs to be re-factored, but this
# should give a good base idea as far as expected behavior should go
try:
- device = dev.sanitizeDeviceInput(self._deviceEntry.get_text())
- wwpn = dev.sanitizeWWPNInput(self._wwpnEntry.get_text())
- lun = dev.sanitizeFCPLInput(self._lunEntry.get_text())
- except ValueError as e:
- _config_error = str(e)
+ device = blockdev.s390.sanitize_dev_input(self._deviceEntry.get_text())
+ wwpn = blockdev.s390.zfcp_sanitize_wwpn_input(self._wwpnEntry.get_text())
+ lun = blockdev.s390.zfcp_sanitize_lun_input(self._lunEntry.get_text())
+ except GLib.GError as err:
+ _config_error = str(err)
self.builder.get_object("deviceErrorLabel").set_text(_config_error)
self._conditionNotebook.set_current_page(2)
diff --git a/pyanaconda/ui/gui/spokes/lib/dasdfmt.py b/pyanaconda/ui/gui/spokes/lib/dasdfmt.py
index 1b94705..049875c 100644
--- a/pyanaconda/ui/gui/spokes/lib/dasdfmt.py
+++ b/pyanaconda/ui/gui/spokes/lib/dasdfmt.py
@@ -27,8 +27,8 @@ from pyanaconda import constants
from pyanaconda.i18n import _
import threading
-from blivet.devicelibs.dasd import format_dasd
-from blivet.errors import DasdFormatError
+from gi.repository import BlockDev as blockdev
+from gi.repository import GLib
import logging
log = logging.getLogger("anaconda")
@@ -90,8 +90,8 @@ class DasdFormatDialog(GUIObject):
for disk in self.to_format:
try:
gtk_call_once(self._formatting_label.set_text, _("Formatting /dev/%s. This may take a moment.") % disk)
- format_dasd(disk)
- except DasdFormatError as err:
+ blockdev.s390.format_dasd(disk)
+ except GLib.GError as err:
# Log errors if formatting fails, but don't halt the installer
log.error(str(err))
continue
diff --git a/pyanaconda/ui/gui/spokes/storage.py b/pyanaconda/ui/gui/spokes/storage.py
index 3cba326..5f04f8f 100644
--- a/pyanaconda/ui/gui/spokes/storage.py
+++ b/pyanaconda/ui/gui/spokes/storage.py
@@ -40,6 +40,7 @@
"""
from gi.repository import Gdk, GLib, AnacondaWidgets
+from gi.repository import BlockDev as blockdev
from pyanaconda.ui.communication import hubQ
from pyanaconda.ui.lib.disks import getDisks, isLocalDisk, applyDiskSelection
@@ -59,9 +60,8 @@ from blivet import arch
from blivet import autopart
from blivet.size import Size
from blivet.devices import MultipathDevice, ZFCPDiskDevice
-from blivet.errors import StorageError, DasdFormatError
+from blivet.errors import StorageError
from blivet.platform import platform
-from blivet.devicelibs.dasd import make_unformatted_dasd_list, format_dasd
from pyanaconda.threads import threadMgr, AnacondaThread
from pyanaconda.product import productName
from pyanaconda.flags import flags
@@ -664,7 +664,7 @@ class StorageSpoke(NormalSpoke, StorageChecker):
# actions on storage devices
threadMgr.wait(constants.THREAD_STORAGE)
- to_format = make_unformatted_dasd_list(d.name for d in getDisks(self.storage.devicetree))
+ to_format = self.storage.devicetree.make_unformatted_dasd_list(d.name for d in getDisks(self.storage.devicetree))
if not to_format:
# nothing to do here; bail
return
@@ -672,8 +672,8 @@ class StorageSpoke(NormalSpoke, StorageChecker):
hubQ.send_message(self.__class__.__name__, _("Formatting DASDs"))
for disk in to_format:
try:
- format_dasd(disk)
- except DasdFormatError as err:
+ blockdev.s390.dasd_format(disk)
+ except GLib.GError as err:
# Log errors if formatting fails, but don't halt the installer
log.error(str(err))
continue
@@ -774,7 +774,7 @@ class StorageSpoke(NormalSpoke, StorageChecker):
if arch.isS390():
# check for unformatted DASDs and launch dasdfmt if any discovered
- dasds = make_unformatted_dasd_list(self.selected_disks)
+ dasds = self.storage.devicetree.make_unformatted_dasd_list(self.selected_disks)
if len(dasds) > 0:
# We want to apply current selection before running dasdfmt to
# prevent this information from being lost afterward
diff --git a/pyanaconda/ui/tui/spokes/storage.py b/pyanaconda/ui/tui/spokes/storage.py
index 1c7b600..bf931b8 100644
--- a/pyanaconda/ui/tui/spokes/storage.py
+++ b/pyanaconda/ui/tui/spokes/storage.py
@@ -33,7 +33,7 @@ from blivet import arch
from blivet.size import Size
from blivet.errors import StorageError, DasdFormatError
from blivet.devices import DASDDevice, FcoeDiskDevice, iScsiDiskDevice, MultipathDevice, ZFCPDiskDevice
-from blivet.devicelibs.dasd import format_dasd, make_unformatted_dasd_list
+from gi.repository import BlockDev as blockdev
from pyanaconda.flags import flags
from pyanaconda.kickstart import doKickstartStorage, resetCustomStorageData
from pyanaconda.threads import threadMgr, AnacondaThread
@@ -89,7 +89,7 @@ class StorageSpoke(NormalTUISpoke):
# dasds, automatically format them. pass in storage.devicetree here
# instead of storage.disks since mediaPresent is checked on disks;
# a dasd needing dasdfmt will fail this media check though
- to_format = make_unformatted_dasd_list(d.name for d in getDisks(self.storage.devicetree))
+ to_format = storage.devicetree.make_unformatted_dasd_list(d.name for d in getDisks(self.storage.devicetree))
if to_format:
self.run_dasdfmt(to_format)
@@ -276,7 +276,7 @@ class StorageSpoke(NormalTUISpoke):
# if we're on s390x, since they need to be formatted before we
# can use them.
if arch.isS390():
- to_format = make_unformatted_dasd_list(self.selected_disks)
+ to_format = self.storage.devicetree.make_unformatted_dasd_list(self.selected_disks)
if to_format:
self.run_dasdfmt(to_format)
return None
@@ -322,7 +322,7 @@ class StorageSpoke(NormalTUISpoke):
for disk in to_format:
try:
print(_("Formatting /dev/%s. This may take a moment.") % disk)
- format_dasd(disk)
+ blockdev.s390.dasd_format(disk)
except DasdFormatError as err:
# Log errors if formatting fails, but don't halt the installer
log.error(str(err))
--
1.9.3
More information about the anaconda-patches
mailing list