[PATCH pve-network v3 6/8] sdn: evpn: validate fabric has prefix at controller update time

[Hannes Laimer <h.laimer@proxmox.com>]

BGP fabrics can be created without an IP prefix. Without this check, an
EVPN controller could reference such a fabric and only fail at config
generation time.
---
 src/PVE/Network/SDN/Controllers/EvpnPlugin.pm | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/PVE/Network/SDN/Controllers/EvpnPlugin.pm b/src/PVE/Network/SDN/Controllers/EvpnPlugin.pm
index ac92ea8b..f9feb35b 100644
--- a/src/PVE/Network/SDN/Controllers/EvpnPlugin.pm
+++ b/src/PVE/Network/SDN/Controllers/EvpnPlugin.pm
@@ -634,6 +634,15 @@ sub on_update_hook {
         die "must have exactly one of peers / fabric defined"
             if ($controller->{peers} && $controller->{fabric})
             || !($controller->{peers} || $controller->{fabric});
+
+        if ($controller->{fabric}) {
+            my $fabric_config = PVE::Network::SDN::Fabrics::config(1);
+            my $fabric = eval { $fabric_config->get_fabric($controller->{fabric}) };
+            die "fabric '$controller->{fabric}' does not exist: $@" if $@;
+            die
+                "fabric '$controller->{fabric}' needs an IPv4 or IPv6 prefix to serve as an EVPN underlay"
+                if !$fabric->{ip_prefix} && !$fabric->{ip6_prefix};
+        }
     }
 }
 
-- 
2.47.3