diff options
| author | Jenkins <jenkins@review.openstack.org> | 2012-01-21 05:18:26 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2012-01-21 05:18:26 +0000 |
| commit | ee75f9efb4d69bca400ad4b5a2c901540963ebb4 (patch) | |
| tree | 6c182b0fae18bb268ed7da1cd78eaa938fe95651 /nova/compute | |
| parent | c7646aa88d564694b99a569c3cdd2c7ffbbb745d (diff) | |
| parent | d7448fc86a929226a34e92ef77684441f451edaf (diff) | |
Merge "Handle FlavorNotFound on server list w/ filter"
Diffstat (limited to 'nova/compute')
| -rw-r--r-- | nova/compute/api.py | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/nova/compute/api.py b/nova/compute/api.py index 2f3749ca0..5254757de 100644 --- a/nova/compute/api.py +++ b/nova/compute/api.py @@ -1020,8 +1020,12 @@ class API(base.Base): filters = {} def _remap_flavor_filter(flavor_id): - instance_type = instance_types.get_instance_type_by_flavor_id( - flavor_id) + try: + instance_type = instance_types.get_instance_type_by_flavor_id( + flavor_id) + except exception.FlavorNotFound: + raise ValueError() + filters['instance_type_id'] = instance_type['id'] def _remap_fixed_ip_filter(fixed_ip): @@ -1054,7 +1058,13 @@ class API(base.Base): if isinstance(remap_object, basestring): filters[remap_object] = value else: - remap_object(value) + try: + remap_object(value) + + # We already know we can't match the filter, so + # return an empty list + except ValueError: + return [] local_zone_only = search_opts.get('local_zone_only', False) |