Re: applied: [PATCH manager/network/proxmox{-ve-rs,-perl-rs} v8 00/21] Generate frr config using jinja templates and rust types

[Gabriel Goller <g.goller@proxmox.com>]

On 28.03.2026 17:55, Thomas Lamprecht wrote:
> On Fri, 27 Mar 2026 16:01:03 +0100, Gabriel Goller wrote:
> > Previously we generated the frr config using one big perl hash, where every
> > controller-plugin and zone-plugin would push their stuff. This is not pretty
> > and also tricky to unite with our new rust-based fabrics. Furthermore the only
> > way to edit or override the frr config is currently the frr.conf.local file,
> > which is merged with the perl-hash in a very janky manner, which has sprouted
> > numerous forum threads. The main problem with the frr.conf.local is the limited
> > control which the user has as to where the override or additional config gets
> > placed. There are also a few config overrides or additions to frr.conf.local
> > that are currently impossible or generate invalid frr config.
> > 
> > [...]
> 
> Applied the following parts, thanks!
> 
> proxmox-ve-rs:
> [1/9] ve-config: firewall: cargo fmt
>       commit: 5901f899a77eb853e02dfef33eac1b07f826e7b6
> [2/9] frr: add proxmox-frr-templates package that contains templates
>       commit: 7d912a192d070d7e68b1e6c00b70a9cad1256ad0
> [3/9] ve-config: remove FrrConfigBuilder struct
>       commit: 4712a934875e117144975b11d2b24f7185990adc
> [4/9] sdn-types: support variable-length NET identifier
>       commit: c6c1be318e1f8941e31abf5580f25b519ee2dd12
> [5/9] frr: add template serializer and serialize fabrics using templates
>       commit: 1ca7bb01f6bd175c4b4496b7146bd13bc113a25d
> [6/9] frr: add isis configuration and templates
>       commit: 4198f041b5f7c70a66f2c60885d3d3858946186b
> [7/9] frr: support custom frr configuration lines
>       commit: 9cdffc5eab293455c075c17c3d21b28200a5d048
> [8/9] frr: add bgp support with templates and serialization
>       commit: 900291179b1c0ee12d67d8fbef429bbd7b97958b
> [9/9] frr: enable minijinja strict undefined behavior mode
>       commit: e97efc0574479b9e7ca0a75147f65627dd34caf5

Note that patch 10 is still in the approval queue!