diff options
| author | Daniel Walton <dwalton@cumulusnetworks.com> | 2017-11-10 17:57:42 +0000 |
|---|---|---|
| committer | Daniel Walton <dwalton@cumulusnetworks.com> | 2017-11-10 17:57:42 +0000 |
| commit | 4c76e59220604af526cb9d78f1d9ca72c393e918 (patch) | |
| tree | a6a48416b7a2c2f3bca0964e6110acf39748c65b /tools/frr-reload.py | |
| parent | 028bcc883f47002e8469cb1fd4ef64457aebc8a5 (diff) | |
tools: frr-reload remove Cumulus Linux release numbers from comments
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
The "3.0", etc in the comments were referring to Cumulus Linux 3.0 which
was confusing now that FRR has a 3.0
Diffstat (limited to 'tools/frr-reload.py')
| -rwxr-xr-x | tools/frr-reload.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tools/frr-reload.py b/tools/frr-reload.py index c038b32891..fd46cc2143 100755 --- a/tools/frr-reload.py +++ b/tools/frr-reload.py @@ -685,7 +685,7 @@ def ignore_delete_re_add_lines(lines_to_add, lines_to_del): lines_to_add_to_del.append((tmp_ctx_keys, swpx_peergroup)) ''' - In 3.0.1 we changed how we display neighbor interface command. Older + We changed how we display the neighbor interface command. Older versions of frr would display the following: neighbor swp1 interface neighbor swp1 remote-as external @@ -735,13 +735,13 @@ def ignore_delete_re_add_lines(lines_to_add, lines_to_del): lines_to_add_to_del.append((tmp_ctx_keys, swpx_remoteas)) ''' - In 3.0, we made bgp bestpath multipath as-relax command - automatically assume no-as-set since the lack of this option caused - weird routing problems and this problem was peculiar to this - implementation. When the running config is shown in relases after - 3.0, the no-as-set is not shown as its the default. This causes - reload to unnecessarily unapply this option to only apply it back - again, causing unnecessary session resets. Handle this. + We made the 'bgp bestpath as-path multipath-relax' command + automatically assume 'no-as-set' since the lack of this option caused + weird routing problems. When the running config is shown in + releases with this change, the no-as-set keyword is not shown as it + is the default. This causes frr-reload to unnecessarily unapply + this option only to apply it back again, causing unnecessary session + resets. ''' if 'multipath-relax' in line: re_asrelax_new = re.search('^bgp\s+bestpath\s+as-path\s+multipath-relax$', line) |