yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

diskmanage: don't set usage for unused partitions

Browse Source 
The disk type is already 'partition' so there's no additional
information here. And it would need to serve as a code-word for
unused partitions. The cleaner approach is to not set the usage.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
This commit is contained in:
Fabian Ebner
2021-10-06 11:18:43 +02:00
committed by Thomas Lamprecht
parent bd46e59b5d
commit ff91cfae35
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
PVE/Diskmanage.pm
View File

@ -661,7 +661,7 @@ sub get_disks {
# for devices, this check is done explicitly later # for devices, this check is done explicitly later
return 'Device Mapper' if !dir_is_empty("$sysdir/holders"); return 'Device Mapper' if !dir_is_empty("$sysdir/holders");
return 'partition'; return; # unused partition
}; };
my $collect_ceph_info = sub { my $collect_ceph_info = sub {
@ -730,7 +730,6 @@ sub get_disks {
my $used = $determine_usage->($devpath, $sysdir, 0); my $used = $determine_usage->($devpath, $sysdir, 0);
if (!$include_partitions) { if (!$include_partitions) {
foreach my $part (sort keys %{$partitions}) { foreach my $part (sort keys %{$partitions}) {
next if $partitions->{$part}->{used} eq 'partition';
$used //= $partitions->{$part}->{used}; $used //= $partitions->{$part}->{used};
} }
} else { } else {