Skip to content

Change BGP peering timeout to 120#22693

Open
fkong-cisco wants to merge 1 commit intosonic-net:masterfrom
fkong-cisco:fkong-master
Open

Change BGP peering timeout to 120#22693
fkong-cisco wants to merge 1 commit intosonic-net:masterfrom
fkong-cisco:fkong-master

Conversation

@fkong-cisco
Copy link
Contributor

Description of PR
In test_bgp_slb.py, the dynamic peers bring up timeout is set at 40 seconds using the number "40", while in many other tests, it is using WAIT_TIMEOUT, with value of 120. So I'm changing it to do the same.

Other uses of "WAIT_TIMEOUT":

fkong@vxr-slurm-254:/nobackup/fkong/sonic-mgmt-tests/sonic-test/sonic-mgmt/tests/bgp$ grep WAIT_TIMEOUT * | grep "="
grep: reliable_tsabgp_helpers.py:WAIT_TIMEOUT = 120
bgp_helpers.py:TCPDUMP_WAIT_TIMEOUT = 20
: Is a directory
grep: templates: Is a directory
test_bgp_peer_shutdown.py:WAIT_TIMEOUT = 120
test_bgp_slb.py:WAIT_TIMEOUT = 120
test_bgp_update_replication.py:WAIT_TIMEOUT = 120
test_bgp_update_timer.py:WAIT_TIMEOUT = 120
fkong@vxr-slurm-254:/nobackup/fkong/sonic-mgmt-tests/sonic-test/sonic-mgmt/tests/bgp$

Summary:
Fixes # MIGSOFTWAR-33363

Type of change
Script change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

@github-actions github-actions bot requested review from cyw233, lolyu and sanjair-git March 2, 2026 21:35
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants