Skip to content

Conversation

@rgarofano-arista
Copy link
Contributor

Description of PR

Summary:
Fixes # (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

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202412
  • 202505

Approach

What is the motivation for this PR?

Packet trimming tests rely on simulating congestion on an egress queue. For this, the tests are trying to apply a shaper that significantly slows down the amount of packets sent out of the target egress queue per second. We found that this shaper doesn't work on th5 with the current config.

How did you do it?

The solution to this is to add "meter_type" packets when creating the blocking scheduler (shaper).

How did you verify/test it?

Verified via tcpdump that packet transmission rate was slowed down as expected on th5.

Any platform specific information?

This issue was discovered when testing on th5

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@davidm-arista davidm-arista added Request for msft-202412 Branch Request for 202511 branch Request to backport a change to 202511 branch labels Nov 26, 2025
@rgarofano-arista rgarofano-arista force-pushed the fix-shaper-packet-trimming branch from d840c81 to 96bbcc8 Compare November 26, 2025 21:37
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Request for msft-202412 Branch Request for 202511 branch Request to backport a change to 202511 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants