Skip to content

[action] [PR:17726] Cisco update to lossy queue voq test expectations#19155

Merged
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17726
Sep 11, 2025
Merged

[action] [PR:17726] Cisco update to lossy queue voq test expectations#19155
mssonicbld merged 1 commit intosonic-net:202411from
mssonicbld:cherry/202411/17726

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Separated VOQ'ing is enabled on specific Cisco-8000 hwskus.
Encode test to expect these platforms.

Fixes several cases where GB asics are expecting the wrong configuration.

Alternative solution here would be to auto-detect which platform is running separate vs shared voq'ing, however this would take away some of the value of the test since the configuration could be swapped incorrectly and the test would just adapt to match rather than flagging an issue.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Validated on Platform/hwsku:

  • x86_64-8101_32fh_o-r0/Cisco-8101-32FH-O (shared VOQ)
  • x86_64-8101_32fh_o-r0/Cisco-8101-O8C48 (separate VOQ)

Any platform specific information?

Cisco-specific test case.

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #17726

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@kevinskwang kevinskwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rbpittman
Copy link
Copy Markdown
Contributor

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 5b2bdef into sonic-net:202411 Sep 11, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants