Skip to content

XFail fib hash/nvgre_hash tests based at issue 18304#18368

Merged
yxieca merged 7 commits intosonic-net:masterfrom
SavchukRomanLv:skip_fib_hash
Jun 16, 2025
Merged

XFail fib hash/nvgre_hash tests based at issue 18304#18368
yxieca merged 7 commits intosonic-net:masterfrom
SavchukRomanLv:skip_fib_hash

Conversation

@SavchukRomanLv
Copy link
Copy Markdown

@SavchukRomanLv SavchukRomanLv commented May 13, 2025

Description of PR

Summary:
Xfail based at #18304. Test fib_hash and test_fib_nvgre_hash are failing due to those 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?

Xfail TC until issue #18304 resolved

How did you do it?

Add xfail

How did you verify/test it?

Run test suite - fib hash xfailed

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

@liat-grozovik
Copy link
Copy Markdown
Collaborator

@yxieca please review and merge

@yxieca yxieca requested review from bingwang-ms and prsunny June 10, 2025 05:14
Copy link
Copy Markdown
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

Skip is not suggested. Please use xfail and limit to the platform(s) you tested and confirmed failure with the known issue.

@yxieca yxieca requested review from Janetxxx and r12f June 10, 2025 05:17
@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).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@AharonMalkin
Copy link
Copy Markdown
Contributor

@yxieca fixed

@AharonMalkin
Copy link
Copy Markdown
Contributor

AharonMalkin commented Jun 15, 2025

@yxieca fixed

Thanks, please limit the xfail to the platforms you have tested. e.g. Mellanox.

Fixed, added hwsku

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca changed the title Skip fib hash/nvgre_hash tests based at issue 18304 XFail fib hash/nvgre_hash tests based at issue 18304 Jun 16, 2025
@yxieca yxieca merged commit 9030f89 into sonic-net:master Jun 16, 2025
16 of 18 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 17, 2025
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19038

mssonicbld pushed a commit that referenced this pull request Jun 22, 2025
What is the motivation for this PR?
Xfail TC until issue #18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

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

nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>

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
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin added a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
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
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com>
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
What is the motivation for this PR?
Xfail TC until issue sonic-net#18304 resolved

How did you do it?
Add xfail

How did you verify/test it?
Run test suite - fib hash xfailed

Co-authored-by: AharonMalkin <94370721+AharonMalkin@users.noreply.github.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.

8 participants