a1365b44c3
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>
26 lines
907 B
Django/Jinja
26 lines
907 B
Django/Jinja
[Unit]
|
|
Description=BGP container
|
|
Requires=database{% if multi_instance == 'true' %}@%i{% endif %}.service
|
|
After=database{% if multi_instance == 'true' %}@%i{% endif %}.service
|
|
Requires=updategraph.service
|
|
After=updategraph.service
|
|
BindsTo=sonic.target
|
|
After=sonic.target
|
|
Before=ntp-config.service
|
|
After=swss{% if multi_instance == 'true' %}@%i{% endif %}.service
|
|
After=interfaces-config.service
|
|
StartLimitIntervalSec=1200
|
|
StartLimitBurst=3
|
|
|
|
[Service]
|
|
User={{ sonicadmin_user }}
|
|
ExecStartPre=/usr/local/bin/{{docker_container_name}}.sh start{% if multi_instance == 'true' %} %i{% endif %}
|
|
ExecStart=/usr/local/bin/{{docker_container_name}}.sh wait{% if multi_instance == 'true' %} %i{% endif %}
|
|
ExecStop=/usr/local/bin/{{docker_container_name}}.sh stop{% if multi_instance == 'true' %} %i{% endif %}
|
|
ExecStopPost=/usr/local/bin/write_standby.py --shutdown bgp
|
|
|
|
RestartSec=30
|
|
|
|
[Install]
|
|
WantedBy=sonic.target
|