Skip to content
This repository has been archived by the owner on Sep 17, 2019. It is now read-only.

Test case for connected routes #162

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

bewing
Copy link
Member

@bewing bewing commented Dec 10, 2016

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

Depends on:

@mirceaulinic
Copy link
Member

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

@dbarrosop
Copy link
Member

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.

@mirceaulinic mirceaulinic modified the milestones: 0.22.0, BLOCKED Jan 4, 2017
@dbarrosop dbarrosop self-assigned this Jan 30, 2017
@dbarrosop dbarrosop removed this from the 0.22.0 milestone Jan 30, 2017
@mirceaulinic mirceaulinic added this to the BLOCKED milestone Feb 7, 2017
@manuel-domke
Copy link

manuel-domke commented Apr 4, 2017

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?

@dbarrosop
Copy link
Member

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

#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
Copy link
Member

Please, see: #238

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants