Am 10.04.25 um 12:59 schrieb Markus Frank:
> Currently you get a "Can't use an undefined value..." error because all
> get_node_mapping functions return undefined if there is no resource
> mapping entry for the id.
>
> Fixes: a52eb3c4e ("check local resources: extend for mapped resources")
> Signed-off-by: Markus Frank <m.fr...@proxmox.com>
> ---
> This patch supersedes "fix: print error message that the resource
> mapping entry does not exist"
>
> PVE/QemuServer.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
> index ccdceedc..2790942c 100644
> --- a/PVE/QemuServer.pm
> +++ b/PVE/QemuServer.pm
> @@ -2512,7 +2512,7 @@ sub check_local_resources {
> } elsif ($type eq 'dir') {
> $entry = PVE::Mapping::Dir::get_node_mapping($dir_map, $id,
> $node);
> }
> - if (!scalar($entry->@*)) {
> + if (!$entry || !scalar($entry->@*)) {
> push @{$missing_mappings_by_node->{$node}}, $key;
> }
> }
Looking at this more closely, I think v1 is better actually. Sorry, I
had misunderstood the exact situation when the issue happens.
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
