matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[BGP] starting BGP service after swss (#12381)

Browse Source 
Why I did it
BGP service has always been starting after interface-config. However, recently we discovered an issue where some BGP sessions are unable to establish due to BGP daemon not able to read the interface IP.

This issue was clearly observed after upgrading to FRR 8.2.2. See more details in #12380.

How I did it
Delaying starting BGP seems to be a workaround for this issue.

However, caution is that this delay might impact warm reboot timing and other timing sequences.

This workaround is reducing the probability of hitting the issue by close to 100X. However, this workaround is not bulletproof as test shows. It is still preferrable to have a proper FRR fix and revert this change in the future.

How to verify it
Continuously issuing config reload and check BGP session status afterwards.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
This commit is contained in:
Ying Xie 2022-10-13 09:24:06 -07:00 committed by Ying Xie
parent 73805fb298
commit a1365b44c3
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
files/build_templates/per_namespace/bgp.service.j2
View File

@ -7,6 +7,7 @@ After=updategraph.service
BindsTo=sonic.target BindsTo=sonic.target
After=sonic.target After=sonic.target
Before=ntp-config.service Before=ntp-config.service
After=swss{% if multi_instance == 'true' %}@%i{% endif %}.service
After=interfaces-config.service After=interfaces-config.service
StartLimitIntervalSec=1200 StartLimitIntervalSec=1200
StartLimitBurst=3 StartLimitBurst=3