[PATCH qemu-server] fix #7828: register unused drive: fix failure to handle non-PVE managed volumes

[Fiona Ebner <f.ebner@proxmox.com>]

Volumes that are not PVE-managed have no storage associated to them,
and in particular cannot be owned by a VM. When removing such a volume
from the configuration, registering it as unused should be skipped.
This was the case before commit 5b5c5768 ("display warnings for
storage errors or if storage no longer exists"), but the new handling
misses this edge case and causes a failure to parse the volume ID
instead. Fix the regression by using the non-erroring variant when
parsing the volume ID and skipping such volumes again.

Fixes: 5b5c5768 ("display warnings for storage errors or if storage no longer exists")
Analyzed-by: Thomas Ellmenreich <t.ellmenreich@proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 src/PVE/QemuServer.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/PVE/QemuServer.pm b/src/PVE/QemuServer.pm
index 191ae549..9aec7f9c 100644
--- a/src/PVE/QemuServer.pm
+++ b/src/PVE/QemuServer.pm
@@ -1576,7 +1576,8 @@ sub vmconfig_register_unused_drive {
         delete $conf->{'special-sections'}->{cloudinit};
     } elsif (!drive_is_cdrom($drive)) {
         my $volid = $drive->{file};
-        my ($storeid, undef) = PVE::Storage::parse_volume_id($volid);
+        my ($storeid, undef) = PVE::Storage::parse_volume_id($volid, 1);
+        return if !$storeid; # Not a PVE-managed volume, don't register as unused.
         if (PVE::Storage::storage_config($storecfg, $storeid, 1)) {
             if (vm_is_volid_owner($storecfg, $vmid, $volid)) {
                 PVE::QemuConfig->add_unused_volume($conf, $volid, $vmid);
-- 
2.47.3