Skip to content

[action] [PR:18144] [Mellanox] Ignore the expected errors in bfd test#509

Merged
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/18144
Jul 9, 2025
Merged

[action] [PR:18144] [Mellanox] Ignore the expected errors in bfd test#509
mssonicbld merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/18144

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

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: sonic-net/sonic-mgmt#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

<!--
Please make sure you've read and understood our contributing guidelines;
https://github.com/sonic-net/SONiC/blob/gh-pages/CONTRIBUTING.md

Please provide following information to help code review process a bit easier:
-->
### Description of PR
<!--
- Please include a summary of the change and which issue is fixed.
- Please also include relevant motivation and context. Where should reviewer start? background context?
- List any dependencies that are required for this change.
-->

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: sonic-net/sonic-mgmt#14025

### Type of change

<!--
- Fill x for your type of change.
- e.g.
- [x] Bug fix
-->

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

### Back port request
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [x] 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
<!--
(If it's a new feature, new test case)
Did you update documentation/Wiki relevant to your implementation?
Link to the wiki page?
-->
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-mgmt#18144

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit a52834f into Azure:202412 Jul 9, 2025
6 of 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.

1 participant