sonic-mgmt
sonic-mgmt copied to clipboard
Test gap for lldp APPL_DB data consistency
Description of PR
Summary: Fixes # (issue) Address https://github.com/sonic-net/sonic-mgmt/issues/13937
ADO:28941599
Type of change
- [ ] Bug fix
- [ ] Testbed and Framework(new/improvement)
- [ ] Test case(new/improvement)
Back port request
- [ ] 202012
- [ ] 202205
- [ ] 202305
- [x] 202311
- [x] 202405
Approach
What is the motivation for this PR?
To address https://github.com/sonic-net/sonic-mgmt/issues/13937 Test plan: https://github.com/sonic-net/sonic-mgmt/pull/14080
How did you do it?
Add a new test script to test lldp APPL_DB data consistency
How did you verify/test it?
run lldp/test_lldp_syncd.py
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation
Hi, @ZhaohuiS , can you help me confirm if this new added test script can run in PR test?
Hi, @ZhaohuiS , can you help me confirm if this new added test script can run in PR test?
@yutongzhang-microsoft It should be able to run on kvm testbed.
@ZhaohuiS do we need it in 202405 branch?
@ZhaohuiS PR conflicts with 202405 branch
@ZhaohuiS PR conflicts with 202311 branch
@ZhaohuiS The test case is failing with error .
File "/var/src/sonic-mgmt_vms2-4-t0-2700_646f1405735219c3e54440f2/tests/lldp/test_lldp_syncd.py", line 36, in get_lldp_entry_content
return db.hget_all("LLDP_ENTRY_TABLE:{}".format(interface))
AttributeError: 'SonicDbCli' object has no attribute 'hget_all'
Is there any dependent PR not merged?
Issue fixed in PR https://github.com/sonic-net/sonic-mgmt/pull/14846