Re: [PATCH pve-network 09/13] sdn: generate route map / prefix list configuration on sdn apply

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

`has_pending_changes` should also include checks for route-maps or
prefix-lists

On 2026-03-25 10:44, Stefan Hanreich wrote:
> Commit the newly introduced configuration files to the running
> configuration when applying the SDN configuration, so the FRR config
> generation logic can use them to generate the FRR configuration for
> them.
> 
> Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
> ---
>  src/PVE/Network/SDN.pm | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/src/PVE/Network/SDN.pm b/src/PVE/Network/SDN.pm
> index 0bb36bf..93b6698 100644
> --- a/src/PVE/Network/SDN.pm
> +++ b/src/PVE/Network/SDN.pm
> @@ -25,6 +25,8 @@ use PVE::Network::SDN::Subnets;
>  use PVE::Network::SDN::Dhcp;
>  use PVE::Network::SDN::Frr;
>  use PVE::Network::SDN::Fabrics;
> +use PVE::Network::SDN::RouteMaps;
> +use PVE::Network::SDN::PrefixLists;
>  
>  my $RUNNING_CFG_FILENAME = "sdn/.running-config";
>  
> @@ -207,12 +209,16 @@ sub compile_running_cfg {
>      my $controllers_cfg = PVE::Network::SDN::Controllers::config();
>      my $subnets_cfg = PVE::Network::SDN::Subnets::config();
>      my $fabrics_cfg = PVE::Network::SDN::Fabrics::config();
> +    my $route_maps_cfg = PVE::Network::SDN::RouteMaps::config();
> +    my $prefix_lists_cfg = PVE::Network::SDN::PrefixLists::config();
>  
>      my $vnets = { ids => $vnets_cfg->{ids} };
>      my $zones = { ids => $zones_cfg->{ids} };
>      my $controllers = { ids => $controllers_cfg->{ids} };
>      my $subnets = { ids => $subnets_cfg->{ids} };
>      my $fabrics = { ids => $fabrics_cfg->to_sections() };
> +    my $route_maps = { ids => $route_maps_cfg->to_sections() };
> +    my $prefix_lists = { ids => $prefix_lists_cfg->to_sections() };
>  
>      $cfg = {
>          version => $version,
> @@ -221,6 +227,8 @@ sub compile_running_cfg {
>          controllers => $controllers,
>          subnets => $subnets,
>          fabrics => $fabrics,
> +        'route-maps' => $route_maps,
> +        'prefix-lists' => $prefix_lists,
>      };
>  
>      return $cfg;
> @@ -425,9 +433,11 @@ configuration.
>  =cut
>  
>  sub generate_frr_raw_config {
> -    my ($running_config, $fabric_config) = @_;
> +    my ($running_config, $fabric_config, $route_map_config, $prefix_list_config) = @_;
>  
>      $running_config = PVE::Network::SDN::running_config() if !$running_config;
> +    $prefix_list_config = PVE::Network::SDN::PrefixLists::config(1) if !$prefix_list_config;
> +    $route_map_config = PVE::Network::SDN::RouteMaps::config(1) if !$route_map_config;
>      $fabric_config = PVE::Network::SDN::Fabrics::config(1) if !$fabric_config;
>  
>      my $frr_config = {};
> @@ -438,7 +448,7 @@ sub generate_frr_raw_config {
>      my $nodename = PVE::INotify::nodename();
>  
>      return PVE::RS::SDN::get_frr_raw_config(
> -        $frr_config->{'frr'}, $fabric_config, $nodename,
> +        $frr_config->{'frr'}, $prefix_list_config, $route_map_config, $fabric_config, $nodename,
>      );
>  }
>