-
Notifications
You must be signed in to change notification settings - Fork 714
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updating LossyQueueVoq test for Cisco-8111 platform #9129
base: 202205
Are you sure you want to change the base?
Conversation
@kevinskwang @yxieca can you please review / merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jsanghra, could you commit to master branch first to make sure PR test pass?
Raised a PR on Master branch : |
tests/saitests/py3/sai_qos_tests.py
Outdated
@@ -3343,28 +3340,25 @@ def runTest(self): | |||
assert diff > 0, "Failed to cause TX drop on port {}".format(self.dst_port_id) | |||
xmit_counters_base = xmit_counters | |||
# Find a separate flow that uses alternate queue | |||
max_iters = 50 | |||
for i in range(max_iters): | |||
for index, (second_pkt, _) in enumerate(pkt_list): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
syntax error, after apply this line to latest 202205:
======================================================================
ERROR: sai_qos_tests.LossyQueueVoqTest
----------------------------------------------------------------------
Traceback (most recent call last):
File \"saitests/py3/sai_qos_tests.py\", line 3378, in runTest
for index, (second_pkt, _) in enumerate(pkt_list):
ValueError: too many values to unpack (expected 2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi ,
The recent merge of PR9362 in 202205 branch caused this error , I have updated this PR to fix that issue .
Description of PR
Updating LossyQueueVoq test for Cisco-8111 platform
What is the motivation for this PR?
Updating LossyQueueVoq test for Cisco-8111 platform
How did you verify/test it?
Verified the changes in local Cisco-8111 setup and the testcase passes .