-
Notifications
You must be signed in to change notification settings - Fork 549
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
ios get_lldp_neighbor fails when parsing blank column rows #1803
Comments
I am not sure it is using You are failing here: Line 990 in 7ecb9b9
Which is referencing Line 989 in 7ecb9b9
Which is defined in template, here:
Can you show me your Compare that to what is showing in mocked responses as shown here: napalm/test/ios/mocked_data/test_get_lldp_neighbors_detail/alternate/show_lldp_neighbors_detail.txt Line 3 in d782cc7
|
Thank you for your response. An uncensored version of the following output can be provided if necessary.
|
I realized I did not provide the full output from
Then the traceback I posted above follows the above text. |
Hi, I found some time to look in to this. Lines 977 to 994 in 7ecb9b9
In the above snippet, we pass the remote_port through the mac() helper function. For the device I'm encountering issues with this function on, the remote_port is seldom a MAC address. Hence why this fails. As a workaround for myself, commenting this line allowed the Line 990 in 7ecb9b9
I suppose this function could be improved by checking to ensure the I also noticed that the I haven't seen the |
Description of Issue/Question
Note: Please check https://guides.github.com/features/mastering-markdown/
to see how to properly format your request.
Did you follow the steps from https://github.com/napalm-automation/napalm#faq
(Place an
x
between the square brackets where applicable)Setup
napalm version
(Paste verbatim output from
pip freeze | grep napalm
between quotes below)Network operating system version
(Paste verbatim output from
show version
- or equivalent - between quotes below)Steps to Reproduce the Issue
Run the following command when one of the rows within the Compatibility column is blank.
Error Traceback
(Paste the complete traceback of the exception between quotes below)
Full CLI output of "show lldp neighbors"
Note:
HOSTNAME indicates a hostname I've removed.
0000.0000.0000 indicates a mac address originally formatted as three groups of four, whose specifics I've removed.
I've included as much of the original formatting as is applicable to the above error message.
The text was updated successfully, but these errors were encountered: