proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-04-30 22:26:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

tools: frr-reload fixes for deleting vrf static routes

Browse Source 
Problems reported that in certain cases, frr-reload.py would
delete vrf static routes inadvertantly due to two different
reasons. First, vrf statics with null0 or Null0 nexthops would
fail the match since rendered as blackholes.  This was already
fixed for non-vrf statics so added for vrf-based.  Second,
frr-reload would fail to match due to different formats for
adding the command. If entered in the old way
"ip route x.x.x.x/x y.y.y.y vrf NAME" and rendered
in the new sway "vrf NAME\nip route x.x.x.x/x y.y.y.y" it would
fail to match do an inadvertant delete.

Ticket: 2570270
Signed-off-by: Don Slice <dslice@nvidia.com>
This commit is contained in:
Don Slice 2021-03-19 15:10:14 -04:00
parent 880dcb06e4
commit 00302a580c
1 changed files with 46 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
tools/frr-reload.py
View File

@ -279,6 +279,35 @@ class Config(object):
if ":" in line:
line = get_normalized_mac_ip_line(line)
"""
vrf static routes can be added in two ways. The old way is:
"ip route x.x.x.x/x y.y.y.y vrf <vrfname>"
but it's rendered in the configuration as the new way::
vrf <vrf-name>
ip route x.x.x.x/x y.y.y.y
exit-vrf
this difference causes frr-reload to not consider them a
match and delete vrf static routes incorrectly.
fix the old way to match new "show running" output so a
proper match is found.
"""
if (
line.startswith("ip route ") or line.startswith("ipv6 route ")
) and " vrf " in line:
newline = line.split(" ")
vrf_index = newline.index("vrf")
vrf_ctx = newline[vrf_index] + " " + newline[vrf_index + 1]
del newline[vrf_index : vrf_index + 2]
newline = " ".join(newline)
self.lines.append(vrf_ctx)
self.lines.append(newline)
self.lines.append("exit-vrf")
line = "end"
self.lines.append(line)
self.load_contexts()
@ -460,6 +489,23 @@ class Config(object):
):
key[0] = re.sub(r"\s+null0(\s*$)", " Null0", key[0])
"""
Similar to above, but when the static is in a vrf, it turns into a
blackhole nexthop for both null0 and Null0. Fix it accordingly
"""
if lines and key[0].startswith("vrf "):
newlines = []
for line in lines:
if line.startswith("ip route ") or line.startswith("ipv6 route "):
if "null0" in line:
line = re.sub(r"\s+null0(\s*$)", " blackhole", line)
elif "Null0" in line:
line = re.sub(r"\s+Null0(\s*$)", " blackhole", line)
newlines.append(line)
else:
newlines.append(line)
lines = newlines
if lines:
if tuple(key) not in self.contexts:
ctx = Context(tuple(key), lines)