Skip to content

Add xfail for generic hash case test_lag_hash due to github issue 22586#19909

Merged
roy-sror merged 1 commit intosonic-net:masterfrom
weiguo-nvidia:skip_22586
Aug 11, 2025
Merged

Add xfail for generic hash case test_lag_hash due to github issue 22586#19909
roy-sror merged 1 commit intosonic-net:masterfrom
weiguo-nvidia:skip_22586

Conversation

@weiguo-nvidia
Copy link
Copy Markdown
Contributor

@weiguo-nvidia weiguo-nvidia commented Jul 30, 2025

Description of PR

Summary: Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586
Fixes #
Add xfail for case hash/test_generic_hash.py::test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Type of change

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

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@roy-sror roy-sror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the xfail should be only on the platforms on which the issue was observed

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Change-Id: Id8c6f67ca40010105ff3c500b2912217879cdfbe
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@weiguo-nvidia
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@roy-sror roy-sror merged commit b5283ad into sonic-net:master Aug 11, 2025
20 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Aug 12, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #20205

mssonicbld pushed a commit that referenced this pull request Aug 12, 2025
…86 (#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586
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
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: Guy Shemesh <[email protected]>
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
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…86 (sonic-net#19909)

Add xfail for generic hash case test_lag_hash due to github issue sonic-net/sonic-buildimage#22586

Signed-off-by: Yael Tzur <[email protected]>
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.

5 participants