sonic-mgmt icon indicating copy to clipboard operation
sonic-mgmt copied to clipboard

New HeadroomPoolSizeTest tests for Nokia-IXR7250E-36x400G & 100G hwsku_broadcom-dnx chassis

Open ansrajpu-git opened this issue 1 year ago • 2 comments

Raising this issue for creating separate test for Nokia-IXR7250E-36x400G & 100G hwsku. Issue found with existing tests: https://github.com/sonic-net/sonic-mgmt/issues/13503

With the introduction of dynamic buffer for broadcom-dnx chassis, the test has to be modified to get appropriate results. This will impact 2 tests in test_qos_sai.py -testQosSaiHeadroomPoolSize -testQosSaiHeadroomPoolWatermark

ansrajpu-git avatar Jul 16 '24 17:07 ansrajpu-git

@arlakshm please help triage.

yxieca avatar Jul 24 '24 15:07 yxieca

@vmittal-msft @ansrajpu-git is this issue still open? Is it applicable of 202205 also?

arlakshm avatar Sep 03 '24 22:09 arlakshm

PR open to skip these test. Issue still valid. @ansrajpu-git to link the issue opened to modify the test

arlakshm avatar Sep 25 '24 17:09 arlakshm

Skip PR #https://github.com/sonic-net/sonic-mgmt/pull/13513 Issue #https://github.com/sonic-net/sonic-mgmt/issues/13503 is closed, since Lossy queue test issue is resolved and PR to skip the Headroom test is raised

ansrajpu-git avatar Sep 25 '24 18:09 ansrajpu-git

HeadroomPoolSizeTest - The new test is based on the same test steps to verify the headroom pool per vsq ,while calculating the shared buffer dynamically. But while test, observed the below issue - Selected no. of src ports = 4, no. of dst_ports =1. 1.Sent packets to fill shared buffer per vsq with no. of packets (less one to trigger pfc) 2.Observe no PFC is triggered. Saw priority4 PFC frames on the vsq, which is already almost filled. But didn't see any PFC frames on the same vsq where packets are sent.

HeadroomPoolTest_P4_pfc_new.txt

ansrajpu-git avatar Nov 13 '24 17:11 ansrajpu-git

PR open for the new test. Needs to merge. @ansrajpu-git to link the test PR to this issue

arlakshm avatar Apr 02 '25 17:04 arlakshm

PR raised for adding new HeadroomPoolSizeTest # https://github.com/sonic-net/sonic-mgmt/pull/16885

ansrajpu-git avatar Apr 02 '25 17:04 ansrajpu-git

Closing this issue as the PR #16885 is merged

ansrajpu-git avatar Apr 23 '25 17:04 ansrajpu-git