tools: Allow deleting of interfaces

FRR reload does not allow removing interfaces as a complete context. Instead it removes each corresponding line of the interface context. However, this leads to errors when an interface is already removed on the system. Since we can do "no interface <ifname>" inside frrouting, this change introduces the ability to delete interfaces. Signed-off-by: Julian Klaiber <[email protected]>
FRRouting · Sep 3, 2024 · 8c23b84 · 8c23b84
1 parent 745cd3e
commit 8c23b84
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/tools/frr-reload.py b/tools/frr-reload.py
@@ -1740,10 +1740,12 @@ def compare_context_objects(newconf, running):
                 delete_bgpd = True
                 lines_to_del.append((running_ctx_keys, None))
 
-            # We cannot do 'no interface' or 'no vrf' in FRR, and so deal with it
+            elif running_ctx_keys[0].startswith("interface"):
+                lines_to_del.append((running_ctx_keys, None))
+
+            # We cannot do 'no vrf' in FRR, and so deal with it
             elif (
-                running_ctx_keys[0].startswith("interface")
-                or running_ctx_keys[0].startswith("vrf")
+                running_ctx_keys[0].startswith("vrf") 
                 or running_ctx_keys[0].startswith("router pim")
             ):
                 for line in running_ctx.lines: