Fixes issue #126 and #129; LLDP neighbor detail bug

napalm-automation / napalm-ios

Apache License 2.0

31 stars 40 forks source link

Fixes issue #126 and #129; LLDP neighbor detail bug #133

Closed ktbyers closed 7 years ago

coveralls commented 7 years ago

Coverage increased (+0.07%) to 66.536% when pulling dd70b4255fcebf937dcf61dbf0b8c671d96264e1 on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.3%) to 66.797% when pulling 9c920e2e65af16d5d1abc80d74d84ef2f740f6da on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

ktbyers commented 7 years ago

@kaage Can you test that this PR fixes the two issues that you reported.

I still think I need the show lldp neighbors output for both issues #126 and #129 (particularly #126) as I am making assumptions.

ktbyers commented 7 years ago

This should be tested by @kaage before integrated in.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling 20254229573a740fccd7fb16b3428ead19c875f2 on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling 20254229573a740fccd7fb16b3428ead19c875f2 on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling 20254229573a740fccd7fb16b3428ead19c875f2 on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling bd241a2870c882d100b12fa2c1433eb628d0d4ed on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling bd241a2870c882d100b12fa2c1433eb628d0d4ed on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

coveralls commented 7 years ago

Coverage increased (+0.5%) to 66.926% when pulling bd241a2870c882d100b12fa2c1433eb628d0d4ed on ktbyers:devel into e4dbf49187894849b7f6ee7c11b3888714fd013a on napalm-automation:develop.

ktbyers commented 7 years ago

@kaage just validated this change was working.