sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
176 stars 700 forks source link

[action] [PR:13452] Enhanced Cisco specific ECN marking config verification test #13535

Closed mssonicbld closed 2 months ago

mssonicbld commented 2 months ago

Description of PR

Summary:

Needed to uplift the case to Validate the ECN configuration

Summary: Fixes # (issue)

1) read PORT_QOS_MAP and check for interfaces and pfc_enable configuration 2) Skip verification if pfc_enable is not set 3) Verify for each PG present in pfc_enable and remove assumption of 3 and 4 as lossless PG

Type of change

Back port request

Approach

What is the motivation for this PR?

the existing script was verifying marking configuration for only lossless 3 and 4. Uplifted to make it generic by reading the configuration for lossless PGs.

How did you do it?

How did you verify/test it?

Verified it on Cisco box 1) By having interfaces with no PFC configuration 2) Having interfaces with selected PFC configuration

Any platform specific information?

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

Documentation

mssonicbld commented 2 months ago

Original PR: https://github.com/sonic-net/sonic-mgmt/pull/13452

mssonicbld commented 2 months ago

/azp run Azure.sonic-mgmt

azure-pipelines[bot] commented 2 months ago
Azure Pipelines successfully started running 1 pipeline(s).
mssonicbld commented 2 months ago

/azp run Azure.sonic-mgmt

azure-pipelines[bot] commented 2 months ago
Azure Pipelines successfully started running 1 pipeline(s).
mssonicbld commented 2 months ago

/azp run Azure.sonic-mgmt

azure-pipelines[bot] commented 2 months ago
Azure Pipelines successfully started running 1 pipeline(s).