-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[LLDP] lldp portidsubtype was NOT set to "locally assigned" but "mac address" #1457
Comments
I am unable to reproduce this problem on two different devices, both also configured for a T0 test topology, and both running a build from last night. Below is my output from two peer Arista VMs of one of the devices. Note that the portidsubtype is correctly set to "Locally assigned" and both Port ID and Port Description are also configured correctly. Could you please provide all log messages from lldpmgrd so that I can determine whether or not it is behaving correctly for you?
|
@jleveque I tried on 4 devices with t0 and t1 and t1-lag topo, all of them still have this issue with the image built on 5th Mar.
check on Arista VM
I filtered out the lldpmgr log as below
|
@jleveque btw, if I run the command "lldpcli configure ports Ethernet120 lldp portidsubtype local Ethernet120 description ARISTA04T1:Ethernet1" explicitly, from the Arista VM can see that the port id and port description changed as expected:
|
@keboliu: I was able to reproduce the issue. It appears to be caused by a race condition when the interfaces are initially created. We are still investigating the root cause, but if you wouldn't mind, could you please test the following workaround to confirm whether this also resolves the issue on your devices? Add a
|
@jleveque good to know it can be reproduced, I can test it, will update you when having the result. |
@jleveque the workaround doesn't work on my testbed, still have the same issue. |
@keboliu: I forgot to mention you would also need to add the line |
@jleveque I sent out the tetst log with Guohan's lldpcli patch. do you still need to test this workaround? |
@keboliu: If you have time, please do. It would help confirm the race condition we found. |
In the latest version
the lldp portidsubtype wasn't set to "locally assigned" but "mac address", it can be verified on the peer Arista VM with t0 topology.
This is different with the version build on 28th Feb, on this version it was set to "locally assigned"
when this happened, the lldpmgrd is running:
The text was updated successfully, but these errors were encountered: