From e35c357ec070a16887c62062b2cd1c46aeabba45 Mon Sep 17 00:00:00 2001 From: Eric Harney Date: Mon, 18 Mar 2013 12:21:40 -0400 Subject: [PATCH] Allow snapshot_delete for NFS/GlusterFS drivers RemoteFsDriver should allow snapshot_delete, since snapshot_create for NFS and GlusterFS volumes will create a snapshot in the error state which cannot be deleted afterward. snapshot_delete() should allow this to be removed rather than throwing NotImplementedError. Fixes bug: 1156686 Change-Id: I4307878fa97fbce5bd733a83fa172509e2fba8ce (cherry picked from commit ea10f1b8291cb07962577699abe13648a4bdca58) --- cinder/volume/drivers/nfs.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/cinder/volume/drivers/nfs.py b/cinder/volume/drivers/nfs.py index 0c1df91da..73f3d10c1 100755 --- a/cinder/volume/drivers/nfs.py +++ b/cinder/volume/drivers/nfs.py @@ -66,6 +66,11 @@ class RemoteFsDriver(driver.VolumeDriver): def delete_volume(self, volume): raise NotImplementedError() + def delete_snapshot(self, snapshot): + """Do nothing for this driver, but allow manager to handle deletion + of snapshot in error state.""" + pass + def ensure_export(self, ctx, volume): raise NotImplementedError() -- 2.45.2