From 296d7672cb6efe1fef7944751f395e387b8ad4ad Mon Sep 17 00:00:00 2001 From: Aharon Malkin Date: Wed, 17 Dec 2025 17:01:15 +0200 Subject: [PATCH 1/2] Added skips by bugs to bgp, fib, snmp, ecn tests Signed-off-by: Aharon Malkin --- .../conditional_mark/tests_mark_conditions.yaml | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml index 6cc4b1183e7..dc348e1fe14 100644 --- a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml +++ b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml @@ -308,6 +308,12 @@ bgp/test_bgp_bbr_default_state.py::test_bbr_disabled_constants_yml_default: conditions: - "'-v6-' in topo_name" +bgp/test_bgp_command.py: + skip: + reason: 'BGP command test is skipped due to github issue https://github.com/sonic-net/sonic-mgmt/issues/17927 on t1-isolated-d28u1 topology on Mellanox platform' + conditions: + - "https://github.com/sonic-net/sonic-mgmt/issues/17927 and topo_name in ['t1-isolated-d28u1'] and asic_type in ['mellanox']" + bgp/test_bgp_dual_asn.py::test_bgp_dual_asn_v4: skip: reason: "Skip for IPv6-only topologies" @@ -2266,9 +2272,10 @@ fdb/test_fdb_mac_learning.py::TestFdbMacLearning::testFdbMacLearning: ####################################### fib/test_fib.py: skip: - reason: 'Skip on t1-isolated-d32/128 topos' + reason: 'Skip on t1-isolated-d32/128 topos or due to github issue https://github.com/sonic-net/sonic-mgmt/issues/17930' conditions: - "topo_name in ['t1-isolated-d128', 't1-isolated-d32']" + - "https://github.com/sonic-net/sonic-mgmt/issues/17930 and topo_name in ['t1-isolated-d28u1']" xfail: reason: "Test case has issue on the t0-isolated-d256u256s2 topo." conditions: @@ -2509,6 +2516,7 @@ generic_config_updater/test_ecn_config_update.py::test_ecn_config_updates: - "hwsku in ['Cisco-8800-LC-48H-C48']" - "topo_type in ['m0', 'mx', 'm1']" - "release in ['202211']" + - "https://github.com/sonic-net/sonic-buildimage/issues/22295 and platform in ['x86_64-nvidia_sn5600-r0', 'x86_64-nvidia_sn5640-r0']" generic_config_updater/test_eth_interface.py::test_replace_fec: skip: @@ -4604,10 +4612,11 @@ snmp/test_snmp_queue.py: snmp/test_snmp_queue_counters.py: skip: conditions_logical_operator: OR - reason: "Have an known issue on kvm testbed / Unsupported in M* topos" + reason: "Have an known issue on kvm testbed / Unsupported in M* topos or issue #17929 on Mellanox platform" conditions: - "asic_type in ['vs'] and https://github.com/sonic-net/sonic-mgmt/issues/14007" - "topo_type in ['m0', 'mx', 'm1']" + - "https://github.com/sonic-net/sonic-mgmt/issues/17929 and asic_type in ['mellanox']" xfail: reason: "Test case has issue on the t0-isolated-d256u256s2 topo." conditions: From 3f6cf0b990e2ff4ae14d86d6d298fd086bdec7fc Mon Sep 17 00:00:00 2001 From: AharonMalkin <94370721+AharonMalkin@users.noreply.github.com> Date: Tue, 27 Jan 2026 09:39:09 +0200 Subject: [PATCH 2/2] Update tests_mark_conditions.yaml Signed-off-by: Aharon Malkin --- .../plugins/conditional_mark/tests_mark_conditions.yaml | 6 ------ 1 file changed, 6 deletions(-) diff --git a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml index ed8017fa0fe..0a23b88c704 100644 --- a/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml +++ b/tests/common/plugins/conditional_mark/tests_mark_conditions.yaml @@ -333,12 +333,6 @@ bgp/test_bgp_bounce.py: conditions: - "https://github.com/sonic-net/sonic-mgmt/issues/20753 and '-v6-' in topo_name" -bgp/test_bgp_command.py: - skip: - reason: 'BGP command test is skipped due to github issue https://github.com/sonic-net/sonic-mgmt/issues/17927 on t1-isolated-d28u1 topology on Mellanox platform' - conditions: - - "https://github.com/sonic-net/sonic-mgmt/issues/17927 and topo_name in ['t1-isolated-d28u1'] and asic_type in ['mellanox']" - bgp/test_bgp_dual_asn.py::test_bgp_dual_asn_v4: skip: reason: "Skip for IPv6-only topologies"