From a799f7529b9c4430fee13e5b939fe3723b650766 Mon Sep 17 00:00:00 2001 From: Fabian Ebner Date: Tue, 19 Oct 2021 09:54:52 +0200 Subject: [PATCH] bump APIVER and APIAGE Added blockers parameter to volume_rollback_is_possible. Replaced volume_snapshot_list with volume_snapshot_info. Signed-off-by: Fabian Ebner --- ApiChangeLog | 16 ++++++++++++++++ PVE/Storage.pm | 4 ++-- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/ApiChangeLog b/ApiChangeLog index 8c119c5..4094ca2 100644 --- a/ApiChangeLog +++ b/ApiChangeLog @@ -6,6 +6,21 @@ without breaking anything unaware of it.) Future changes should be documented in here. +## Version 10: + +* Replace `volume_snapshot_list` with `volume_snapshot_info`: + + `volume_snapshot_list` was used exclusively by replication and currently, replication is only + allowed for the storage type `zfspool`. Thus, no external plugins should be affected by this + change and `APIAGE` is *not* reset. + + `volume_snapshot_info` returns a hash with snapshot names as keys and `id` and `timestamp` data + for each snapshot, rather than just an array of snaphsot names like `volume_snapshot_list` did. + +* Add `blockers` parameter to `volume_rollback_is_possible`: + + The parameter *can* be used to return a list of snapshots that is currently preventing rollback. + ## Version 9: (AGE resets to 0): * volume_import_formats gets a new parameter *inserted*: @@ -23,3 +38,4 @@ Future changes should be documented in here. * $with_snapshots *may* now be an array reference containing an ordered list of snapshots, but *may* also just be a boolean, and the contained list *may* be ignored, so it can still be treated as a boolean. + diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 5d389be..bc477c1 100755 --- a/PVE/Storage.pm +++ b/PVE/Storage.pm @@ -41,11 +41,11 @@ use PVE::Storage::PBSPlugin; use PVE::Storage::BTRFSPlugin; # Storage API version. Increment it on changes in storage API interface. -use constant APIVER => 9; +use constant APIVER => 10; # Age is the number of versions we're backward compatible with. # This is like having 'current=APIVER' and age='APIAGE' in libtool, # see https://www.gnu.org/software/libtool/manual/html_node/Libtool-versioning.html -use constant APIAGE => 0; +use constant APIAGE => 1; # load standard plugins PVE::Storage::DirPlugin->register();