[pve-devel] [PATCH docs] lvm: add note about requirements of snapshot-as-volume-chains

[Friedrich Weber]

On 28/07/2025 10:28, Fabian Grünbichler wrote:
> On July 27, 2025 2:03 pm, DERUMIER, Alexandre via pve-devel wrote:
>> about discard, maybe we should add a note about enabling
>> issue_discards=1 in lvm.conf. (don't known if it could be enabled by
>> default for new install ?)
> 
> yes, maybe. or alternatively, we could call blkdiscard as part of the
> removal logic in the storage plugin?

Just remembered this because it has occasionally come up in enterprise
support that users are confused that deleting LVs doesn't free up space
on (thin-provisioned) LUNs by default. Adding a note about
issue_discards (or, of course, blkdiscard'ing on removal in the storage
plugin) would be nice.