Enable dynamic port selection from setup in snappi pfcwd test

[ysmanman]

Description of PR

Enable dynamic port selection from setup in snappi pfcwd test

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

• 202012
• 202205
• 202305
• 202311
• 202405
• 202411

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[ysmanman]

@sdszhang @kenneth-arista

[sdszhang]

@ysmanman can you upload the test result once it's ready.

[sdszhang]

After #18067, this part of code needs to be changed.
You can refer to the changes in 4613e88#diff-912c62180b5b6381f1a14d8abc87ff10b45231902932c1f0e4ad8500b2801afe

[sdszhang]

@ysmanman, any update on this one?

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

[ysmanman]

@ysmanman, any update on this one?

Hi @sdszhang addressed the review comments.

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[sdszhang]

@ysmanman can you fix the static anlysis error and upload the test result?

[sdszhang]

@ysmanman , can you fix the static analysis error? Then I can help to verify on our testbed.

[ysmanman]

/Azpw run Azure.sonic-mgmt

[mssonicbld]

/AzurePipelines run Azure.sonic-mgmt

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[ysmanman]

@ysmanman , can you fix the static analysis error? Then I can help to verify on our testbed.

Hi @sdszhang , fixed static analysis error.

[sdszhang]

@ysmanman I got the following error when testing it on my testbed. can you take a look?

_ ERROR at setup of test_pfcwd_basic_single_lossless_prio[multidut_port_info0-tgen_port_info0-True] _
ScopeMismatch: You tried to access the function scoped fixture number_of_tx_rx_ports with a module scoped request object, involved factories:
common/snappi_tests/snappi_fixtures.py:1429:  def snappi_port_selection(get_snappi_ports, number_of_tx_rx_ports, mixed_speed=None)
snappi_tests/pfcwd/test_pfcwd_basic_with_snappi.py:30:  def number_of_tx_rx_ports()

[sdszhang]

@ediwibowo-msft can you pick up this PR and raise a new PR to get this one merged?

[sdszhang]

merged in #23225 with test logs. closing this one.