napalm-automation / napalm-ios

Apache License 2.0
31 stars 40 forks source link

Fixing get_interfaces for pseudowire #206

Closed ktbyers closed 7 years ago

ktbyers commented 7 years ago

Replaces PR #180 and relevant to issue #179

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.2%) to 70.069% when pulling dcbf4e5d4ed0e3b01ef14c0f66b37482cf984a78 on ktbyers:fix_pseudowire into eaaaec6d800ec16717413e255643fc3493df0252 on napalm-automation:develop.

ktbyers commented 7 years ago

Note, I currently don't have information on what the pseudowire looks like when shutdown so I made some assumptions here. I think they are reasonable assumptions, but I have asked on the original issue if the submitter has data on the admin down format.

ktbyers commented 7 years ago

Okay, pseudowire is always up so what I have should work.