diff options
| author | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-09-26 20:54:21 +0000 |
|---|---|---|
| committer | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-09-26 20:54:21 +0000 |
| commit | 2d0a78f413d8a6cddef86f348affd9e2a1580ad6 (patch) | |
| tree | eb80b70465f19c1789404391db3055c3f18adae3 /nova | |
| parent | 064506df6dc21efb9bdde21f958fc733563e0b48 (diff) | |
| download | nova-2d0a78f413d8a6cddef86f348affd9e2a1580ad6.tar.gz nova-2d0a78f413d8a6cddef86f348affd9e2a1580ad6.tar.xz nova-2d0a78f413d8a6cddef86f348affd9e2a1580ad6.zip | |
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
Diffstat (limited to 'nova')
| -rw-r--r-- | nova/compute/api.py | 6 |
1 files changed, 0 insertions, 6 deletions
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) |