I initially read this as like it was a recent (code) cleanup that broke it.

Fabian already applied it, so only two suggestion as for how to improve
this for a next similar time..

subject: "qm: fix net regex in 'cleanup' command"

Am 20/09/2022 um 16:50 schrieb Dominik Csapak:
> the '+' needs to be in the brackets, otherwise '$1' is not the right id
> 

Add

Fixes: 3ea84aeb7c75d7b0bfbd9cb04440ad00072bfe0c

> Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
> ---
>  PVE/CLI/qm.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
> index 6a2e161..ca5d25f 100755
> --- a/PVE/CLI/qm.pm
> +++ b/PVE/CLI/qm.pm
> @@ -809,7 +809,7 @@ __PACKAGE__->register_method({
>               # we have to cleanup the tap devices after a crash
>  
>               foreach my $opt (keys %$conf) {
> -                 next if $opt !~  m/^net(\d)+$/;
> +                 next if $opt !~  m/^net(\d+)$/;
>                   my $interface = $1;
>                   PVE::Network::tap_unplug("tap${vmid}i${interface}");
>               }



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to