From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id 828BF1FF183 for ; Wed, 8 Oct 2025 17:12:15 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id AC359C01B; Wed, 8 Oct 2025 17:12:20 +0200 (CEST) From: Fiona Ebner To: pve-devel@lists.proxmox.com Date: Wed, 8 Oct 2025 17:11:31 +0200 Message-ID: <20251008151140.127355-2-f.ebner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20251008151140.127355-1-f.ebner@proxmox.com> References: <20251008151140.127355-1-f.ebner@proxmox.com> MIME-Version: 1.0 X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1759936273786 X-SPAM-LEVEL: Spam detection results: 0 AWL -0.073 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_SHORT 0.001 Use of a URL Shortener for very short URL PROLO_LEO1 0.1 Meta Catches all Leo drug variations so far RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_RPBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_SAFE_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [proxmox.com, plugin.pm, config.pm, gnu.org, storage.pm] Subject: [pve-devel] [RFC storage 1/2] close #6669: plugin api: introduce on_update_hook_full() method X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox VE development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" The original on_update_hook() method is limited, because only the updated properties and values are passed in. Introduce a new on_update_hook_full() method which also receives the current storage configuration and the list of which properties are to be deleted. This allows detecting and reacting to all changes and knowing how values changed. Deletion of properties is deferred to after the on_update_hook(_full) call. This makes it possible to pass the unmodified current storage configuration to the method. The default implementation of on_update_hook_full() just falls back to the original on_update_hook(). Bump APIVER and APIAGE. Signed-off-by: Fiona Ebner --- ApiChangeLog | 13 +++++++++++++ src/PVE/API2/Storage/Config.pm | 18 +++++++++++++++--- src/PVE/Storage.pm | 4 ++-- src/PVE/Storage/Plugin.pm | 33 +++++++++++++++++++++++++++++++++ 4 files changed, 63 insertions(+), 5 deletions(-) diff --git a/ApiChangeLog b/ApiChangeLog index d80bfb3..13e4339 100644 --- a/ApiChangeLog +++ b/ApiChangeLog @@ -6,6 +6,19 @@ without breaking anything unaware of it.) Future changes should be documented in here. +## Version 13: + +* Introduce `on_update_hook_full()` plugin method + + The original `on_update_hook()` plugin method was limited, because only the updated properties and + values would be passed in. The new `on_update_hook_full()` plugin method also receives the current + storage configuration and the list of which properties are to be deleted. This allows detecting + and reacting to all changes and knowing how values changed. See also bug #6669 [0] for the initial + motiviation. If a plugin implements `on_update_hook_full()`, that method will be called rather + than the `on_update_hook()` method. + + [0]: https://bugzilla.proxmox.com/show_bug.cgi?id=6669 + ## Version 12: * Introduce `qemu_blockdev_options()` plugin method diff --git a/src/PVE/API2/Storage/Config.pm b/src/PVE/API2/Storage/Config.pm index 34f2d85..c10ccf8 100755 --- a/src/PVE/API2/Storage/Config.pm +++ b/src/PVE/API2/Storage/Config.pm @@ -360,6 +360,9 @@ __PACKAGE__->register_method({ my $plugin = PVE::Storage::Plugin->lookup($type); my $opts = $plugin->check_config($storeid, $param, 0, 1); + # Do checks for deletion up-front, but defer actual deletion until after the + # on_update_hook(_full) call. This makes it possible to pass the unmodified current + # storage configuration to the method. if ($delete) { my $options = $plugin->private()->{options}->{$type}; foreach my $k (@$delete) { @@ -368,12 +371,21 @@ __PACKAGE__->register_method({ die "unable to delete fixed option '$k'\n" if $d->{fixed}; die "cannot set and delete property '$k' at the same time!\n" if defined($opts->{$k}); - - delete $scfg->{$k}; } } - $returned_config = $plugin->on_update_hook($storeid, $opts, %$sensitive); + if ($plugin->can('api') && $plugin->api() < 13) { + $returned_config = $plugin->on_update_hook($storeid, $opts, %$sensitive); + } else { + $returned_config = + $plugin->on_update_hook_full($storeid, $scfg, $opts, $delete, $sensitive); + } + + if ($delete) { + for my $k ($delete->@*) { + delete $scfg->{$k}; + } + } for my $k (keys %$opts) { $scfg->{$k} = $opts->{$k}; diff --git a/src/PVE/Storage.pm b/src/PVE/Storage.pm index 1dde2b7..ca0bf0e 100755 --- a/src/PVE/Storage.pm +++ b/src/PVE/Storage.pm @@ -41,11 +41,11 @@ use PVE::Storage::BTRFSPlugin; use PVE::Storage::ESXiPlugin; # Storage API version. Increment it on changes in storage API interface. -use constant APIVER => 12; +use constant APIVER => 13; # 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 => 3; +use constant APIAGE => 4; our $KNOWN_EXPORT_FORMATS = ['raw+size', 'tar+size', 'qcow2+size', 'vmdk+size', 'zfs', 'btrfs']; diff --git a/src/PVE/Storage/Plugin.pm b/src/PVE/Storage/Plugin.pm index 8acd214..9dc684d 100644 --- a/src/PVE/Storage/Plugin.pm +++ b/src/PVE/Storage/Plugin.pm @@ -672,6 +672,39 @@ sub on_update_hook { return undef; } +=head3 on_update_hook_full + + $returned_config = $plugin->on_update_hook_full($storeid, $scfg, $update, $delete, $sensitive); + +Most plugins use an empty C, so that C<$returned_config> will be C. While a plugin +can return auto-generated properties via C<$returned_config>, this is currently only used for the +C<'encryption-key'> for the PBS plugin. + +Arguments: + +=over + +=item C<$storeid>: The storage ID. + +=item C<$scfg>: The current storage configuration. + +=item C<$update>: Hash reference with properties to be updated and their new values. + +=item C<$delete>: Array reference with properties to be deleted. + +=item C<$sensitive>: Hash reference with sensitive properties and their new values. Sensitive +properties are declared via the plugin data method C. + +=back + +=cut + +sub on_update_hook_full { + my ($class, $storeid, $scfg, $update, $delete, $sensitive) = @_; + + return $class->on_update_hook($storeid, $update, $sensitive->%*); +} + # called during deletion of storage (before the new storage config got written) # and if the activate check on addition fails, to cleanup all storage traces # which on_add_hook may have created. -- 2.47.3 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel