From e7748440324b99c7b95c660a34104430b8cd77f1 Mon Sep 17 00:00:00 2001 From: sresri2 <125845681+sresri2@users.noreply.github.com> Date: Fri, 7 Aug 2026 16:10:27 -0700 Subject: [PATCH] Initial Change + Skip Removal Signed-off-by: sresri2 <125845681+sresri2@users.noreply.github.com> --- tests/bgp/test_bgpmon_v6.py | 9 +++------ .../conditional_mark/tests_mark_conditions_vs_t2.yaml | 5 ----- 2 files changed, 3 insertions(+), 11 deletions(-) diff --git a/tests/bgp/test_bgpmon_v6.py b/tests/bgp/test_bgpmon_v6.py index 7fbf2ba064c..bf6d04cd0d9 100644 --- a/tests/bgp/test_bgpmon_v6.py +++ b/tests/bgp/test_bgpmon_v6.py @@ -9,6 +9,7 @@ from tests.common.fixtures.ptfhost_utils import remove_ip_addresses # noqa:F401 from tests.common.helpers.generators import generate_ip_through_default_route, generate_ip_through_default_v6_route from tests.common.helpers.assertions import pytest_assert +from tests.common.helpers.bgp import get_asic_config_facts from tests.common.utilities import wait_until from tests.common.utilities import wait_tcp_connection from bgp_helpers import BGPMON_TEMPLATE_FILE, BGPMON_CONFIG_FILE, BGP_MONITOR_NAME, BGP_MONITOR_PORT @@ -90,12 +91,8 @@ def common_v6_setup_teardown(duthosts, tbinfo, enum_rand_one_per_hwsku_frontend_ peer_addr = str(IPNetwork(peer_addr).ip) peer_ports, local_ports = get_all_uplink_ptf_recv_ports(duthosts, tbinfo) - # Get loopback4096 address - if enum_rand_one_frontend_asic_index: - cfg_facts = duthost.config_facts( - source='persistent', asic_index='all')[enum_rand_one_frontend_asic_index]['ansible_facts'] - else: - cfg_facts = duthost.config_facts(source='persistent', asic_index='all')[0]['ansible_facts'] + # Get loopback4096 address for the selected frontend ASIC + cfg_facts = get_asic_config_facts(duthost, enum_rand_one_frontend_asic_index) if 'Loopback4096' in cfg_facts['LOOPBACK_INTERFACE']: lbs4096 = list(cfg_facts['LOOPBACK_INTERFACE']['Loopback4096'].keys()) diff --git a/tests/common/plugins/conditional_mark/tests_mark_conditions_vs_t2.yaml b/tests/common/plugins/conditional_mark/tests_mark_conditions_vs_t2.yaml index 1cb5e2d3460..fd19759fd1b 100644 --- a/tests/common/plugins/conditional_mark/tests_mark_conditions_vs_t2.yaml +++ b/tests/common/plugins/conditional_mark/tests_mark_conditions_vs_t2.yaml @@ -78,11 +78,6 @@ bgp/test_bgpmon.py: conditions: - asic_type in ['vs'] and 't2' in topo_name reason: This test case either cannot pass or should be skipped on virtual chassis -bgp/test_bgpmon_v6.py: - skip: - conditions: - - asic_type in ['vs'] and 't2' in topo_name - reason: This test case either cannot pass or should be skipped on virtual chassis bgp/test_ipv6_nlri_over_ipv4.py: skip: conditions: