Skip to content
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

Reg: New Test Case of SAI VOQ Counter #16222

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vikshaw-Nokia
Copy link
Contributor

Description of PR

Summary:
Refer:#13926. Original PR with Comments
New Test Cases for SAI_SWITCH_STAT_PACKET_INTEGRITY_DROP counter in show dropcounter counts command and show SAI_QUEUE_STAT_CREDIT_WD_DELETED_PACKETS counters in show queue counter --voq command.
With Reference of sonic-net/sonic-utilities#3322

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

New Test Case for New SAI VOQ Counter

How did you do it?

For 'test_voq_queue_counter'.
Disabling the fabric port using BCMCMD causes the queue counters Credit-WD-Del/pkts to increase on VOQ0 for Ethernet-IBs

For test_voq_drop_counter.
Please refer: #16140

How did you verify/test it?

Any platform specific information?

Verified on a T2 VOQ Chassis with dnx platform

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

T2 VOQ Chassis

Documentation

Refer sonic-net/sonic-utilities#3322
Refer #13926

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -2277,6 +2277,13 @@ voq/test_fabric_reach.py:
- "not any(i in platform for i in ['arista_7800', 'x86_64-nokia_ixr7250e'])"
- "(asic_type in ['cisco-8000'])"

voq/test_voq_counter.py::test_voq_drop_counter:
Copy link
Contributor

Choose a reason for hiding this comment

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

I have updated the issue with information on how to trigger this. Please try and let me know.

@vmittal-msft
Copy link
Contributor

Please rebase as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants