From 2d0a78f413d8a6cddef86f348affd9e2a1580ad6 Mon Sep 17 00:00:00 2001 From: Johannes Erdfelt Date: Wed, 26 Sep 2012 20:54:21 +0000 Subject: Remove unnecessary check if migration_ref is not None migration_get_by_instance_and_status will raise MigrationNotFoundByStatus if the migration isn't found, so the extra check is unnecessary since migration_ref can never be None. Change-Id: I640c4f64234f3991e03fcf23789c428854becc87 --- nova/compute/api.py | 6 ------ 1 file changed, 6 deletions(-) (limited to 'nova') diff --git a/nova/compute/api.py b/nova/compute/api.py index 531aa4414..e08fb8fc0 100644 --- a/nova/compute/api.py +++ b/nova/compute/api.py @@ -1439,9 +1439,6 @@ class API(base.Base): context = context.elevated() migration_ref = self.db.migration_get_by_instance_and_status(context, instance['uuid'], 'finished') - if not migration_ref: - raise exception.MigrationNotFoundByStatus( - instance_id=instance['uuid'], status='finished') # reverse quota reservation for increased resource usage deltas = self._reverse_upsize_quota_delta(context, migration_ref) @@ -1466,9 +1463,6 @@ class API(base.Base): context = context.elevated() migration_ref = self.db.migration_get_by_instance_and_status(context, instance['uuid'], 'finished') - if not migration_ref: - raise exception.MigrationNotFoundByStatus( - instance_id=instance['uuid'], status='finished') # reserve quota only for any decrease in resource usage deltas = self._downsize_quota_delta(context, migration_ref) -- cgit