[pve-devel] applied: [PATCH v2 guest-common 2/2] config: activate affected storages for snapshot operations

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Feb 4 17:38:25 CET 2022


On 13.01.22 12:04, Fabian Ebner wrote:
> For snapshot creation, the storage for the vmstate file is activated
> via vdisk_alloc when the state file is created.
> 
> Do not activate the volumes themselves, as that has unnecessary side
> effects (e.g. waiting for zvol device link for ZFS, mapping the volume
> for RBD). If a storage can only do snapshot operations on a volume
> that has been activated, it needs to activate the volume itself.
> 
> The actual implementation will be in the plugins to be able to skip
> CD ROM drives and bind-mounts, etc.
> 
> Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
> ---
>  src/PVE/AbstractConfig.pm | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
>

applied, thanks! But I defused the fixme comment, this is internal and doesn't
need to wait until a major release.





More information about the pve-devel mailing list