rraghav-cisco

@XuChen-MSFT , @abdosi : pls review.

@XuChen-MSFT , @abdosi : Any comments? Should I move this change to be only for chassis-packet ?

Verified on T2: ``` =========================================================================================================== PASSES =========================================================================================================== _____________________________________________________________________________________ TestQosSai.testQosSaiPfcXonLimit[multi_dut-xon_1] ______________________________________________________________________________________ _____________________________________________________________________________________ TestQosSai.testQosSaiPfcXonLimit[multi_dut-xon_2] ______________________________________________________________________________________ ------------------------------ generated xml file: /run_logs/verify-10319/2024-03-08-18-44-06/AJ-100g-SS-gb-pac/qos/test_qos_sai.py::TestQosSai::testQosSaiPfcXonLimit_2024-03-08-18-44-06.xml ------------------------------- INFO:root:Can not get Allure report URL. Please check logs --------------------------------------------------------------------------------------------------- live...

> can you please rebase and only keep the relevant changes under cisco specific change. @abdosi : Simplified, and verified on both T2 and T1.

@abdosi : Verified the diff is passing for T1: ====================================================================================================== warnings summary ====================================================================================================== ../../usr/local/lib/python3.8/dist-packages/_yaml/__init__.py:18 /usr/local/lib/python3.8/dist-packages/_yaml/__init__.py:18: DeprecationWarning: The _yaml extension module is now located at yaml._yaml and its location is subject...

I applied this diff, but the problem persists. The DUT counts it as a non-global pause: ``` root@yy39top-lc4:/home/cisco# show pfc count | head Last cached time was 2024-04-11T21:41:50.139290 Port Rx...

@kevinskwang : Randall has approved the PR. Pls review.

@kevinskwang , I have updated this PR to include only the traffic-pfcwd-triggered test. I will generate another PR with a different test which verifies that the pfcwd is not triggered...

@lipxu , I have run both with and without the -e --fake-storm=True option:=========================================================================================================== PASSES =========================================================================================================== ________________________________________________________________________________________ TestPfcwdFunc.test_pfcwd_actions[mth-t0-64] _________________________________________________________________________________________ _______________________________________________________________________________________ TestPfcwdFunc.test_pfcwd_multi_port[mth-t0-64] _______________________________________________________________________________________ _______________________________________________________________________________________ TestPfcwdFunc.test_pfcwd_mmu_change[mth-t0-64] _______________________________________________________________________________________ ______________________________________________________________________________________ TestPfcwdFunc.test_pfcwd_port_toggle[mth-t0-64] _______________________________________________________________________________________ ---------------------------------------------------------------------------- generated xml...

@kevinskwang , @lipxu : Pls review.