Skip to content

[dualtor-io] Update the ignored kill-bgp errors#17173

Merged
yxieca merged 1 commit intosonic-net:masterfrom
lolyu:fix_bgp_errors
Feb 26, 2025
Merged

[dualtor-io] Update the ignored kill-bgp errors#17173
yxieca merged 1 commit intosonic-net:masterfrom
lolyu:fix_bgp_errors

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Feb 26, 2025

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

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@lolyu lolyu requested a review from yxieca February 26, 2025 07:34
Signed-off-by: Longxiang <lolv@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit df81d2a into sonic-net:master Feb 26, 2025
11 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Feb 28, 2025
Signed-off-by: Longxiang <lolv@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202411: #17239

@pragnya-arista
Copy link
Copy Markdown
Contributor

Updated the regex to match on any decimal digit(s) instead of only 33 in 17370

nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
Signed-off-by: Longxiang <lolv@microsoft.com>
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.

4 participants