Comment 6 for bug 1903319

Revision history for this message
Rajat Dhasmana (whoami-rajat) wrote :

The code seems strange to me for deleting snapshot, It doesn't check if the volume is attached, doesn't perform any rebase of data in the base file to the snapshot file while deleting. It is just locating the file and deleting it.[1]
Looks very wrong to me or I'm just missing something here.

[1] https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/dataontap/nfs_base.py#L266-L280