diff options
| author | Chirag Shah <chirag@nvidia.com> | 2023-11-28 18:54:14 -0800 |
|---|---|---|
| committer | Chirag Shah <chirag@nvidia.com> | 2023-11-28 19:04:22 -0800 |
| commit | 00b96f70df4bfa40b0676b180b01df49d7546e33 (patch) | |
| tree | 66f34062e64533a530d193fc4ac175f78f1cb5cd /tools/frr-reload.py | |
| parent | bb6fe6beb9dc5ca891e064576644be0fd160da15 (diff) | |
tools: fix frr-reload interface desc cmd
Fix frr-reload script to only render 'no description'
rather than 'no description blah'
Ticket:#3628756
Testing Done:
Before:
2023-11-29 02:38:55,758 INFO: Failed to execute interface hostbond_1
no description hostbond_1_to_host exit
2023-11-29 02:38:55,758 ERROR: "interface hostbond_1 -- no description
hostbond_1_to_host -- exit" we failed to remove this command
2023-11-29 02:38:55,758 ERROR: % Unknown command: no description
hostbond_1_to_host
Signed-off-by: Chirag Shah <chirag@nvidia.com>
Diffstat (limited to 'tools/frr-reload.py')
| -rwxr-xr-x | tools/frr-reload.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/tools/frr-reload.py b/tools/frr-reload.py index 8fc3469f6d..b39b183656 100755 --- a/tools/frr-reload.py +++ b/tools/frr-reload.py @@ -1106,6 +1106,17 @@ def ignore_delete_re_add_lines(lines_to_add, lines_to_del): lines_to_del.remove((ctx_keys, line)) lines_to_del.insert(index, (ctx_keys, "description")) + # interface x ; description blah + # no form of description does not accept any argument, + # strip arg before rendering + if ( + ctx_keys[0].startswith("interface ") + and line + and line.startswith("description ") + ): + lines_to_del.remove((ctx_keys, line)) + lines_to_del.insert(index, (ctx_keys, "description")) + # If there is a change in the segment routing block ranges, do it # in-place, to avoid requesting spurious label chunks which might fail if line and "segment-routing global-block" in line: |