diff --git a/nova/tests/test_volume.py b/nova/tests/test_volume.py index be589d3e247..fca593c7c29 100644 --- a/nova/tests/test_volume.py +++ b/nova/tests/test_volume.py @@ -275,7 +275,27 @@ def test_cant_delete_volume_with_snapshots(self): 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 26d56c0578c..76de55131ea 100644 --- a/nova/volume/api.py +++ b/nova/volume/api.py @@ -321,8 +321,8 @@ def create_snapshot_force(self, context, volume, name, description): @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'})