search

sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
173 stars 689 forks source link

[test_stress_acl.py] TC failed on T1 topology #6963

Open OleksandrKozodoi opened 1 year ago

OleksandrKozodoi commented 1 year ago

Description test_stress_acl.py TC failed on the T1 topology due to absent LAG ports on the DUT

Steps to reproduce the issue:

  1. Deploy SONiC
  2. Deploy T1 topology.
  3. Run TC: acl/test_stress_acl.py

Describe the results you received:

    def test_acl_add_del_stress(duthosts, rand_one_dut_hostname, get_function_conpleteness_level):
        duthost = duthosts[rand_one_dut_hostname]
        generate_acl_rule(duthost, "ipv4")
        duthost.shell("config acl add table -p PortChannel101,PortChannel102,PortChannel103,PortChannel104 \
>                     IP_STRESS_ACL L3")

E                   "Error: Failed to parse ACL table config: exception=Cannot bind ACL to specified port PortChannel103"

Describe the results you expected: TC is skipped on the T1 topology

Additional information you deem important:

SONiC Software Version: SONiC.master.178866-dirty-20221123.180536
Distribution: Debian 11.5
Kernel: 5.10.0-18-2-amd64
Build commit: faef4c69e
Build date: Wed Nov 23 18:10:56 UTC 2022
ASIC: barefoot
OleksandrKozodoi commented 1 year ago

@xwjiang-ms Could you please take a look?

xwjiang-ms commented 1 year ago

@xwjiang-ms Could you please take a look?

Sure, I'm fixing it