Skip to content

[action] [PR:18934] Fix tests/common/snappi_tests/traffic_generation.py#19384

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18934
Jul 3, 2025
Merged

[action] [PR:18934] Fix tests/common/snappi_tests/traffic_generation.py#19384
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18934

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

reboot call is still using the outdated argument plt_reboot_ctrl_overwrite instead of return_after_reconnect that was introduced in #16031

Description of PR

Summary: fix snappi_tests/pfc/warm_reboot/test_pfc_pause_lossless_warm_reboot.py failure
Fixes # #18933

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?

Test is failing with: TypeError: reboot() got an unexpected keyword argument 'plt_reboot_ctrl_overwrite'

How did you do it?

Passed return_after_reconnect instead of removed argument plt_reboot_ctrl_overwrite in reboot() call of common/snappi_tests/traffic_generation.py

How did you verify/test it?

Any platform specific information?

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

Documentation

reboot call is still using the outdated argument `plt_reboot_ctrl_overwrite`
instead of `return_after_reconnect` that was introduced in sonic-net#16031
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #18934

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 3b88a6a into sonic-net:202505 Jul 3, 2025
15 of 17 checks passed
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