napalm-base icon indicating copy to clipboard operation
napalm-base copied to clipboard

Published 20 hours ago • verified publisher icon napalm-automation

Test case for connected routes

Open bewing opened this issue 8 years ago • 5 comments

I don't know how napalm-automation/napalm-base#157 got so horribly screwed, so I just deleted everything and started over.

Depends on:

  • [x] eos - napalm-automation/napalm-eos/pull/101
  • [ ] junos
  • [ ] iosxr

bewing avatar Dec 10 '16 17:12 bewing

Putting this on hold till we'll have the implementation available on all drivers.

mirceaulinic avatar Dec 12 '16 03:12 mirceaulinic

Updated the description of the PR adding a list of devices we have to implement this for. We can create issues if we prefer and link them here if we prefer.

dbarrosop avatar Dec 13 '16 09:12 dbarrosop

Hi, I just added a IPv6 implementation to get_route_to in the iosxr driver.

Now I need to extend this testcase to IPv6 destinations for my changes to pass the CI. Any comments on that?

manuel-domke avatar Apr 04 '17 12:04 manuel-domke

This other PR, which is similar to this one, made me think:

https://github.com/napalm-automation/napalm-base/pull/233

Quoting myself from that other PR: "I was wondering if we should just remove the "protocol" from test_get_route_to so we can control with the test cases (default and the rest) the data we get and parse instead."

dbarrosop avatar Apr 06 '17 11:04 dbarrosop

Please, see: https://github.com/napalm-automation/napalm-base/issues/238

dbarrosop avatar Apr 16 '17 07:04 dbarrosop