Skip to content

Commit

Permalink
[chassis] Do not configure LLDP on recirc ports (#7909)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
ysmanman committed Apr 25, 2022
1 parent 0606add commit aa62e33
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions dockers/docker-lldp/lldpmgrd
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ try:

from sonic_py_common import daemon_base
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:
raise ImportError("%s - required module not found" % str(err))

Expand Down Expand Up @@ -133,9 +133,9 @@ class LldpManager(daemon_base.DaemonBase):
"""
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.
if port_name.startswith(inband_prefix()):
if port_name.startswith(inband_prefix()) or port_name.startswith(recirc_prefix()):
return

# Get the port alias. If None or empty string, use port name instead
Expand Down

0 comments on commit aa62e33

Please sign in to comment.