Skip to content

Commit

Permalink
Skip test_cacl_application to unblock sub module update of sonic-host…
Browse files Browse the repository at this point in the history
…-services (#6585)

What is the motivation for this PR?
Recent change in caclmgrd caused the expected ACL list to change.
test_cacl_application fails because the expected ACLs are not present on DUT.
This is occurring with latest sonic-host-services changes, and is blocking sub module update.
Test case has to be updated, but updated test case will fail without the latest changes from sonic-host-services. This PR is to ensure that sub module is updated so that the test case can pass.
issue: sonic-net/sonic-buildimage#12464
submodule update PR that is blocked: sonic-net/sonic-buildimage#12195

How did you do it?
skip test_cacl_application.
  • Loading branch information
SuvarnaMeenakshi authored Oct 21, 2022
1 parent e5921fa commit b9e8b16
Showing 1 changed file with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,11 @@ bgp/test_bgpmon.py:
#######################################
##### cacl #####
#######################################
cacl/test_cacl_application.py:
skip:
reason: "Caclmgrd change to allow mgmt traffic by default requires test case update"
conditions: https://github.com/sonic-net/sonic-buildimage/issues/12464

cacl/test_cacl_application.py::test_cacl_application_dualtor:
skip:
reason: "test_cacl_application_dualtor is only supported on dualtor topology"
Expand Down

0 comments on commit b9e8b16

Please sign in to comment.