Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[T0 T1 merge] update multidut pfcwd tests #16657

Merged

Conversation

auspham
Copy link
Contributor

@auspham auspham commented Jan 23, 2025

Summary: Update the missing T0 T1 tests feature in multidut

Fixes # (issue) 30929279

The following PR were migrated for pfcwd

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

snappi_tests/multidut/pfcwd/test_multidut_pfcwd_m2o_with_snappi.py


-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
- generated xml file: /var/src/sonic-mgmt_vmsvc5-t2-8800-ixia_669de75fbbfea63e8b73b319/tests/logs/snappi_tests/multidut/pfcwd/test_multidut_pfcwd_m2o_with_snappi.xml -
================== 4 passed, 7 warnings in 1235.72s (0:20:35) ==================

snappi_tests/multidut/pfcwd/test_multidut_pfcwd_burst_storm_with_snappi.py

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
- generated xml file: /var/src/sonic-mgmt_vmsvc5-t2-8800-ixia_669de75fbbfea63e8b73b319/tests/logs/snappi_tests/multidut/pfcwd/test_multidut_pfcwd_burst_storm_with_snappi.xml -
================== 2 passed, 5 warnings in 858.60s (0:14:18) ===================

snappi_tests/multidut/pfcwd/test_multidut_pfcwd_basic_with_snappi.py (known failure on T2, fail as expected)

SKIPPED [4] snappi_tests/multidut/pfcwd/test_multidut_pfcwd_basic_with_snappi.py:190: Reboot type fast is not supported on cisco-8000 switches
FAILED snappi_tests/multidut/pfcwd/test_multidut_pfcwd_basic_with_snappi.py::test_pfcwd_basic_multi_lossless_prio_restart_service[multidut_port_info0-True-swss]
===== 1 failed, 23 passed, 16 skipped, 27 warnings in 27779.49s (7:42:59) ======

snappi_tests/multidut/pfcwd/test_multidut_pfcwd_a2a_with_snappi.py

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
- generated xml file: /var/src/sonic-mgmt_vmsvc5-t2-8800-ixia_669de75fbbfea63e8b73b319/tests/logs/snappi_tests/multidut/pfcwd/test_multidut_pfcwd_a2a_with_snappi.xml -
================== 4 passed, 7 warnings in 1157.40s (0:19:17) ==================

snappi_tests/multidut/pfcwd/test_multidut_pfcwd_runtime_traffic_with_snappi.py

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
- generated xml file: /var/src/sonic-mgmt_vmsvc5-t2-8800-ixia_669de75fbbfea63e8b73b319/tests/logs/snappi_tests/multidut/pfcwd/test_multidut_pfcwd_runtime_traffic_with_snappi.xml -
================== 2 passed, 6 warnings in 1162.35s (0:19:22) ==================

fix: migrate sonic-net#7370 to multidut code

Signed-off-by: Austin Pham <[email protected]>

fix: migrate sonic-net#9567 to multidut code

Signed-off-by: Austin Pham <[email protected]>

fix: migrate sonic-net#7367 to multidut code

Signed-off-by: Austin Pham <[email protected]>

fix: migrate sonic-net#7712

Signed-off-by: Austin Pham <[email protected]>

fix: migrate sonic-net#7712 to pfcwd_multidut_runtime_traffic_helper

Signed-off-by: Austin Pham <[email protected]>

chore: update a2a parameterize

Signed-off-by: Austin Pham <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

@auspham auspham requested a review from sdszhang January 23, 2025 21:51
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang
Copy link
Contributor

@auspham can you update the description to include the PRs which are cherry-picked in this change.

@developfast @amitpawar12 @selldinesh @kamalsahu0001 @rraghav-cisco @vmittal-msft for review, this is the T0/T1 merge for pfcwd test cases.

@auspham
Copy link
Contributor Author

auspham commented Jan 24, 2025

@sdszhang updated the prs in description

Copy link
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

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

LGTM

@yejianquan yejianquan merged commit 817a6af into sonic-net:master Jan 29, 2025
11 checks passed
@yejianquan
Copy link
Collaborator

@arlakshm for awareness

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202405: Azure/sonic-mgmt.msft#29

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jan 29, 2025
Summary: Update the missing T0 T1 tests feature in multidut

Fixes # (issue) 30929279

The following PR were migrated for pfcwd

[Snappi] Increase number of calls for ARP wait sonic-net#7370

[Snappi] PFCWD Tolerance Fixes sonic-net#9567

[Snappi] Modifications to pfcwd a2a test design for mlnx 4600c sonic-net#7367 to multidut code

[Snappi] Modified pfcwd tests to add warm-up traffic sonic-net#7712

Signed-off-by: Austin Pham <[email protected]>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #16707

mssonicbld pushed a commit that referenced this pull request Jan 29, 2025
Summary: Update the missing T0 T1 tests feature in multidut

Fixes # (issue) 30929279

The following PR were migrated for pfcwd

[Snappi] Increase number of calls for ARP wait #7370

[Snappi] PFCWD Tolerance Fixes #9567

[Snappi] Modifications to pfcwd a2a test design for mlnx 4600c #7367 to multidut code

[Snappi] Modified pfcwd tests to add warm-up traffic #7712

Signed-off-by: Austin Pham <[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.

4 participants