]> git.puffer.fish Git - mirror/pve-network.git/commit
sdn: fix comparison of pending configuration values
authorStefan Hanreich <s.hanreich@proxmox.com>
Fri, 7 Feb 2025 13:40:37 +0000 (14:40 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Tue, 11 Feb 2025 20:24:28 +0000 (21:24 +0100)
commitbd7c331536ed3aa9ba53b94a9440f4c18b724c26
tree204a087ee3a9fafc2fe6151d1b7008d9f71ba349
parent39fb956fb8f20e7d6e162548db24ac1fb3df31a4
sdn: fix comparison of pending configuration values

The conditional assignment caused falsy values to be converted to
undef when comparing them. This led to the behavior that configuration
values that are interpreted by perl as falsy would get wrongly
compared and always show up as pending changes.

As an example the 'bgp-multipath-as-path-relax' or 'ebgp' keys of the
bgp controller configuration are booleans and get stored as 0 in the
controller configuration when they're turned off. They always showed
up as a pending change, because of the behavior described above.

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