Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance get_arp_table and get_mac_address_table with filtering arguments #48

Closed
mirceaulinic opened this issue Mar 17, 2017 · 4 comments
Labels
breaking change Breaking change. Scheduled for next major release.

Comments

@mirceaulinic
Copy link
Member

See details in the parent issue napalm-automation/napalm-base#217

@mirceaulinic mirceaulinic added breaking change Breaking change. Scheduled for next major release. enhancement labels Mar 17, 2017
@brotherdust
Copy link

brotherdust commented Dec 15, 2017

I ran into this myself not too long ago. Although the API change was approved for NAPALM, there's a limitation in librouteros: it doesn't accept any filter arguments with the print command. I modified librouteros to make it work, but the change broke compatibility with NAPALM, so I ended up just using librouteros by itself. =/ If you're interested, here's my fork with the changes:
brotherdust/librouteros@1aa8189

@luqasz
Copy link
Collaborator

luqasz commented Dec 20, 2017

Currently librouteros does not have any "official" query support. I am planning to add this. For now I can implement a rather crude workaround.

@luqasz
Copy link
Collaborator

luqasz commented Dec 29, 2017

I am confused. napalm-base have some pull requests but napalm-ros uses napalm which does not have this extended method. @mirceaulinic can you help explaining ?

luqasz pushed a commit that referenced this issue Dec 29, 2017
@luqasz
Copy link
Collaborator

luqasz commented Apr 19, 2020

Closing since napalm-base does not have any filtering.

@luqasz luqasz closed this as completed Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Breaking change. Scheduled for next major release.
Projects
None yet
Development

No branches or pull requests

3 participants