]> git.puffer.fish Git - mirror/pve-network.git/commit
frr: enable and start frr on reloading the controller config
authorStefan Hanreich <s.hanreich@proxmox.com>
Tue, 8 Apr 2025 22:15:13 +0000 (00:15 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Wed, 9 Apr 2025 06:12:01 +0000 (08:12 +0200)
commitdb03d26176dafcbd1c1f15fa928cd0d3eba3f376
tree2216b8d105e1dc0076ccb4598275317e2b42c7d9
parent873b1431d671295809a073eb93e85d4e9e7789b3
frr: enable and start frr on reloading the controller config

Since we now ship frr with Proxmox VE, the frr service is available on
the nodes but disabled on install. Prior to that, users had to
manually install frr, which automatically enabled the service. When
first applying a SDN configuration with an EVPN controller, we always
fell back to restarting the frr service, because reloading fails when
the daemon isn't running. This fallback to restarting leads to the
service running but still being in the disabled state. This means that
the EVPN setup is working until the next reboot. To avoid the
situation where users configure an EVPN controller and everything
seems to be working, until a restart breaks the EVPN setup,
additionally enable and start the frr service before trying to reload
the configuration.

We enable the service after checking for the existence of
frr-pythontools in order to avoid the situation where users apply an
SDN configuration with an EVPN controller, but reloading fails due to
a missing frr-pythontools package. Since we do an early return there,
we never fell back to restarting the service in case frr-pythontools
was not available. If we enabled the service before the check, the
configuration would apply after a reboot since it already got written
to the frr configuration file.

Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
src/PVE/Network/SDN/Controllers/EvpnPlugin.pm