diff options
| author | Alex Kudentsov <43482574+alexk37@users.noreply.github.com> | 2026-10-07 02:25:50 +0700 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2026-10-07 19:50:29 +0200 |
| commit | 057be82cc48c3ccc256270caed4d46e66d822a24 (patch) | |
| tree | 5d1c85048b8a2293387d70c46d9d8e9e71c42f85 /interface-definitions/include/segment-routing/algorithm.xml.i | |
| parent | c5fcf4bcde6582f963e1fb0fb884bfe75bcc8b49 (diff) | |
| download | vyos-1x-057be82cc48c3ccc256270caed4d46e66d822a24.tar.gz vyos-1x-057be82cc48c3ccc256270caed4d46e66d822a24.zip | |
bgp: T9407: do not render "bgp reject-as-sets" when it is the FRR default
Since FRR 10.5 "bgp reject-as-sets" is the default. FRR does not print
the default in show running-config, so frr-reload never finds the line in
the running config and sends it again on every reload. The command
handler resets every BGP session of the instance (NOTIFICATION Cease,
other configuration change) whether or not the value changed,
so with "protocols bgp parameters reject-as-sets" set, every commit that
reloads FRR reset every established session, not only commits that touch
BGP.
Render only the non-default form: "no bgp reject-as-sets" when the
option is not set, as T7664 added it, and nothing when it is set. The
option is a valueless leaf, so it has no XML default to read; the
rendered "no" form already was the only place that knows what FRR's
default is. Changing the option still reaches FRR: frr-reload sends
"bgp reject-as-sets" when the option is set (the "no" line leaves the
rendered config) and "no bgp reject-as-sets" when it is deleted, and
each change resets the sessions once, as before.
Add a smoketest that reads the rendered /run/frr/config/vyos.frr.conf,
because FRR never prints the default form and vtysh cannot show it.
Diffstat (limited to 'interface-definitions/include/segment-routing/algorithm.xml.i')
0 files changed, 0 insertions, 0 deletions
