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

Verify no tx_drop counter increment on egress DUT port in oper down state #15705

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sreejithsreekumaran
Copy link
Contributor

@sreejithsreekumaran sreejithsreekumaran commented Nov 22, 2024

Description of PR

Summary:
Fixes # (issue)

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?

Egress DUT port in oper down state shouldn't increase the tx drop counter of the egress port.

How did you do it?

Used Ixia snappi framework to inject traffic and verify the data path.
Bring down the Ixia port to get the egress DUT port to go to Oper down state.
Check for no tx_drop counter increment after 90 seconds.

How did you verify/test it?

On a Ixia connected DUT

Any platform specific information?

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

Documentation

@alpeshspatel
Copy link
Contributor

FYI: @kevinwangsk @wsycqyz for the test

Copy link
Contributor

@bpar9 bpar9 left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants