Skip to content

[dualtor-aa] Add dualtor-aa support to test_nvgre_hash#18870

Merged
kevinskwang merged 2 commits intosonic-net:masterfrom
lolyu:nvgre_hash_dualtor_aa
Jun 10, 2025
Merged

[dualtor-aa] Add dualtor-aa support to test_nvgre_hash#18870
kevinskwang merged 2 commits intosonic-net:masterfrom
lolyu:nvgre_hash_dualtor_aa

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Jun 9, 2025

Description of PR

Summary:
Fixes # (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

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

Approach

What is the motivation for this PR?

Enable test_nvgre_hash run on dualtor-aa testbeds.
This fixes: #17214
Work Item: 32794931

Signed-off-by: Longxiang Lyu [email protected]

How did you do it?

By using dualtor-aa supported ptf port mapping for the nvgre hashing ptf test.

How did you verify/test it?

fib/test_fib.py::test_nvgre_hash[ipv4-ipv4] PASSED                                                               [ 25%]
fib/test_fib.py::test_nvgre_hash[ipv4-ipv6] PASSED                                                               [ 50%]
fib/test_fib.py::test_nvgre_hash[ipv6-ipv6] PASSED                                                               [ 75%]
fib/test_fib.py::test_nvgre_hash[ipv6-ipv4] PASSED                                                               [100%]

====================================== 4 passed, 3 warnings in 1129.28s (0:18:49) ======================================

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

Signed-off-by: Longxiang Lyu <[email protected]>
@lolyu lolyu force-pushed the nvgre_hash_dualtor_aa branch from 0ebe153 to 7ddaaa0 Compare June 9, 2025 14:30
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@kevinskwang kevinskwang merged commit 3f79866 into sonic-net:master Jun 10, 2025
15 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 10, 2025
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #18889

@mssonicbld
Copy link
Copy Markdown
Collaborator

@lolyu PR conflicts with 202411 branch

mssonicbld pushed a commit that referenced this pull request Jun 10, 2025
* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

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

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
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
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

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

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

Signed-off-by: Longxiang Lyu <[email protected]>
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
…8870)

* [dualtor-aa] Add dualtor-aa support to `test_nvgre_hash`

Signed-off-by: Longxiang Lyu <[email protected]>

* Refine fixtures

Signed-off-by: Longxiang Lyu <[email protected]>

---------

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

[Bug] [test_fib] [dualtor aa]: Nvgre and vxlan hash cases failed due to its relay on the active_dut_map which is generated based on mux simulater

4 participants