Skip to content

fix import errors in test_pfc_pause_single_lossless_prio_reboot#19634

Merged
yejianquan merged 1 commit intosonic-net:masterfrom
sdszhang:pfc_lossless_fix
Jul 31, 2025
Merged

fix import errors in test_pfc_pause_single_lossless_prio_reboot#19634
yejianquan merged 1 commit intosonic-net:masterfrom
sdszhang:pfc_lossless_fix

Conversation

@sdszhang
Copy link
Copy Markdown
Contributor

@sdszhang sdszhang commented Jul 15, 2025

Description of PR

Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found

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
  • 202505

Approach

What is the motivation for this PR?

Update to use tgen_port_info

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

@sdszhang sdszhang marked this pull request as draft July 15, 2025 14:33
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Jul 15, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Jul 15, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang sdszhang marked this pull request as ready for review July 28, 2025 11:41
@sdszhang
Copy link
Copy Markdown
Contributor Author

sdszhang commented Jul 28, 2025 •
edited
Loading

Test result:

snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio[multidut_port_info0-dut-lc1-1|3] PASSED                                                                   [  4%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio[multidut_port_info0-dut-lc1-1|4] PASSED [  9%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio[multidut_port_info1-dut-lc1-1|3] PASSED [ 13%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio[multidut_port_info1-dut-lc1-1|4] PASSED [ 18%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_counter_check[multidut_port_info0-dut-lc1-1|3] PASSED [ 22%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_counter_check[multidut_port_info0-dut-lc1-1|4] PASSED [ 27%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_counter_check[multidut_port_info1-dut-lc1-1|3] PASSED [ 31%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_counter_check[multidut_port_info1-dut-lc1-1|4] PASSED [ 36%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio[multidut_port_info0] PASSED [ 40%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio[multidut_port_info1] PASSED [ 45%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info0-warm-dut-lc1-1|3] SKIPPED [ 50%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info0-cold-dut-lc1-1|3] PASSED [ 54%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info0-fast-dut-lc1-1|3] SKIPPED [ 59%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info1-warm-dut-lc1-1|3] SKIPPED [ 63%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info1-cold-dut-lc1-1|3] PASSED [ 68%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_single_lossless_prio_reboot[multidut_port_info1-fast-dut-lc1-1|3] SKIPPED [ 72%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info0-warm] SKIPPED [ 77%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info0-cold] PASSED [ 81%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info0-fast] SKIPPED [ 86%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info1-warm] SKIPPED [ 90%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info1-cold] PASSED [ 95%]
snappi_tests/pfc/test_pfc_pause_lossless_with_snappi.py::test_pfc_pause_multi_lossless_prio_reboot[multidut_port_info1-fast] SKIPPED [100%]
------------------------------ live log teardown -------------------------------

@sdszhang
Copy link
Copy Markdown
Contributor Author

sdszhang commented Jul 29, 2025

@amitpawar12 can you review? Thanks.

@sdszhang sdszhang moved this to To be Merged in SONiC Snappi Jul 31, 2025
@yejianquan yejianquan merged commit b165f2b into sonic-net:master Jul 31, 2025
11 checks passed
@github-project-automation github-project-automation bot moved this from To be Merged to Done in SONiC Snappi Jul 31, 2025
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 31, 2025
@sdszhang @mssonicbld
fix import errors (sonic-net#19634)
00e9d21 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Jul 31, 2025

Cherry-pick PR to 202505: #19960

@mssonicbld mssonicbld mentioned this pull request Jul 31, 2025
Merged
11 tasks
mssonicbld pushed a commit that referenced this pull request Jul 31, 2025
@sdszhang @mssonicbld
fix import errors (#19634)
8aa9495 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
@sdszhang sdszhang mentioned this pull request Aug 6, 2025
Closed
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
@sdszhang @nissampa
fix import errors (sonic-net#19634)
8d26d7a 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
ashutosh-agrawal pushed a commit to ashutosh-agrawal/sonic-mgmt that referenced this pull request Aug 14, 2025
@sdszhang @ashutosh-agrawal
fix import errors (sonic-net#19634)
a0c277d 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
@sdszhang sdszhang deleted the pfc_lossless_fix branch September 13, 2025 10:22
kanchanavelusamy pushed a commit to kanchanavelusamy/sonic-mgmt that referenced this pull request Oct 7, 2025
@sdszhang @kanchanavelusamy
fix import errors (sonic-net#19634)
3e63ee6 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
vidyac86 pushed a commit to vidyac86/sonic-mgmt that referenced this pull request Oct 23, 2025
@sdszhang @vidyac86
fix import errors (sonic-net#19634)
79a0251 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
@sdszhang
fix import errors (sonic-net#19634)
bdfc636 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
@sdszhang @gshemesh2
fix import errors (sonic-net#19634)
6dd2b01 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
@sdszhang @AharonMalkin
fix import errors (sonic-net#19634)
6f9af39 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
@sdszhang @gshemesh2
fix import errors (sonic-net#19634)
426468b 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
@sdszhang @venu-nexthop
fix import errors (sonic-net#19634)
a3787be 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
@sdszhang @gshemesh2
fix import errors (sonic-net#19634)
aa6cb22 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
@sdszhang @ytzur1
fix import errors (sonic-net#19634)
7afb566 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
@sdszhang
fix import errors (sonic-net#19634)
5c4b445 
Description of PR
Summary:
Fixes the following import error.

  def test_pfc_pause_single_lossless_prio_reboot(snappi_api,                   # noqa: F811
file /data/tests/snappi_tests/files/helper.py, line 127
  @pytest.fixture(autouse=True)
  def setup_ports_and_dut(
E       fixture 'multidut_port_info' not found
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
 202505
Approach
What is the motivation for this PR?
Update to use tgen_port_info

signed-off-by: jianquanye@microsoft.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@auspham auspham auspham approved these changes

Assignees

No one assigned

Labels

Approved for 202505 branch Included in 202505 branch Request for 202505 branch

Projects

SONiC Snappi
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sdszhang @mssonicbld @auspham @yejianquan