Skip to content

Commit b62bc9e

Browse files
JibinBaomssonicbld
authored andcommitted
Update qos sai tests (#14342)
1. update qos sai test for sn5400 according to the two PRs below a. #9583 b. #12848 2. To stabilize testQosSaiQSharedWatermark[single_asic-wm_q_shared_lossy], update margin from 4 to 5 for spc4 Change-Id: Ic3919a9123cd968fb9d89013687cf31791684208
1 parent 1c62b30 commit b62bc9e

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

tests/qos/files/mellanox/special_qos_config.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,5 +33,6 @@ qos_params:
3333
pkts_num_margin: 7
3434
wm_pg_shared_lossy:
3535
packet_size: 600
36+
pkts_num_margin: 5
3637
wm_q_shared_lossy:
3738
packet_size: 600

tests/qos/qos_sai_base.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,9 @@ def __getBufferProfile(self, request, dut_asic, os_version, table, port, priorit
352352

353353
# Update profile static threshold value if profile threshold is dynamic
354354
if "dynamic_th" in list(bufferProfile.keys()):
355-
if dut_asic.sonichost.facts['platform'] == "x86_64-nvidia_sn5600-r0":
355+
platform_support_nvidia_new_algorithm_cal_buffer_thr = ["x86_64-nvidia_sn5600-r0",
356+
"x86_64-nvidia_sn5400-r0"]
357+
if dut_asic.sonichost.facts['platform'] in platform_support_nvidia_new_algorithm_cal_buffer_thr:
356358
self.__compute_buffer_threshold_for_nvidia_device(dut_asic, table, port, bufferProfile)
357359
else:
358360
self.__computeBufferThreshold(dut_asic, bufferProfile)

tests/saitests/py3/sai_qos_tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5113,7 +5113,7 @@ def runTest(self):
51135113

51145114
if pkts_num_fill_min:
51155115
assert (q_wm_res[queue] == 0)
5116-
elif 'cisco-8000' in asic_type or "SN5600" in hwsku:
5116+
elif 'cisco-8000' in asic_type or "SN5600" in hwsku or "SN5400" in hwsku:
51175117
assert (q_wm_res[queue] <= (margin + 1) * cell_size)
51185118
else:
51195119
if platform_asic and platform_asic == "broadcom-dnx":

0 commit comments

Comments
 (0)