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.

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

Fix issue of lldp_syncd crashes and cause docker lldp restarts #73

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Th1592-aster
Copy link

lldp-syncd throws exception from get_sys_capability_list The get_sys_capability_list() shall consider both the system name and system capabilities of remote device is null case.

Why I did it
During the usage process, we encountered the following issues.This has also led to the restart of the docker-lldp.

Nov 3 17:50:40.476595 sonic INFO lldp#supervisord: lldp-syncd capability_list = if_attributes['chassis'].values()[0]['capability']
Nov 3 17:50:40.476668 sonic INFO lldp#supervisord: lldp-syncd TypeError: string indices must be integers

lldp-syncd throws exception from get_sys_capability_list
The get_sys_capability_list() shall consider both the system name and system capabilities of remote device is null case.
@mssonicbld
Copy link

/azp run

Copy link

linux-foundation-easycla bot commented Dec 22, 2024

CLA Signed


The committers listed above are authorized under a signed CLA.

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants