]> git.puffer.fish Git - matthieu/frr.git/commitdiff
tools: fix frr-reload multiple no description cmds
authorChirag Shah <chirag@nvidia.com>
Wed, 13 Dec 2023 01:31:40 +0000 (17:31 -0800)
committerChristian Breunig <christian@breunig.cc>
Wed, 7 Feb 2024 19:43:14 +0000 (20:43 +0100)
Ensure to change description for index 0 from the list.

Ticket: #3628756
Testing Done:

After fix:
start with three interfaces description delete in lines_to_del:
(Pdb) lines_to_del
[(('interface swp1',), "description swp1 -> sp1's swp1"),
(('interface swp2',), "description swp2 -> sp2's swp
1"), (('interface swp3',), "description swp3 -> sp3's swp1")]

After first iteration swp1:
(Pdb) index
0
(Pdb) lines_to_del
[(('interface swp1',), 'description'), (('interface swp2',),
"description swp2 -> sp2's swp1"), (('interface swp
1s2',), "description swp3 -> sp3's swp1")]

After second iteration swp2:
(Pdb) lines_to_del
[(('interface swp1',), 'description'), (('interface swp2',),
'description'), (('interface swp3',), "description
swp3 -> sp3's swp1")]

After third iteration swp3 fix
(Pdb) lines_to_del
[(('interface swp1',), 'description'), (('interface swp2',),
'description'), (('interface swp3',), 'description'
)]

Signed-off-by: Chirag Shah <chirag@nvidia.com>
(cherry picked from commit 8a5c6e4389a8f56c1a25941977a1f6acaf92f53e)

tools/frr-reload.py

index b39b1836564491df17daf0b3fe378ab1ccf8e51c..ba2d54c96300dfd4f449ba71ae120c3c4f0a8c4d 100755 (executable)
@@ -1090,7 +1090,7 @@ def ignore_delete_re_add_lines(lines_to_add, lines_to_del):
     lines_to_add_to_del = []
     lines_to_del_to_del = []
 
-    index = 0
+    index = -1
     for ctx_keys, line in lines_to_del:
         deleted = False