[Snappi] Fix duthost port mapping in multidut snappi tests (PFCWD, ECN) - Githubissues

sonic-net / sonic-mgmt

Configuration management examples for SONiC

Other

173 stars 689 forks source link

[Snappi] Fix duthost port mapping in multidut snappi tests (PFCWD, ECN) #13541

Open sdszhang opened 5 days ago

sdszhang commented 5 days ago

Description of PR

Summary: Fixes #13443

Type of change

[x] Bug fix
[ ] Testbed and Framework(new/improvement)
[ ] Test case(new/improvement)

Back port request

[ ] 202012
[ ] 202205
[ ] 202305
[ ] 202311
[x] 202405

Approach

What is the motivation for this PR?

Fix duthost and port mapping mismatch issue in multidut PFCWD and ECN test cases. PFC test cases will be changed in another PR to avoid large commit.

How did you do it?

Add 'duthost' into get_multidut_snappi_ports, so that we can retrieve duthost based on port information.

How did you verify/test it?

Test it manually. all PFCWD test cases can run successfully. There are test failures but are not related to this change.

Any platform specific information?

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

Documentation

sdszhang commented 5 days ago

@selldinesh @amitpawar12 @rraghav-cisco can you help to review this change? Thanks.