summaryrefslogtreecommitdiff
path: root/tools/frr-reload.py
diff options
context:
space:
mode:
Diffstat (limited to 'tools/frr-reload.py')
-rwxr-xr-xtools/frr-reload.py23
1 files changed, 22 insertions, 1 deletions
diff --git a/tools/frr-reload.py b/tools/frr-reload.py
index a9151e3e7d..a7a04be639 100755
--- a/tools/frr-reload.py
+++ b/tools/frr-reload.py
@@ -357,6 +357,7 @@ end
"ip ",
"ipv6 ",
"log ",
+ "mpls",
"no ",
"password ",
"ptm-enable",
@@ -1009,6 +1010,21 @@ if __name__ == '__main__':
# the "neighbor 1.1.1.1 route-map FOO out" line...so we compare the
# configs again to put this line back.
+ # There are many keywords in quagga that can only appear one time under
+ # a context, take "bgp router-id" for example. If the config that we are
+ # reloading against has the following:
+ #
+ # router bgp 10
+ # bgp router-id 1.1.1.1
+ # bgp router-id 2.2.2.2
+ #
+ # The final config needs to contain "bgp router-id 2.2.2.2". On the
+ # first pass we will add "bgp router-id 2.2.2.2" but then on the second
+ # pass we will see that "bgp router-id 1.1.1.1" is missing and add that
+ # back which cancels out the "bgp router-id 2.2.2.2". The fix is for the
+ # second pass to include all of the "adds" from the first pass.
+ lines_to_add_first_pass = []
+
for x in range(2):
running = Config()
running.load_from_show_running()
@@ -1016,8 +1032,13 @@ if __name__ == '__main__':
(lines_to_add, lines_to_del) = compare_context_objects(newconf, running)
+ if x == 0:
+ lines_to_add_first_pass = lines_to_add
+ else:
+ lines_to_add.extend(lines_to_add_first_pass)
+
# Only do deletes on the first pass. The reason being if we
- # configure a bgp neighbor via "neighbor swp1 interface" frr
+ # configure a bgp neighbor via "neighbor swp1 interface" quagga
# will automatically add:
#
# interface swp1