[PATCH qemu-server v2 1/2] network: do not fail on missing bridge fdb entries when migrating

["Rémi PAETA" <remi.paeta@comaite.com>]

del_nets_bridge_fdb() is called at the end of a live migration, to
remove the forwarding database entries of the NICs on the source
before the VM resumes on the target. If an entry is missing, for
example after a NIC got re-plugged (see bug #6490), 'bridge fdb del'
fails and so does the migration, after the VM state was transferred:
the VM stays paused and locked on the target, and the paused QEMU stays
on the source.

The source tap devices go away with the source QEMU anyway, so only
warn about entries that could not be deleted.

Suggested-by: Gabriel Goller <g.goller@proxmox.com>
Signed-off-by: Rémi PAETA <remi.paeta@comaite.com>
---
 src/PVE/QemuServer/Network.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/PVE/QemuServer/Network.pm b/src/PVE/QemuServer/Network.pm
index b4893c01..01b21f86 100644
--- a/src/PVE/QemuServer/Network.pm
+++ b/src/PVE/QemuServer/Network.pm
@@ -309,7 +309,8 @@ sub del_nets_bridge_fdb {
         my $mac = $net->{macaddr} or next;
 
         my $bridge = $net->{bridge};
-        PVE::Network::SDN::Zones::del_bridge_fdb($iface, $mac, $bridge);
+        eval { PVE::Network::SDN::Zones::del_bridge_fdb($iface, $mac, $bridge) };
+        log_warn("failed to delete forwarding database entries of '$iface' - $@") if $@;
     }
 }
 
-- 
2.47.3