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

Test case for connected routes #162

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

Test case for connected routes #162

bewing wants to merge 1 commit into from

Conversation

@bewing
Copy link
Member

@bewing bewing commented Dec 10, 2016 •
edited by mirceaulinic
Loading

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

Depends on:
@mirceaulinic mirceaulinic added this to the BLOCKED milestone Dec 12, 2016
@mirceaulinic
Copy link
Member

mirceaulinic commented Dec 12, 2016

Putting this on hold till we'll have the implementation available on all drivers.
@mirceaulinic mirceaulinic mentioned this pull request Dec 12, 2016
Open
@dbarrosop
Copy link
Member

dbarrosop commented Dec 13, 2016

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 •
edited
Loading

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

dbarrosop commented Apr 6, 2017

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

dbarrosop commented Apr 16, 2017

Please, see: #238
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

blocked enhancement testing_framework

4 participants

@bewing @mirceaulinic @dbarrosop @manuel-domke