Skip to content

[action] [PR:17800] Restore fixture disable_packet_aging#17853

Open
mssonicbld wants to merge 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/17800
Open

[action] [PR:17800] Restore fixture disable_packet_aging#17853
mssonicbld wants to merge 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/17800

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
This PR is to fix an issue introduced by #17728

The auto-used PR is still required for the test cases in tests/qos/test_tunnel_qos_remap.py because fixture update_docker_services is only called for several test cases.

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?

This PR is to fix an issue introduced by #17728

How did you do it?

Add the fixture back.

How did you verify/test it?

Verified on a physical SN4700 dualtor testbed.

collected 4 items 

qos/test_tunnel_qos_remap.py::test_xoff_for_pcbb[pcbb_xoff_1] PASSED [ 25%]
qos/test_tunnel_qos_remap.py::test_xoff_for_pcbb[pcbb_xoff_2] PASSED [ 50%]
qos/test_tunnel_qos_remap.py::test_xoff_for_pcbb[pcbb_xoff_3] PASSED [ 75%]
qos/test_tunnel_qos_remap.py::test_xoff_for_pcbb[pcbb_xoff_4] PASSED [100%]

Any platform specific information?

Mellanox platform specific.

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

Not a new test case.

Documentation

@mssonicbld
Copy link
Collaborator Author

Original PR: #17800

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants