For LVM storages using 'snapshot-as-volume-chain', it's necessary to
ensure that the size for the LV of the target snapshot volume is large
enough before doing a commit operation. This affects both the offline
and the online case, which have different code paths.
To make this possible via the storage (plugin) API, the
volume_resize() method gains a new 'snapname' parameter and the
'virtual-size' of each snapshot is returned in the result of
volume_snapshot_info().
The series also adds documentation for volume_resize() and
volume_snapshot_info(), the latter of which is a bit hairy, since it
was previously only used for ZFS replication and then re-used for
snapshot-as-volume-chain which returns quite different information.
It's not too bad though.
Lastly, there's some cleanups for the volume_snapshot_info() method.
Storage plugin API age and version are bumped in the series.
Dependency bump from qemu-server to libpve-storage-perl needed!
storage:
Fiona Ebner (10):
plugin: document volume_snapshot_info() method
plugin: volume snapshot info: also return virtual size
plugin: document volume_resize() method
plugin/storage: volume resize: add snapname parameter
lvm plugin: snapshot delete: clarify comment about using commit
bump API age and version
partially fix #7094: lvm: snapshot delete: ensure commit target is
large enough
plugin: volume snapshot info: don't set 'order' for internal snapshots
plugin: volume snapshot info: correctly return internal snapshot
information
plugin: volume snapshot info: do not set 'ext' property
ApiChangeLog | 20 ++++++
src/PVE/Storage.pm | 8 +--
src/PVE/Storage/BTRFSPlugin.pm | 3 +-
src/PVE/Storage/ESXiPlugin.pm | 2 +-
src/PVE/Storage/ISCSIDirectPlugin.pm | 2 +-
src/PVE/Storage/ISCSIPlugin.pm | 2 +-
src/PVE/Storage/LVMPlugin.pm | 21 ++++--
src/PVE/Storage/LvmThinPlugin.pm | 8 ++-
src/PVE/Storage/PBSPlugin.pm | 2 +-
src/PVE/Storage/Plugin.pm | 103 +++++++++++++++++++++------
src/PVE/Storage/RBDPlugin.pm | 4 +-
src/PVE/Storage/ZFSPlugin.pm | 5 +-
src/PVE/Storage/ZFSPoolPlugin.pm | 4 +-
13 files changed, 145 insertions(+), 39 deletions(-)
qemu-server:
Fiona Ebner (1):
partially fix #7094: external snapshot delete: ensure commit target is
large enough
src/PVE/QemuServer.pm | 9 +++++++++
1 file changed, 9 insertions(+)
Summary over all repositories:
14 files changed, 154 insertions(+), 39 deletions(-)
--
Generated by git-murpp 0.5.0
_______________________________________________
pve-devel mailing list
[email protected]
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
