[pve-devel] [PATCH pve-storage] lvm: use blkdiscard instead cstream to saferemove drive

[DERUMIER, Alexandre]

> +        saferemove_discard => {
> +            description => "Wipe with discard instead zeroing.",
> +            type => 'boolean',

>>Not sure we need this (not sure when this is actually useful), but
>>it's
>>cheap enough to have around. Should add a `default => 0` for
>>documentation purposes, though.

Some storage allow overprovisioning. (create a lun bigger than the real
storage size), it can be interesting to discard instead zeroing (to
free space on storage side).
as snapshots currently use lvm volume with same size than the main
volume, it can be interesting to have overprovisioning.


btw, blockbridge guys have made a nice article here:
https://kb.blockbridge.com/technote/proxmox-qcow-snapshots-on-lvm/