Re: [PATCH pve-network 01/13] controller: bgp: evpn: adapt to new match / set frr config syntax

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

On 2026-03-25 10:41, Stefan Hanreich wrote:
> proxmox-frr has changed the representation of match / set statements
> in its structs. It has changed to generic key / value fields, where
> the keys specify the literal name of the key in the FRR configuration.
> 
> Adapt the controllers to use the new fields so the configuration gets
> generated properly.
> 
> Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
> ---

[..]

> diff --git a/src/PVE/Network/SDN/Controllers/BgpPlugin.pm b/src/PVE/Network/SDN/Controllers/BgpPlugin.pm
> index 8891541..d54c9ec 100644
> --- a/src/PVE/Network/SDN/Controllers/BgpPlugin.pm
> +++ b/src/PVE/Network/SDN/Controllers/BgpPlugin.pm
> @@ -136,13 +136,12 @@ sub generate_frr_config {
>          $config->{frr}->{protocol_routemaps}->{bgp}->{v4} = "correct_src";
>  
>          my $routemap_config = {
> -            protocol_type => 'ip',
> -            match_type => 'address',
> -            value => { list_type => 'prefixlist', list_name => 'loopbacks_ips' },
> +            key => 'ip address prefix-list',
> +            value => 'loopbacks_ips',

given that
```
+        if ($id =~ m/^(only_default|only_default_v6|loopback_ips|)$/) {
+            return undef if $noerr;
+            die "prefix list ID '$id' is currently reserved and cannot
be used\n";
+        }
```
is added in the next patch(02/13 pve-network), it looks like a `s`
slipped onto the end of `loopback` here :)

>          };
>          my $routemap = {
>              matches => [$routemap_config],
> -            sets => [{ set_type => 'src', value => $ifaceip }],
> +            sets => [{ key => 'src', value => $ifaceip }],
>              action => "permit",
>              seq => 1,
>          };

[...]