Skip to content

[action] [PR:21439] Fix GCU packet trimming tests for TH5#21545

Merged
vmittal-msft merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21439
Dec 15, 2025
Merged

[action] [PR:21439] Fix GCU packet trimming tests for TH5#21545
vmittal-msft merged 1 commit intosonic-net:202511from
mssonicbld:cherry/202511/21439

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:

Fixes failing test cases for TH5:

  • test_packet_trimming_config_symmetric.py
  • test_packet_trimming_config_asymmetric.py

This fixes packet size to the supported constant of 206, and configures the trim queue to either the asic default (9) or the per SKU configured value (4, for the C448 and C512)

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

How did you verify/test it?

Manual test runs

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #21439

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@vmittal-msft
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).

@vmittal-msft
Copy link
Copy Markdown
Contributor

@yxieca please help merge.

@vmittal-msft vmittal-msft self-requested a review December 15, 2025 18:01
@vmittal-msft vmittal-msft merged commit bf34f8b into sonic-net:202511 Dec 15, 2025
16 checks passed
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Feb 11, 2026
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