sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
200 stars 731 forks source link

Fixed swss feature name for test_lldp_neighbor_post_orchagent_reboot #15715

Open anamehra opened 2 days ago

anamehra commented 2 days ago

Signed-off-by: Anand Mehra anamehra@cisco.com

Description of PR

Summary: Fixes # https://github.com/sonic-net/sonic-mgmt/issues/14689

Type of change

Back port request

Approach

What is the motivation for this PR?

The test test_lldp_neighbor_post_orchagent_reboot fails on multi-asic system. The test tries to disable autorestart feature for swss by using the namespace container name, e.g., swss0, swss1, etc

For config feature autorestart disable, it needs to use 'swss' as global feature name

How did you do it?

Changed code to use 'swss' as feature name without using namespace id

How did you verify/test it?

run sonic-mgmt test_lldp.py

------------------------------------------------------------------------------------------------------------------- live log sessionfinish --------------------------------------------------------------------------------------------------------------------
15:30:20 __init__.pytest_terminal_summary         L0067 INFO   | Can not get Allure report URL. Please check logs
=================================================================================================================== short test summary info ===================================================================================================================
PASSED lldp/test_lldp.py::test_lldp_neighbor_post_orchagent_reboot[sfd-t2-lc0-0]
PASSED lldp/test_lldp.py::test_lldp_neighbor_post_orchagent_reboot[sfd-t2-lc0-1]
PASSED lldp/test_lldp.py::test_lldp_neighbor_post_orchagent_reboot[sfd-t2-lc0-2]
========================================================================================================== 3 passed, 1 warning in 472.45s (0:07:52) ===========================================================================================================```

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

anamehra commented 2 days ago

Hi @arlakshm , @SuvarnaMeenakshi , please review. Thanks