Skip to content

[202405]Refactor test_acl.py with multi-thread for multi-dut scenario (#14545)#14561

Merged
yejianquan merged 1 commit intosonic-net:202405from
yejianquan:jianquanye/202405_merge
Sep 13, 2024
Merged

[202405]Refactor test_acl.py with multi-thread for multi-dut scenario (#14545)#14561
yejianquan merged 1 commit intosonic-net:202405from
yejianquan:jianquanye/202405_merge

Conversation

@yejianquan
Copy link
Collaborator

cherry-pick and fix conflict #14545

Description of PR
Summary:
Refactor test_acl.py to better fit for chassis test. It reboot/config_reload the dut one by one, on Cisco Chassis, test_acl spend 12 hours. Use multi-thread to speed up the test.

Approach
What is the motivation for this PR?
Refactor test_acl.py to better fit for chassis test. It reboot/config_reload the dut one by one, on Cisco Chassis, test_acl spend 12 hours. Use multi-thread to speed up the test.

How did you do it?
Use multi-thread to speed up the test.

How did you verify/test it?
Run on physical testbed

co-authorized by: jianquanye@microsoft.com

…net#14545)

Description of PR
Summary:
Refactor test_acl.py to better fit for chassis test.
It reboot/config_reload the dut one by one, on Cisco Chassis, test_acl spend 12 hours.
Use multi-thread to speed up the test.

Approach
What is the motivation for this PR?
Refactor test_acl.py to better fit for chassis test.
It reboot/config_reload the dut one by one, on Cisco Chassis, test_acl spend 12 hours.
Use multi-thread to speed up the test.

How did you do it?
Use multi-thread to speed up the test.

How did you verify/test it?
Run on physical testbed

co-authorized by: jianquanye@microsoft.com
Copy link
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yejianquan yejianquan merged commit 225fb94 into sonic-net:202405 Sep 13, 2024
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