From 0f04d190ad30d76749cc688fd6e465f844a112e1 Mon Sep 17 00:00:00 2001 From: Aaron Bernardino Date: Sun, 5 Jul 2026 17:16:54 +0000 Subject: [PATCH] Enable iface_loopback_action test on t1-lag-vpp Add iface_loopback_action/test_iface_loopback_action.py to the t1-lag-vpp PR test set and allow asic_type==vpp in conditional_mark, gated on the prerequisite saivpp/VPP-plugin changes (sonic-net/sonic-buildimage#25788). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Aaron Bernardino --- .azure-pipelines/pr_test_scripts.yaml | 1 + .../conditional_mark/tests_mark_conditions.yaml | 10 ++++++++-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.azure-pipelines/pr_test_scripts.yaml b/.azure-pipelines/pr_test_scripts.yaml index 9b4de20d2d7..f02868c269b 100644 --- a/.azure-pipelines/pr_test_scripts.yaml +++ b/.azure-pipelines/pr_test_scripts.yaml @@ -730,6 +730,7 @@ t1-lag-vpp: - vxlan/test_vnet_bgp_route_precedence.py - vxlan/test_vxlan_multi_tunnel.py - vxlan/test_vxlan_route_advertisement.py + - iface_loopback_action/test_iface_loopback_action.py multi-asic-t1-lag: diff --git a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml index d03be4f75ac..720b6d3b36d 100644 --- a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml +++ b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml @@ -3437,12 +3437,18 @@ iface_loopback_action/: - "'bmc' in topo_type" iface_loopback_action/test_iface_loopback_action.py: + # Enabled on the VPP virtual platform (sonic-net/sonic-buildimage#25788). Requires: + # - VPP: iface_loopback plugin drops+counts hairpin (same-RIF) routed packets + # (sonic-platform-vpp iface_loopback plugin). + # - libsaivs: SAI_ROUTER_INTERFACE_ATTR_LOOPBACK_PACKET_ACTION handler + RIF stats + # (sonic-sairedis vslib/vpp). + # - testbed: qemu virtio-net ctrl_vlan=off so VLAN-tagged sub-port traffic reaches VPP. skip: - reason: "Test only supported on Mellanox SKUs, didn't supported on dualtor topology." + reason: "Test supported on Mellanox SKUs and VPP virtual platform, not supported on dualtor topology." conditions_logical_operator: or conditions: - "'dualtor' in topo_name" - - "asic_type not in ['mellanox']" + - "asic_type not in ['mellanox', 'vpp']" ####################################### ##### iface_namingmode #####