Skip to content

Fix for dualtor_io/test_tor_bgp_failure.py on dt/dt120#17370

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
pragnya-arista:master-bgp-tor-failure
Mar 6, 2025
Merged

Fix for dualtor_io/test_tor_bgp_failure.py on dt/dt120#17370
StormLiangMS merged 1 commit intosonic-net:masterfrom
pragnya-arista:master-bgp-tor-failure

Conversation

@pragnya-arista
Copy link
Copy Markdown
Contributor

@pragnya-arista pragnya-arista commented Mar 5, 2025

Description of PR

Summary:
Fixes #445

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

Improving regex match in 17173

Some examples of the syslog entries encountered when running dualtor_io/test_tor_bgp_failure.py -
ERR bgp#mgmtd[34]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[38]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[32]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect

How did you do it?

Changed regex to match any decimal digits instead of just 33

How did you verify/test it?

Ran dualtor_io/test_tor_bgp_failure.py on DCS-7260CX3 and DCS-7050CX3

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 5d09f4c into sonic-net:master Mar 6, 2025
11 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Mar 6, 2025
What is the motivation for this PR?
Improving regex match in 17173

Some examples of the syslog entries encountered when running dualtor_io/test_tor_bgp_failure.py -
ERR bgp#mgmtd[34]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[38]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[32]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect

How did you do it?
Changed regex to match any decimal digits instead of just 33

How did you verify/test it?
Ran dualtor_io/test_tor_bgp_failure.py on DCS-7260CX3 and DCS-7050CX3
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202411: #17403

kperumalbfn pushed a commit that referenced this pull request Mar 11, 2025
Fix for dualtor_io/test_tor_bgp_failure.py on dt/dt120
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
What is the motivation for this PR?
Improving regex match in 17173

Some examples of the syslog entries encountered when running dualtor_io/test_tor_bgp_failure.py -
ERR bgp#mgmtd[34]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[38]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[32]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect

How did you do it?
Changed regex to match any decimal digits instead of just 33

How did you verify/test it?
Ran dualtor_io/test_tor_bgp_failure.py on DCS-7260CX3 and DCS-7050CX3
OriTrabelsi pushed a commit to OriTrabelsi/sonic-mgmt that referenced this pull request Apr 1, 2025
What is the motivation for this PR?
Improving regex match in 17173

Some examples of the syslog entries encountered when running dualtor_io/test_tor_bgp_failure.py -
ERR bgp#mgmtd[34]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[38]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
ERR bgp#mgmtd[32]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect

How did you do it?
Changed regex to match any decimal digits instead of just 33

How did you verify/test it?
Ran dualtor_io/test_tor_bgp_failure.py on DCS-7260CX3 and DCS-7050CX3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants