support route-reflector-client in bgp_neighbors_detail

napalm-automation / napalm-junos

Apache License 2.0

22 stars 42 forks source link

support route-reflector-client in bgp_neighbors_detail #213

Closed ckishimo closed 6 years ago

mirceaulinic commented 6 years ago

Hi @ckishimo - thanks for this idea, but unfortunately this kind of improvements are very hard to add at the time being, as this will break the entire API. But please file an API change request under napalm-base and we'll get this sorted after we reunify napalm into a single library. I am going to close this for now, should you have further questions, feel free to reopen!

mirceaulinic commented 6 years ago

Shit - that field actually already exists. Shame on me I forgot!

mirceaulinic commented 6 years ago

Yeah, not my best day. That field exists but in get_bgp_config, not in get_bgp_neighbors_detail. Sorry, got to close this... for the second time...