Skip to content

Optimize the execution time of everflow testbed test cases on dualtor testbed#24659

Draft
congh-nvidia wants to merge 1 commit into
sonic-net:masterfrom
congh-nvidia:everflow_runtime
Draft

Optimize the execution time of everflow testbed test cases on dualtor testbed#24659
congh-nvidia wants to merge 1 commit into
sonic-net:masterfrom
congh-nvidia:everflow_runtime

Conversation

@congh-nvidia
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
The everflow testbed test use the fixture setup_standby_ports_on_rand_unselected_tor_unconditionally to toggle dualtor muxcable status.
This is a function scope fixture, it takes a few minutes in each test case and wastes 2 hours in the whole tests.
Use a module level fixture setup_standby_ports_on_rand_unselected_tor_unconditionally_module to reduce the run time.

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

Approach

What is the motivation for this PR?

Optimize the execution time of everflow testbed test cases on dualtor testbed

How did you do it?

Use a module level fixture setup_standby_ports_on_rand_unselected_tor_unconditionally_module to reduce the run time.

How did you verify/test it?

Run the tests/everflow/test_everflow_testbed.py on SN4700 dualtor-aa testbed

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

… setup

Signed-off-by: Cong Hou <congh@nvidia.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@congh-nvidia
Copy link
Copy Markdown
Contributor Author

Depends on #24504.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants