diff --git a/tests/qos/test_tunnel_qos_remap.py b/tests/qos/test_tunnel_qos_remap.py index 74cc8dc5b4e..45441c9f3a2 100644 --- a/tests/qos/test_tunnel_qos_remap.py +++ b/tests/qos/test_tunnel_qos_remap.py @@ -346,14 +346,19 @@ def pfc_pause_test(storm_handler, peer_info, prio, ptfadapter, dut, port, queue, start_pfc_storm(storm_handler, peer_info, prio) ptfadapter.dataplane.flush() # Record the queue counter before sending test packet - base_queue_count = get_queue_counter(dut, port, queue, True) + base_queue_count = get_queue_counter(dut, port, queue, False) # noqa F841 # Send testing packet again testutils.send_packet(ptfadapter, src_port, pkt, 1) # The packet should be paused testutils.verify_no_packet_any(ptfadapter, exp_pkt, dst_ports) # Check the queue counter didn't increase - queue_count = get_queue_counter(dut, port, queue, False) - assert base_queue_count == queue_count + queue_count = get_queue_counter(dut, port, queue, False) # noqa F841 + # after 10 sec delay in queue counter reading, pfc frames sending might actually had already stopped. + # so bounce back packet might still send out, and queue counter increased accordingly. + # and then caused flaky test faiure. + # temporarily disable the assert queue counter here until find a better solution, + # such as reading counter using sai thrift API + # assert base_queue_count == queue_count return True finally: stop_pfc_storm(storm_handler)