diff options
author | Kieran Spear <kispear@gmail.com> | 2013-08-13 16:54:59 +1000 |
---|---|---|
committer | Kieran Spear <kispear@gmail.com> | 2013-08-13 17:21:32 +1000 |
commit | 242c8df495d8a22bd58797691989852d1b714824 (patch) | |
tree | 519d761d1989ca1aca0f98f9dd0cccfaa9f99874 /openstack_dashboard/dashboards/project/instances/workflows/create_instance.py | |
parent | fe659b231a9542e4703c13e7e1173aa7e0767cfc (diff) | |
download | horizon-242c8df495d8a22bd58797691989852d1b714824.tar.gz |
Enable H201: do not write "except:"
Fixes all occurrences of this. We have a custom exception handler
in Horizon anyway that only catches ClientException in most of these
cases, but this commit lets us gate on the other cases.
Change-Id: Iea3dc13817f3e5b775b2024424bf3a906da5584b
Closes-Bug: #1211657
Diffstat (limited to 'openstack_dashboard/dashboards/project/instances/workflows/create_instance.py')
-rw-r--r-- | openstack_dashboard/dashboards/project/instances/workflows/create_instance.py | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/openstack_dashboard/dashboards/project/instances/workflows/create_instance.py b/openstack_dashboard/dashboards/project/instances/workflows/create_instance.py index d95c0391a..b8c1618ea 100644 --- a/openstack_dashboard/dashboards/project/instances/workflows/create_instance.py +++ b/openstack_dashboard/dashboards/project/instances/workflows/create_instance.py @@ -128,7 +128,7 @@ class VolumeOptionsAction(workflows.Action): if v.status == api.cinder.VOLUME_STATE_AVAILABLE] volume_options.extend([self._get_volume_display_name(vol) for vol in volumes]) - except: + except Exception: exceptions.handle(self.request, _('Unable to retrieve list of volumes.')) return volume_options @@ -141,7 +141,7 @@ class VolumeOptionsAction(workflows.Action): if s.status == api.cinder.VOLUME_STATE_AVAILABLE] volume_options.extend([self._get_volume_display_name(snap) for snap in snapshots]) - except: + except Exception: exceptions.handle(self.request, _('Unable to retrieve list of volume ' 'snapshots.')) @@ -277,7 +277,7 @@ class SetInstanceDetailsAction(workflows.Action): flavor_list = [(flavor.id, "%s" % flavor.name) for flavor in sorted(flavors, key=key, reverse=rev)] - except: + except Exception: flavor_list = [] exceptions.handle(request, _('Unable to retrieve instance flavors.')) @@ -286,7 +286,7 @@ class SetInstanceDetailsAction(workflows.Action): def populate_availability_zone_choices(self, request, context): try: zones = api.nova.availability_zone_list(request) - except: + except Exception: zones = [] exceptions.handle(request, _('Unable to retrieve availability zones.')) @@ -308,7 +308,7 @@ class SetInstanceDetailsAction(workflows.Action): flavors = json.dumps([f._info for f in api.nova.flavor_list(self.request)]) extra['flavors'] = flavors - except: + except Exception: exceptions.handle(self.request, _("Unable to retrieve quota information.")) return super(SetInstanceDetailsAction, self).get_help_text(extra) @@ -373,7 +373,7 @@ class SetAccessControlsAction(workflows.Action): try: keypairs = api.nova.keypair_list(request) keypair_list = [(kp.name, kp.name) for kp in keypairs] - except: + except Exception: keypair_list = [] exceptions.handle(request, _('Unable to retrieve keypairs.')) @@ -389,7 +389,7 @@ class SetAccessControlsAction(workflows.Action): try: groups = api.network.security_group_list(request) security_group_list = [(sg.name, sg.name) for sg in groups] - except: + except Exception: exceptions.handle(request, _('Unable to retrieve list of security groups')) security_group_list = [] @@ -465,7 +465,7 @@ class SetNetworkAction(workflows.Action): for n in networks: n.set_id_as_name_if_empty() network_list = [(network.id, network.name) for network in networks] - except: + except Exception: network_list = [] exceptions.handle(request, _('Unable to retrieve networks.')) @@ -550,6 +550,6 @@ class LaunchInstance(workflows.Workflow): instance_count=int(context['count']), admin_pass=context['admin_pass']) return True - except: + except Exception: exceptions.handle(request) return False |