Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Test Gap][lldp] Verify the lldp entries in APPL_DB after restart syncd/swss #15889

Closed
ZhaohuiS opened this issue Dec 5, 2024 · 1 comment
Closed

[Test Gap][lldp] Verify the lldp entries in APPL_DB after restart syncd/swss #15889

ZhaohuiS opened this issue Dec 5, 2024 · 1 comment
Assignees
@ZhaohuiS
Labels
P1 Test gap

Comments

@ZhaohuiS
Copy link
Contributor

ZhaohuiS commented Dec 5, 2024

Test Gap Description

We have an issue that there is only lldp_rem_time_mark for eth0

It still has the following syslog:
2024 Dec 2 18:22:18.291732 bjw2-can-7260-3 WARNING snmp#snmp-subagent [sonic_ax_impl] WARNING: Exception when updating lldpRemTable: 'lldp_rem_index'

It always happened for eth0, the LLDP_ENTRY_TABLE in APPL_DB for eth0 is like this:
{'lldp_rem_time_mark': '3886'}

It's fixed by
sonic-net/sonic-dbsyncd#71

Test Plan

Issue reproduce steps:

docker exec syncd supervisorctl stop syncd
docker exec swss supervisorctl stop orchagent

docker exec syncd supervisorctl start syncd
docker exec swss supervisorctl start orchagent

Check the LLDP entries for all interfaces in APPL_DB are full and completed.

Test environment

No response

Attach files (if any)

No response
@ZhaohuiS ZhaohuiS self-assigned this Dec 5, 2024
@ZhaohuiS ZhaohuiS mentioned this issue Dec 5, 2024
Merged
8 tasks
StormLiangMS pushed a commit that referenced this issue Dec 18, 2024
@ZhaohuiS
[test gap] Add a new test case to verify lldp entry after restart syn…
9450127 
…cd and orchagent (#15911)

What is the motivation for this PR?
Add a new test case to address #15889

How did you do it?
Restart syncd and orchagent, then check lldp entries in APPL_DB

How did you verify/test it?
run test_lldp_syncd.py
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Dec 18, 2024
@ZhaohuiS @mssonicbld
[test gap] Add a new test case to verify lldp entry after restart syn…
a9df857 
…cd and orchagent (sonic-net#15911)

What is the motivation for this PR?
Add a new test case to address sonic-net#15889

How did you do it?
Restart syncd and orchagent, then check lldp entries in APPL_DB

How did you verify/test it?
run test_lldp_syncd.py
@mssonicbld mssonicbld mentioned this issue Dec 18, 2024
Merged
8 tasks
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this issue Dec 19, 2024
@ZhaohuiS @mssonicbld
[test gap] Add a new test case to verify lldp entry after restart syn…
dbb5db9 
…cd and orchagent (sonic-net#15911)

What is the motivation for this PR?
Add a new test case to address sonic-net#15889

How did you do it?
Restart syncd and orchagent, then check lldp entries in APPL_DB

How did you verify/test it?
run test_lldp_syncd.py
mssonicbld pushed a commit that referenced this issue Dec 19, 2024
@ZhaohuiS @mssonicbld
[test gap] Add a new test case to verify lldp entry after restart syn…
ca7816a 
…cd and orchagent (#15911)

What is the motivation for this PR?
Add a new test case to address #15889

How did you do it?
Restart syncd and orchagent, then check lldp entries in APPL_DB

How did you verify/test it?
run test_lldp_syncd.py
@ZhaohuiS
Copy link
Contributor Author

ZhaohuiS commented Feb 7, 2025

The test gap was addressed in #15911

@ZhaohuiS ZhaohuiS closed this as completed Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ZhaohuiS ZhaohuiS

Labels
P1 Test gap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ZhaohuiS