]> git.proxmox.com Git - qemu-server.git/commitdiff
scan volids: remove superfluous parameter
authorFabian Ebner <f.ebner@proxmox.com>
Fri, 4 Jun 2021 13:49:29 +0000 (15:49 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Tue, 8 Jun 2021 12:56:30 +0000 (14:56 +0200)
The only caller that didn't use 'images' was removed as part of the migration
refactoring in commit 62a4c963b824c923a4fc82a48c81d0f63ebaddae, so this is not
even a breaking change as the 'PVE 7' comment might've suggested.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
Reviewed-by: Stefan Reiter <s.reiter@proxmox.com>
PVE/QemuServer.pm

index 25ac05240cb6f599f75ed4b8c1d4d72876f6209b..d77981c765afa868b6377bdec4e3a1c4a6242531 100644 (file)
@@ -6091,11 +6091,10 @@ my $restore_destroy_volumes = sub {
     }
 };
 
-# FIXME For PVE 7.0, remove $content_type and always use 'images'
 sub scan_volids {
-    my ($cfg, $vmid, $content_type) = @_;
+    my ($cfg, $vmid) = @_;
 
-    my $info = PVE::Storage::vdisk_list($cfg, undef, $vmid, undef, $content_type);
+    my $info = PVE::Storage::vdisk_list($cfg, undef, $vmid, undef, 'images');
 
     my $volid_hash = {};
     foreach my $storeid (keys %$info) {
@@ -6189,7 +6188,7 @@ sub rescan {
     my $cfg = PVE::Storage::config();
 
     print "rescan volumes...\n";
-    my $volid_hash = scan_volids($cfg, $vmid, 'images');
+    my $volid_hash = scan_volids($cfg, $vmid);
 
     my $updatefn =  sub {
        my ($vmid) = @_;