From c72da9bc96bf339f2d4d235aea2e955834f12857 Mon Sep 17 00:00:00 2001 From: Harish Kalyanaraman <92752170+harish-kalyanaraman@users.noreply.github.com> Date: Wed, 25 Dec 2024 19:58:39 -0500 Subject: [PATCH] Skipping enable_queue_counterpoll_type fixture for Marvell platforms (#16019) What is the motivation for this PR? Fixture enable_queue_counterpoll_type in snmp is enabling queue counterpolls. But for Nokia 7215 armhf platforms, as queue statistics are not supported, there is an error while executing this command : "SAI_OBJECT_TYPE_QUEUE is not Supported". So, added a skip condition inside the fixture for Nokia 7215 armhf platforms How did you do it? Added condition inside fixture to skip for Nokia 7215 armhf platform How did you verify/test it? Run snmp test suite and verify fixture is not executed on Nokia-7215 armhf platforms --- tests/snmp/conftest.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/snmp/conftest.py b/tests/snmp/conftest.py index 88db6acec52..3e7649da59b 100644 --- a/tests/snmp/conftest.py +++ b/tests/snmp/conftest.py @@ -95,7 +95,8 @@ def check_redis_output(duthost, key): @pytest.fixture(scope="module", autouse=True) def enable_queue_counterpoll_type(duthosts): for duthost in duthosts: - duthost.command('counterpoll queue enable') + if duthost.facts['platform'] not in ['armhf-nokia_ixs7215_52x-r0']: + duthost.command('counterpoll queue enable') def pytest_addoption(parser):