[pve-devel] [PATCH storage v3 2/2] BTRFSPlugin: reuse DirPlugin update/get_volume_attribute

Dominik Csapak d.csapak at proxmox.com
Thu Jun 2 10:52:14 CEST 2022


this allows setting notes+protected for backups on btrfs

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller at proxmox.com>
---
changes from v2:
* parameter style fix
* added Acked-by

 PVE/Storage/BTRFSPlugin.pm | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm
index be613f4..1db4e4f 100644
--- a/PVE/Storage/BTRFSPlugin.pm
+++ b/PVE/Storage/BTRFSPlugin.pm
@@ -138,9 +138,22 @@ sub status {
     return PVE::Storage::DirPlugin::status($class, $storeid, $scfg, $cache);
 }
 
-# TODO: sub get_volume_attribute {}
+sub get_volume_attribute {
+    my ($class, $scfg, $storeid, $volname, $attribute) = @_;
+    return PVE::Storage::DirPlugin::get_volume_attribute($class, $scfg, $storeid, $volname, $attribute);
+}
 
-# TODO: sub update_volume_attribute {}
+sub update_volume_attribute {
+    my ($class, $scfg, $storeid, $volname, $attribute, $value) = @_;
+    return PVE::Storage::DirPlugin::update_volume_attribute(
+	$class,
+	$scfg,
+	$storeid,
+	$volname,
+	$attribute,
+	$value,
+    );
+}
 
 # croak would not include the caller from within this module
 sub __error {
-- 
2.30.2






More information about the pve-devel mailing list