diff options
author | Chris Lumens <clumens@redhat.com> | 2010-02-08 11:38:05 -0500 |
---|---|---|
committer | Chris Lumens <clumens@redhat.com> | 2010-02-08 11:38:05 -0500 |
commit | a1780c595fe916570a9b82ae93fcee896c814f20 (patch) | |
tree | 8e5e0f4a646301c9648ed5779b72d1d15e25cf0f /packages.py | |
parent | 6e2b508fb660e679611a7793d6a240015b9c060d (diff) | |
download | anaconda-a1780c595fe916570a9b82ae93fcee896c814f20.tar.gz anaconda-a1780c595fe916570a9b82ae93fcee896c814f20.tar.xz anaconda-a1780c595fe916570a9b82ae93fcee896c814f20.zip |
Clarify which storage exceptions are bugs (#557928).
In turnOnFilesystems, some exceptions are bugs in anaconda and some are just
fatal errors reported by the underlying tools. An example of the latter is
not being able to resize a filesystem due to space issues. These should not
get the exception dialog since that looks like anaconda is at fault. For all
other cases, it's likely an anaconda bug and should get the exception
dialog.
Diffstat (limited to 'packages.py')
-rw-r--r-- | packages.py | 52 |
1 files changed, 10 insertions, 42 deletions
diff --git a/packages.py b/packages.py index 6a5a05c9a..4ae9f4da3 100644 --- a/packages.py +++ b/packages.py @@ -107,28 +107,6 @@ def turnOnFilesystems(anaconda): try: anaconda.storage.doIt() - except DeviceResizeError as (msg, device): - # XXX does this make any sense? do we support resize of - # devices other than partitions? - title = _("Device Resize Failed") - message = _("An error was encountered while " - "resizing device %s.") % (device,) - details = msg - except DeviceCreateError as (msg, device): - title = _("Device Creation Failed") - message = _("An error was encountered while " - "creating device %s.") % (device,) - details = msg - except DeviceDestroyError as (msg, device): - title = _("Device Removal Failed") - message = _("An error was encountered while " - "removing device %s.") % (device,) - details = msg - except DeviceError as (msg, device): - title = _("Device Setup Failed") - message = _("An error was encountered while " - "setting up device %s.") % (device,) - details = msg except FSResizeError as (msg, device): title = _("Resizing Failed") message = _("There was an error encountered while " @@ -143,27 +121,17 @@ def turnOnFilesystems(anaconda): message = _("An error was encountered while " "migrating filesystem on device %s.") % (device,) details = msg - except FormatCreateError as (msg, device): - title = _("Formatting Failed") - message = _("An error was encountered while " - "formatting device %s.") % (device,) - details = msg except Exception as e: - # catch-all - title = _("Storage Activation Failed") - message = _("An error was encountered while " - "activating your storage configuration.") - details = str(e) - - if title: - rc = anaconda.intf.detailedMessageWindow(title, message, details, - type = "custom", - custom_buttons = [_("_File Bug"), _("_Exit installer")]) - - if rc == 0: - raise - elif rc == 1: - sys.exit(1) + raise + + rc = anaconda.intf.detailedMessageWindow(title, message, details, + type = "custom", + custom_buttons = [_("_File Bug"), _("_Exit installer")]) + + if rc == 0: + raise + elif rc == 1: + sys.exit(1) if not anaconda.upgrade: anaconda.storage.turnOnSwap() |