Skip to content

Commit da8bdc9

Browse files
arista-nwolfeJavier-Tan
authored andcommitted
Correcting client arguments to dynamically_compensate_leakout (sonic-net#16169)
In sonic-net#8149 the multi-asic and multi-dut variants were added to test_qos_sai.py. This required updating calls to dynamically_compensate_leakout to specify either the src_client or dst_clientbut a couple calls inPGSharedWatermarkTest` passed the wrong client. For more details on the failure this causes see sonic-net#16167 Summary: Fixes sonic-net#16167
1 parent bd39c67 commit da8bdc9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/saitests/py3/sai_qos_tests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4515,7 +4515,7 @@ def runTest(self):
45154515
time.sleep(8)
45164516

45174517
if pg_min_pkts_num > 0 and check_leackout_compensation_support(asic_type, hwsku):
4518-
dynamically_compensate_leakout(self.src_client, asic_type, sai_thrift_read_port_counters,
4518+
dynamically_compensate_leakout(self.dst_client, asic_type, sai_thrift_read_port_counters,
45194519
port_list['dst'][dst_port_id], TRANSMITTED_PKTS,
45204520
xmit_counters_history, self, src_port_id, pkt, 40)
45214521

@@ -4580,7 +4580,7 @@ def runTest(self):
45804580
and (pkts_num <= 1 + margin)
45814581
and check_leackout_compensation_support(asic_type, hwsku)
45824582
):
4583-
dynamically_compensate_leakout(self.src_client, asic_type, sai_thrift_read_port_counters,
4583+
dynamically_compensate_leakout(self.dst_client, asic_type, sai_thrift_read_port_counters,
45844584
port_list['dst'][dst_port_id], TRANSMITTED_PKTS,
45854585
xmit_counters_history, self, src_port_id, pkt, 40)
45864586

0 commit comments

Comments
 (0)