[PATCH pve-network v6 4/9] sdn: fabrics: register bgp as a fabric protocol type

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

Add 'bgp' to the pve-sdn-fabric-protocol enum so it can be selected
when creating or updating fabric configurations via the API.

Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
---
 src/PVE/Network/SDN/Fabrics.pm   | 63 +++++++++++++++++++++++++++++++-
 src/PVE/Network/SDN/RouteMaps.pm | 15 ++++++++
 2 files changed, 77 insertions(+), 1 deletion(-)

diff --git a/src/PVE/Network/SDN/Fabrics.pm b/src/PVE/Network/SDN/Fabrics.pm
index fa76367..7e5f09b 100644
--- a/src/PVE/Network/SDN/Fabrics.pm
+++ b/src/PVE/Network/SDN/Fabrics.pm
@@ -51,7 +51,7 @@ PVE::JSONSchema::register_standard_option(
     {
         description => "Type of configuration entry in an SDN Fabric section config",
         type => 'string',
-        enum => ['openfabric', 'ospf', 'wireguard'],
+        enum => ['openfabric', 'ospf', 'wireguard', 'bgp'],
     },
 );
 
@@ -263,6 +263,21 @@ sub node_properties {
                     description => 'List of WireGuard network interfaces for this node.',
                     optional => 1,
                 },
+                {
+                    'instance-types' => ['bgp'],
+                    items => {
+                        type => 'string',
+                        format => {
+                            name => {
+                                type => 'string',
+                                format => 'pve-iface',
+                                description => 'Name of the network interface',
+                            },
+                        },
+                    },
+                    description => 'BGP network interface',
+                    optional => 1,
+                },
             ],
         },
         public_key => {
@@ -350,6 +365,15 @@ sub node_properties {
             type => 'array',
             'type-property' => 'protocol',
             oneOf => [
+                {
+                    type => 'array',
+                    'instance-types' => ['bgp'],
+                    items => {
+                        type => 'string',
+                        enum => ['interfaces', 'ip', 'ip6'],
+                    },
+                    optional => 1,
+                },
                 {
                     type => 'array',
                     'instance-types' => ['openfabric', 'ospf'],
@@ -478,6 +502,34 @@ sub fabric_properties {
                     },
                     optional => 1,
                 },
+                {
+                    type => 'array',
+                    'instance-types' => ['bgp'],
+                    items => {
+                        type => 'string',
+                        format => {
+                            source => {
+                                type => 'string',
+                                description =>
+                                    'The protocol from which to redistribute routes from.',
+                                enum => [
+                                    'connected',
+                                    'kernel',
+                                    'ospf',
+                                    'static',
+                                ],
+                            },
+                            'route-map' => {
+                                type => 'string',
+                                format => 'pve-sdn-route-map-id',
+                                description =>
+                                    'Route map to filter or transform redistributed routes from this source.',
+                                optional => 1,
+                            },
+                        },
+                    },
+                    optional => 1,
+                },
             ],
         },
     };
@@ -497,6 +549,15 @@ sub fabric_properties {
                     },
                     optional => 1,
                 },
+                {
+                    type => 'array',
+                    'instance-types' => ['bgp'],
+                    items => {
+                        type => 'string',
+                        enum => ['redistribute', 'route_filter', 'route_map_in', 'route_map_out'],
+                    },
+                    optional => 1,
+                },
                 {
                     type => 'array',
                     'instance-types' => ['ospf'],
diff --git a/src/PVE/Network/SDN/RouteMaps.pm b/src/PVE/Network/SDN/RouteMaps.pm
index c02a38f..71fa0e7 100644
--- a/src/PVE/Network/SDN/RouteMaps.pm
+++ b/src/PVE/Network/SDN/RouteMaps.pm
@@ -126,6 +126,21 @@ sub check_references {
                 if $controller->{'route-map-out'} eq $route_map_id;
         }
     }
+
+    my $fabrics = PVE::Network::SDN::Fabrics::config()->list_fabrics();
+    for my $fabric_id (keys $fabrics->%*) {
+        my $fabric = $fabrics->{$fabric_id};
+
+        if ($fabric->{route_map_in}) {
+            die "route map $route_map_id still referenced by fabric $fabric_id"
+                if $fabric->{route_map_in} eq $route_map_id;
+        }
+
+        if ($fabric->{route_map_out}) {
+            die "route map $route_map_id still referenced by fabric $fabric_id"
+                if $fabric->{route_map_out} eq $route_map_id;
+        }
+    }
 }
 
 sub route_map_properties {
-- 
2.47.3