Skip to content

Commit bbd6e3c

Browse files
authored
extend frr reconnect bmp retry interval from 1-2s into 10-15s. (sonic-net#22251)
<!-- Please make sure you've read and understood our contributing guidelines: https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md failure_prs.log Make sure all your commits include a signature generated with `git commit -s` ** If this is a bug fix, make sure your description includes "fixes #xxxx", or "closes #xxxx" or "resolves #xxxx" Please provide the following information: --> #### Why I did it In order to lower risk from FRR side, currently if bmp service is down, FRR will try to reconnect bmp every 2s, update it into 15s instead. ##### Work item tracking - Microsoft ADO **(number only)**:30807821 #### How I did it update bgpd config to adjust the by default reconnect interval. #### How to verify it lower risk, pre-check verification is good enough. <!-- If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012. --> #### Which release branch to backport (provide reason below if selected) <!-- - Note we only backport fixes to a release branch, *not* features! - Please also provide a reason for the backporting below. - e.g. - [x] 202006 --> - [ ] 201811 - [ ] 201911 - [ ] 202006 - [ ] 202012 - [ ] 202106 - [ ] 202111 - [ ] 202205 - [ ] 202211 - [ ] 202305 #### Tested branch (Please provide the tested image version) <!-- - Please provide tested image version - e.g. - [x] 20201231.100 --> - [ ] <!-- image version 1 --> - [ ] <!-- image version 2 --> #### Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: --> <!-- Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU. --> #### Link to config_db schema for YANG module changes <!-- Provide a link to config_db schema for the table for which YANG model is defined Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md --> #### A picture of a cute animal (not mandatory but encouraged)
1 parent 3807de0 commit bbd6e3c

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

dockers/docker-fpm-frr/frr/bgpd/bgpd.main.conf.j2

+1-1
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ router bgp {{ DEVICE_METADATA['localhost']['bgp_asn'] }}
9999
bmp stats interval 1000
100100
bmp monitor ipv4 unicast pre-policy
101101
bmp monitor ipv6 unicast pre-policy
102-
bmp connect 127.0.0.1 port 5000 min-retry 1000 max-retry 2000
102+
bmp connect 127.0.0.1 port 5000 min-retry 10000 max-retry 15000
103103
!
104104
!
105105
{% endif %}

src/sonic-config-engine/tests/sample_output/py2/bgpd_frr_bmp.conf

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ router bgp 65100
6060
bmp stats interval 1000
6161
bmp monitor ipv4 unicast pre-policy
6262
bmp monitor ipv6 unicast pre-policy
63-
bmp connect 127.0.0.1 port 5000 min-retry 1000 max-retry 2000
63+
bmp connect 127.0.0.1 port 5000 min-retry 10000 max-retry 15000
6464
!
6565
!
6666
bgp router-id 10.1.0.32

src/sonic-config-engine/tests/sample_output/py3/bgpd_frr_bmp.conf

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ router bgp 65100
6060
bmp stats interval 1000
6161
bmp monitor ipv4 unicast pre-policy
6262
bmp monitor ipv6 unicast pre-policy
63-
bmp connect 127.0.0.1 port 5000 min-retry 1000 max-retry 2000
63+
bmp connect 127.0.0.1 port 5000 min-retry 10000 max-retry 15000
6464
!
6565
!
6666
bgp router-id 10.1.0.32

0 commit comments

Comments
 (0)