Skip to content

Commit

Permalink
Disable pfcwd for pfc_lossless. (#15887)
Browse files Browse the repository at this point in the history
Description of PR
The pfc lossless test assume pfcwd is disabled. But this test: test_pfc_pause_multi_lossless_prio is not provided the fixture: "disable_pfcwd" which is used by all other tests in the same script. This causes this test to fail since the pfcwd is triggered and packets are dropped.

Approach
What is the motivation for this PR?
this test is failing with this error:
Failed: Total TX bytes   841563217920 should be smaller than DUT buffer size 67108864

How did you do it?
Disable pfcwd during the test using the fixture, that is already used by other tests in the same script.

How did you verify/test it?
Ran it on my TB:

co-authorized by: jianquanye@microsoft.com
  • Loading branch information
rraghav-cisco authored and mssonicbld committed Dec 6, 2024
1 parent 71690ab commit c28f591
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ def test_pfc_pause_multi_lossless_prio(snappi_api, # noqa: F81
lossless_prio_list, # noqa: F811
get_snappi_ports, # noqa: F811
tbinfo,
disable_pfcwd, # noqa: F811
setup_ports_and_dut): # noqa: F811

"""
Expand Down

0 comments on commit c28f591

Please sign in to comment.