From: Daniel Walton <dwalton@cumulusnetworks.com>
Date: Fri, 17 Feb 2017 14:05:56 +0000 (+1200)
Subject: tools: frr-reload removes "ipv6 nd ra-interval" and "no ipv6 nd suppress-ra"
X-Git-Tag: frr-3.0-branchpoint~24^2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=53bddc224c88c8457ae051ff0e865e6ab72cbe62;p=matthieu%2Ffrr.git

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 2dbc60d1f4..80e259961a 100755
--- 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 == '!':