tools: frr-reload removes "ipv6 nd ra-interval" and "no ipv6 nd suppress-ra"

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

diff --git a/tools/frr-reload.py b/tools/frr-reload.py
index 2dbc60d1f4835be41c713014eeddbdedbd691d75..80e259961a6db71b0592d3f5c0840c21d6520375 100755 (executable)
--- a/tools/frr-reload.py
+++ b/tools/frr-reload.py
@@ -993,7 +993,20 @@ if __name__ == '__main__':
 
             (lines_to_add, lines_to_del) = compare_context_objects(newconf, running)
 
-            if lines_to_del:
+            # Only do deletes on the first pass. The reason being if we
+            # configure a bgp neighbor via "neighbor swp1 interface" quagga
+            # will automatically add:
+            #
+            # interface swp1
+            #  ipv6 nd ra-interval 10
+            #  no ipv6 nd suppress-ra
+            # !
+            #
+            # but those lines aren't in the config we are reloading against so
+            # on the 2nd pass they will show up in lines_to_del.  This could
+            # apply to other scenarios as well where configuring FOO adds BAR
+            # to the config.
+            if lines_to_del and x == 0:
                 for (ctx_keys, line) in lines_to_del:
 
                     if line == '!':