Skip to content

[Mellanox] Ignore the expected errors in bfd test#18144

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
congh-nvidia:ignore_bfd_errors
Jul 3, 2025
Merged

[Mellanox] Ignore the expected errors in bfd test#18144
bingwang-ms merged 1 commit intosonic-net:masterfrom
congh-nvidia:ignore_bfd_errors

Conversation

@congh-nvidia
Copy link
Copy Markdown
Contributor

@congh-nvidia congh-nvidia commented Apr 28, 2025

Description of PR

Summary:
On Nvidia paltforms, while running this test BFD session creation can fail sometime resulting in some syslog errors.
THe syncd retries in such a case and is able to create the BFD session subsequently. So these errors can be ignored.

There is already a PR handling the same errors in tests/vxlan/test_vxlan_ecmp.py: #14025

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?

How did you do it?

How did you verify/test it?

Any platform specific information?

Tested on Nvidia plarforms, errors can be ignored.

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).

@congh-nvidia
Copy link
Copy Markdown
Contributor Author

Hi @bingwang-ms could you please approve this PR?

@congh-nvidia
Copy link
Copy Markdown
Contributor Author

Hi @bingwang-ms , kindly reminder. And we should also have this in 202412.

@liat-grozovik
Copy link
Copy Markdown
Collaborator

@bingwang-ms as you are the owner of the referred PR i recommend you to review this one as well.
@congh-nvidia @nhe-NV in general note, is there a common way t have these list ignored as a function to be called from different test instead of having this implemented on every test code.

@nhe-NV
Copy link
Copy Markdown
Contributor

nhe-NV commented Jun 13, 2025

@bingwang-ms as you are the owner of the referred PR i recommend you to review this one as well. @congh-nvidia @nhe-NV in general note, is there a common way t have these list ignored as a function to be called from different test instead of having this implemented on every test code.

HI @liat-grozovik Community currently don't has such was to skip the err msg.

@bingwang-ms bingwang-ms merged commit ed3eb08 into sonic-net:master Jul 3, 2025
15 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 4, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19409

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-mgmt.msft#509

nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
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.

7 participants