matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[chassis] Do not configure LLDP on recirc ports (#7909)

Browse Source 
Why I did it
Recirc port is used to only forward traffic from one asic to another asic. So it's not required to configure LLDP on it.

How I did it
Add interface prefix helper for recirc port. Similar to skip configuring LLDP on inband port, add check in lldpmgrd to skip recirc port by checking interface prefix.
This commit is contained in:
Song Yuan 2022-04-25 13:14:17 -07:00 committed by GitHub
parent 0606add017
commit aa62e33339
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
dockers/docker-lldp/lldpmgrd
View File

@ -22,7 +22,7 @@ try:
from sonic_py_common import daemon_base from sonic_py_common import daemon_base
from swsscommon import swsscommon from swsscommon import swsscommon
from sonic_py_common.interface import inband_prefix from sonic_py_common.interface import inband_prefix, recirc_prefix
except ImportError as err: except ImportError as err:
raise ImportError("%s - required module not found" % str(err)) raise ImportError("%s - required module not found" % str(err))
@ -133,9 +133,9 @@ class LldpManager(daemon_base.DaemonBase):
""" """
port_desc = None port_desc = None
# Skip inband interface prefixes. These are recycle ports exposed in PORT_TABLE for # Skip recirc and inband interface prefixes. These are recycle ports exposed in PORT_TABLE for
# asic-to-asic communication in VOQ based chassis system. We do not configure LLDP on these. # asic-to-asic communication in VOQ based chassis system. We do not configure LLDP on these.
if port_name.startswith(inband_prefix()): if port_name.startswith(inband_prefix()) or port_name.startswith(recirc_prefix()):
return return
# Get the port alias. If None or empty string, use port name instead # Get the port alias. If None or empty string, use port name instead