Resolved 'failed to detect EUI verion: '

napalm-automation-community / napalm-ftos

NAPALM Driver for Dell/Force10 FTOS

Apache License 2.0

9 stars 7 forks source link

Resolved 'failed to detect EUI verion: ' #6

Closed Michael-Swartz closed 1 year ago

Michael-Swartz commented 4 years ago

I resolved an issue of receiving a netaddr.core.AddrFormatError while using get_lldp_neighbors and get_lldp_neighbors_details with Netbox. It seems as though the mac addresses that were returned on line 323 would would throw a netaddr.core.AddrFormatError exception. So now instead of returning an error, it will return lldp_entry[k] as ' ' if it cannot parse the mac address.

Michael-Swartz commented 4 years ago

Just realized this caused another issue with the Netbox integration where the interfaces now always show up as blank. Going to investigate more.

skoef commented 4 years ago

I'd like to see a test case dealing with this particular behaviour.

Also, isn't the "model" expecting either a MAC address or nothing? Now it could be potentially anything.