Skip to content

Commit 14768ea

Browse files
Remove MGMT_INTERFACE from ip2me block rule list (#7092)
What is the motivation for this PR? Provides fix for sonic-net/sonic-buildimage#12464 MGMT_INTERFACE was removed from default ip2me block rules in PR sonic-net/sonic-host-services#6 This PR is to modify the test case according to the change in caclmgrd How did you do it? Remove MGMT_INTERFACE when generating ip2me block rule list. How did you verify/test it? Ran test on multi-asic vs DUT, without skipping this test: .. cacl/test_cacl_application.py::test_cacl_application_nondualtor[vlab-08-1] PASSED [ 70%] cacl/test_cacl_application.py::test_cacl_application_dualtor[active_tor-vlab-08-1] SKIPPED [ 80%] cacl/test_cacl_application.py::test_cacl_application_dualtor[standby_tor-vlab-08-1] SKIPPED [ 90%] cacl/test_cacl_application.py::test_multiasic_cacl_application[vlab-08-1] PASSED SKIPPED [4] cacl/test_cacl_application.py: test_cacl_application_dualtor is only supported on dualtor topology SKIPPED [1] cacl/test_cacl_application.py: caclmgrd may crash after loading scale ipv4 cacl rules. SKIPPED [1] cacl/test_cacl_application.py: caclmgrd may crash after loading scale ipv6 cacl rules. ============================================================================================================ 4 passed, 6 skipped in 769.78 seconds =============================================================================================================
1 parent d9ccae6 commit 14768ea

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

tests/cacl/test_cacl_application.py

-1
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,6 @@ def get_cacl_tables_and_rules(duthost):
314314
def generate_and_append_block_ip2me_traffic_rules(duthost, iptables_rules, ip6tables_rules, asic_index):
315315
INTERFACE_TABLE_NAME_LIST = [
316316
"LOOPBACK_INTERFACE",
317-
"MGMT_INTERFACE",
318317
"VLAN_INTERFACE",
319318
"PORTCHANNEL_INTERFACE",
320319
"INTERFACE"

0 commit comments

Comments
 (0)