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

[lldpmgrd] Fix lldpmgrd issue where it sets the lldp system name to None #11907

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[lldpmgrd] Fix lldpmgrd issue where it sets the lldp system name to None #11907

wants to merge 2 commits into from

Commits on Aug 31, 2022

  1. [lldpmgrd] Fix lldpmgrd issue where it sets the lldp system name to None 

    - Only check hostname field in key DEVICE_METADATA|localhost of configDB
- If hostname does not exist, don't set lldp system name

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
    @zhenggen-xu
    zhenggen-xu committed Aug 31, 2022
    Configuration menu
    Copy the full SHA
    a63cb16 View commit details
    Browse the repository at this point in the history

  2. [lldpmgrd] Address the review feedback 

    Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
    @zhenggen-xu
    zhenggen-xu committed Aug 31, 2022
    Configuration menu
    Copy the full SHA
    9f0933a View commit details
    Browse the repository at this point in the history