Skip to content

[action] [PR:18276] [Mellanox] Remove the skip/xfail for the dualtor_io link failure test#18646

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18276
May 27, 2025
Merged

[action] [PR:18276] [Mellanox] Remove the skip/xfail for the dualtor_io link failure test#18646
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18276

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:

  1. Issue #16161 is fixed.
  2. Some test cases that were failing in active-standby due to issue Test cases in dualtor_io/test_link_failure.py fails on Nvidia platforms due to the flooding packets are counted as duplications #8272 have been passing in active-active. Only test_active_link_down_downstream_active_soc[active-active] is still failing.

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?

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

Documentation

…#18276)

Issue sonic-net#16161 is fixed.
And some test cases that were failing in active-standby due to issue sonic-net#8272 have been passing in active-active.
Only test_active_link_down_downstream_active_soc[active-active] is still failing, but the behavior is not exactly the same as active-standby. Probably we need to debug the failure again in active-active scenario.
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #18276

@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 a5b8d2c into sonic-net:202505 May 27, 2025
17 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