search

napalm-automation / napalm-base

Apache License 2.0
32 stars 48 forks source link

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

Open WilliamMarti opened 7 years ago

WilliamMarti commented 7 years ago

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 commented 7 years ago

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

mirceaulinic commented 7 years ago

Hi @WilliamMarti - are you still interested in this?

WilliamMarti commented 7 years ago

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 commented 7 years ago

@WilliamMarti no problem at all - just checking

dbarrosop commented 7 years ago

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