sonic-mgmt
sonic-mgmt copied to clipboard
New HeadroomPoolSizeTest tests for Nokia-IXR7250E-36x400G & 100G hwsku_broadcom-dnx chassis
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
@arlakshm please help triage.
@vmittal-msft @ansrajpu-git is this issue still open? Is it applicable of 202205 also?
PR open to skip these test. Issue still valid. @ansrajpu-git to link the issue opened to modify the test
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
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.
PR open for the new test. Needs to merge. @ansrajpu-git to link the test PR to this issue
PR raised for adding new HeadroomPoolSizeTest # https://github.com/sonic-net/sonic-mgmt/pull/16885
Closing this issue as the PR #16885 is merged