-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Support for RouterOS v7 #133
Comments
Below an overview of all commands:
|
Could you paste output of bgp items (anomized of course) ? I don't have any working BGP peers to test. |
Sure, I have identified that the first part is already where the main issue occurs. Currently the code is calling for menu's like: For example you'll need the
I think the main question is: do we want to support both RouterOS v6 and RouterOS v7 in the same version of the driver? I can imagine creating a 2.X release that supports RouterOS 7 but lacks support for RouterOS 6 (which would require 1.X releases). I am open to contributing to this issue myself, I am not an experienced developer. But I believe in contributing to the community and this might be a useful way to develop my python-skills. |
Driver can support both versions. It's just a matter of executing different functions depending on version. |
Description of Issue/Question
The BGP syntax has changed between RouterOS v6 and RouterOS v7. The bgp-functionality in this driver is not working on RouterOS v7.
Setup
napalm-ros version
(Paste verbatim output from
pip freeze | grep napalm-ros
between quotes below)ROS version
(Paste verbatim output from
/system package print
between quotes below)librouteros version
(Paste verbatim output from
pip freeze | grep librouteros
between quotes below)python version
(paste output of python --version)
Steps to Reproduce the Issue
Error Traceback
(Paste the complete traceback of the exception between quotes below)
The text was updated successfully, but these errors were encountered: