diff options
| author | Jenkins <jenkins@review.openstack.org> | 2012-03-31 01:14:23 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2012-03-31 01:14:23 +0000 |
| commit | 3842f2f7bdfcf21d2ea44b23e4b9e7d575561d2d (patch) | |
| tree | b9ac2f131f08682e8ec2a771bdb88bc50daec234 | |
| parent | 3b3c6fbf0527f9709a8f2306b512be542257bb8b (diff) | |
| parent | 46d65edcc948781aeea1d611735d81c2b29322f3 (diff) | |
Merge "Allow snapshots in error state to be deleted."
| -rw-r--r-- | nova/tests/test_volume.py | 20 | ||||
| -rw-r--r-- | nova/volume/api.py | 4 |
2 files changed, 22 insertions, 2 deletions
diff --git a/nova/tests/test_volume.py b/nova/tests/test_volume.py index be589d3e2..fca593c7c 100644 --- a/nova/tests/test_volume.py +++ b/nova/tests/test_volume.py @@ -275,7 +275,27 @@ class VolumeTestCase(test.TestCase): volume_api.delete, self.context, volume) + self.volume.delete_snapshot(self.context, snapshot_id) + self.volume.delete_volume(self.context, volume['id']) + + def test_can_delete_errored_snapshot(self): + """Test snapshot can be created and deleted.""" + volume = self._create_volume() + self.volume.create_volume(self.context, volume['id']) + snapshot_id = self._create_snapshot(volume['id']) + self.volume.create_snapshot(self.context, volume['id'], snapshot_id) + snapshot = db.snapshot_get(context.get_admin_context(), + snapshot_id) + + volume_api = nova.volume.api.API() + + snapshot['status'] = 'badstatus' + self.assertRaises(exception.InvalidVolume, + volume_api.delete_snapshot, + self.context, + snapshot) + snapshot['status'] = 'error' self.volume.delete_snapshot(self.context, snapshot_id) self.volume.delete_volume(self.context, volume['id']) diff --git a/nova/volume/api.py b/nova/volume/api.py index 26d56c057..76de55131 100644 --- a/nova/volume/api.py +++ b/nova/volume/api.py @@ -321,8 +321,8 @@ class API(base.Base): @wrap_check_policy def delete_snapshot(self, context, snapshot): - if snapshot['status'] != "available": - msg = _("must be available") + if snapshot['status'] not in ["available", "error"]: + msg = _("Volume Snapshot status must be available or error") raise exception.InvalidVolume(reason=msg) self.db.snapshot_update(context, snapshot['id'], {'status': 'deleting'}) |