[ACL] Add acl stress test#6903
Merged
xwjiang-ms merged 1 commit intosonic-net:masterfrom Nov 29, 2022
Merged
Conversation
What is the motivation for this PR? Dynamic acl will add/delete acl entries frequently, need a testcase to test stress acl. Issue link: sonic-net#6801 How did you do it? Send a template sh file to dut, run 1000 times add/delete acl, and check dut status How did you verify/test it? Run testcase Any platform specific information? Supported testbed topology if it's a new test case?
wsycqyz
approved these changes
Nov 28, 2022
Collaborator
|
A few comments on this test case
|
wangxin
pushed a commit
that referenced
this pull request
Dec 7, 2022
What is the motivation for this PR? Dynamic acl will add/delete acl entries frequently, need a testcase to test stress acl. Issue link: #6801 How did you do it? Send a template sh file to dut, run 1000 times add/delete acl, and check dut status How did you verify/test it? Run testcase Any platform specific information? Supported testbed topology if it's a new test case?
Collaborator
|
@xwjiang-ms add one more comment, we cannot assume all topo have 4 port channel. I think the intention here is to add/del acl on up links, can we use conn_graph to find up links (or port channels)? |
wangxin
pushed a commit
that referenced
this pull request
Jan 30, 2023
What is the motivation for this PR? Dynamic acl will add/delete acl entries frequently, need a testcase to test stress acl. Issue link: #6801 How did you do it? Send a template sh file to dut, run 1000 times add/delete acl, and check dut status How did you verify/test it? Run testcase Any platform specific information? Supported testbed topology if it's a new test case?
bingwang-ms
pushed a commit
to bingwang-ms/sonic-mgmt
that referenced
this pull request
Jul 27, 2023
…ic-mgmt into internal-202205 Fix merge conflicts. - [pre-commit] Fix style issues in test scripts under `tests/acl` folder (sonic-net#6679) - Moving check for reboot cause after interface status check (sonic-net#6721) - Adding watchdog timeout values for Cisco 8808 Supervisor and Different LCs (sonic-net#6776) - add Ether check in macsec_dp_poll (sonic-net#6828) - Disable PFC watchdog in test_cpu_memory_usage_counterpoll (sonic-net#6851) - Testcase to verify that lossless traffic is not dropped during congesion. (sonic-net#6853) - Ignore Broadcom sai sai unbind ERR log for now (sonic-net#6539) - [chassis][multi-asic] update the loganalyser regex for multi asic (sonic-net#6885) - [mx] Fix test_acl failed on mx topo (sonic-net#6971) (sonic-net#6983) - [202205][mx] Add support for mx in test_null_route_helper (sonic-net#6967) (sonic-net#6982) - [m0][everflow] Add m0 support for everflow and refactor everflow setup_info (sonic-net#6900) - [ACL] Add acl stress test (sonic-net#6903) - Enhance test_tor_ecn (sonic-net#6906) - Fix erros - Added unique IPV6 address for the missed ACL rules PR sonic-net#6390 (sonic-net#6909) - enabled bfd tests (sonic-net#6919) - Skip bgp speaker test on backend topo (sonic-net#6922) - [advanced-reboot] Handle logs in tmpfs: backup two log files before reboot (sonic-net#6923) - Fix missing definition (sonic-net#6930) - [Mellanox] Add minimal table definition for SN2201 (sonic-net#6943) - Update qos test param for dualtor topology (sonic-net#6948) - fix setup for single asic lc (sonic-net#6951) - Fix QoS sai test for running with python3 (sonic-net#6961) - Don't fail if logrotate cron job file isn't present (sonic-net#6964) - Disable post sanity check for vxlan test (sonic-net#6980) - Merge branch 'azure-202205' into dev/yaqiangzhu/202205_merge
5 tasks
wangxin
pushed a commit
that referenced
this pull request
Jan 13, 2024
The goal of test_stress_acl.py is to add/delete acl entries frequently. Refer: #6903 test_stress_acl.py is adding new acl table and it will fail if the TCAM space is not available. As done in other acl tests, removing DATAACL to free TCAM resources to run test_stress_acl.py.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
Dynamic acl will add/delete acl entries frequently, need a testcase to test stress acl.
Issue link: #6801
How did you do it?
Send a template sh file to dut, run 1000 times add/delete acl, and check dut status
How did you verify/test it?
Run testcase
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation