Skip to content

[action] [PR:17173] [dualtor-io] Update the ignored kill-bgp errors#17239

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17173
Mar 4, 2025
Merged

[action] [PR:17173] [dualtor-io] Update the ignored kill-bgp errors#17239
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17173

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Fixes # (issue)

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?

The following errors should be ignored as bgpd is killed:

E Match Messages:
E 2025 Feb 5 23:19:11.881870 str2-7050cx3-acs-07 ERR bgp#bgpmon: *ERROR* Failed with rc:1 when execute: ['vtysh', '-H', '/dev/null', '-c', 'show bgp summary json']
E 
E 2025 Feb 5 23:19:25.232074 str2-7050cx3-acs-07 ERR bgp#mgmtd[33]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect
E 
E 2025 Feb 5 23:19:25.565904 str2-7050cx3-acs-07 ERR bgp#mgmtd[33]: [X3G8F-PM93W] BE-adapter: mgmt_msg_read: got EOF/disconnect

Signed-off-by: Longxiang lolv@microsoft.com

How did you do it?

Add regex strings to the ignore list.

How did you verify/test it?

dualtor_io/test_tor_bgp_failure.py::test_standby_tor_kill_bgpd_downstream_active[active-standby] PASSED [100%]

============================================================================= 1 passed, 14 warnings in 615.54s (0:10:15) =============================================================================

Any platform specific information?

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

Documentation

Signed-off-by: Longxiang <lolv@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #17173

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit e2448f3 into sonic-net:202411 Mar 4, 2025
14 checks passed
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.

2 participants