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

Adding functions for get_eigrp_config() and get_eigrp_neighbors() #183

Open
WilliamMarti opened this issue Jan 4, 2017 · 5 comments
Open
Milestone

Comments

@WilliamMarti
Copy link

So today there are the following methods in napalm-base

get_bgp_neighbors()
get_bgp_config()

In the network I support we are heavy EIGRP users and could use similar functionality. I think this comes back to a larger discussion from slack and here, talking about vendor specific protocols. So my proposal is to add:

get_eigrp_neighbors()
get_eigrp_config()

Realistically these would only ever be implemented in the Cisco napalm flavors.

Looking at the get_route_to function, it should already support EIGRP routes.

@dbarrosop
Copy link
Member

dbarrosop commented Jan 5, 2017

Yeah, I think those methods are fine. Is this something you are looking to implement yourself?

@mirceaulinic mirceaulinic added this to the APPROVED milestone Jan 5, 2017
@mirceaulinic
Copy link
Member

Hi @WilliamMarti - are you still interested in this?

@WilliamMarti
Copy link
Author

Apologies, totally fell off my radar.

Yes I would like to implement this. I will get working on the PR for napalm-base now.

@mirceaulinic
Copy link
Member

@WilliamMarti no problem at all - just checking

@dbarrosop
Copy link
Member

This looks good in principle, I would like to see an initial implementation though before approving and merging.

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

No branches or pull requests

3 participants