[pve-devel] [PATCH] RBD: snap purge does not support automatic unprotect so list all snapshots and then unprotect and delete them

[Fabian Grünbichler]

On Thu, Nov 17, 2016 at 08:21:33AM +0100, Stefan Priebe - Profihost AG wrote:
> Am 17.11.2016 um 07:33 schrieb Dietmar Maurer:
> > AFAIK we only protect base volumes, and we 'unprotect' that in the code.
> > So what is the purpose of this patch?
> > 
> 
> good question ;-) I just remember that i had this situation where i did
> clones from snapshots which results in protected snapshots.
> 
> Greets,
> Stefan

if you create clones manually, aren't you also responsible for cleaning
them up / unprotecting the parent snapshots when deleting the last
clone?

I don't have a very good feeling when auto-deleting snapshots that were
protected by the user/admin. On the other hand, ceph prevents us from
unprotecting a snapshot that is still in use as parent of a clone.. But
who knows, maybe some users used protection as a "safeguard" against
deletion by PVE ;)