search

sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
173 stars 688 forks source link

[Bug]: testQosSaiHeadroomPoolSize does not take into account cable length, speed, headroom size #13478

Closed SavchukRomanLv closed 4 days ago

SavchukRomanLv commented 6 days ago

Issue Description

Test testQosSaiHeadroomPoolSize calculates all numbers assuming all ingress ports share the same speed, cable length tuple. However, the speed, and cable length tuple could not be the same among all the ingress ports. It calculates the numbers based on a larger headroom but applies it on a smaller headroom, causing drop.

Results you see

TC fails with error message "There are some unexpected 95 packet drop", number of packets can be random, depends on cable

Results you expected to see

TC passed, covers scenario when Testbed has different cables connected to setup

Is it platform specific

generic

Relevant log output

No response

Output of show version

No response

Attach files (if any)

No response

bingwang-ms commented 4 days ago

The issue is caused by cable length change. It's a testbed issue. Will fix the configuration to address the issue.