[pve-devel] [PATCH storage 1/2] lvm thin: status: code cleanup
Fabian Ebner
f.ebner at proxmox.com
Fri Nov 5 11:29:44 CET 2021
Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---
PVE/Storage/LvmThinPlugin.pm | 16 ++++------------
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/PVE/Storage/LvmThinPlugin.pm b/PVE/Storage/LvmThinPlugin.pm
index 4ba6f90..34e57b2 100644
--- a/PVE/Storage/LvmThinPlugin.pm
+++ b/PVE/Storage/LvmThinPlugin.pm
@@ -194,23 +194,15 @@ sub list_thinpools {
sub status {
my ($class, $storeid, $scfg, $cache) = @_;
- my $lvname = "$scfg->{vgname}/$scfg->{thinpool}";
+ my $lvs = $cache->{lvs} ||= PVE::Storage::LVMPlugin::lvm_list_volumes();
- $cache->{lvs} = PVE::Storage::LVMPlugin::lvm_list_volumes() if !$cache->{lvs};
-
- my $lvs = $cache->{lvs};
-
- return undef if !$lvs->{$scfg->{vgname}};
+ return if !$lvs->{$scfg->{vgname}};
my $info = $lvs->{$scfg->{vgname}}->{$scfg->{thinpool}};
- return undef if !$info;
+ return if !$info || $info->{lv_type} ne 't' || !$info->{lv_size};
- return undef if $info->{lv_type} ne 't';
-
- return ($info->{lv_size}, $info->{lv_size} - $info->{used}, $info->{used}, 1) if $info->{lv_size};
-
- return undef;
+ return ($info->{lv_size}, $info->{lv_size} - $info->{used}, $info->{used}, 1);
}
sub activate_volume {
--
2.30.2
More information about the pve-devel
mailing list