Skip to content

Refactor test_acl.py with multi-thread for multi-dut scenario#14545

Merged
yejianquan merged 3 commits intosonic-net:masterfrom
yejianquan:jianquanye/test_acl_refine
Sep 13, 2024
Merged

Refactor test_acl.py with multi-thread for multi-dut scenario#14545
yejianquan merged 3 commits intosonic-net:masterfrom
yejianquan:jianquanye/test_acl_refine

Conversation

@yejianquan
Copy link
Collaborator

@yejianquan yejianquan commented Sep 12, 2024

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.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

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

Any platform specific information?

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

Documentation

@yejianquan yejianquan force-pushed the jianquanye/test_acl_refine branch from c5e03e1 to 7334e9a Compare September 12, 2024 08:14
code refine

Refine config reload from minigraph check for chassis
@yejianquan yejianquan force-pushed the jianquanye/test_acl_refine branch from 7334e9a to f89bc49 Compare September 12, 2024 08:20
@yejianquan yejianquan changed the title Refine test_acl.py with multi-thread for multi-dut scenario Refactor test_acl.py with multi-thread for multi-dut scenario Sep 13, 2024
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

@mssonicbld
Copy link
Collaborator

@yejianquan PR conflicts with 202405 branch

yejianquan added a commit to yejianquan/sonic-mgmt that referenced this pull request Sep 13, 2024
…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
yejianquan added a commit that referenced this pull request Sep 13, 2024
#14561)

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
hdwhdw pushed a commit to hdwhdw/sonic-mgmt that referenced this pull request Sep 20, 2024
…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
arista-hpandya pushed a commit to arista-hpandya/sonic-mgmt that referenced this pull request Oct 2, 2024
…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
vikshaw-Nokia pushed a commit to vikshaw-Nokia/sonic-mgmt that referenced this pull request Oct 23, 2024
…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
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.

3 participants