-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Adding Support For EVPN in BGP Module #55503
Conversation
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha <anasb@mellanox.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @anasbadaha
looks great.
However, I have two small comments:
1- All regular expressions started with: ^\s.router
should be changed to: ^\srouter
2- in function _generate_bgp_cmds
when checking: if curr_as_num != as_number:
you should also check if the vrf has been changed, then you have to remove the bgp and create a new one.
Signed-off-by: Anas Badaha <anasb@mellanox.com>
Hi @samerd , I have fixed your comments related to function _generate_bgp_cmds and added a unit test for it as well. |
Signed-off-by: Anas Badaha <anasb@mellanox.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent Job @anasbadaha
Thanks
Hi @justjais, Can you please merge this PR if it is OK for you? Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
rebuild_merge |
* Adding Support For EVPN in BGP Module Signed-off-by: Anas Badaha <anasb@mellanox.com> * Fix Pep8 Failures in onyx_bgp.py Signed-off-by: Anas Badaha <anasb@mellanox.com> * Fix Pep8 Failures in onyx_bgp.py Phase 2 Signed-off-by: Anas Badaha <anasb@mellanox.com> * Fix Samer's Comments Signed-off-by: Anas Badaha <anasb@mellanox.com> * Remove file Variable Signed-off-by: Anas Badaha <anasb@mellanox.com>
Signed-off-by: Anas Badaha anasb@mellanox.com
SUMMARY
Adding Support for EVPN in onyx_bgp for onyx switches
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/network/onyx/onyx_bgp.py
test/units/modules/network/onyx/fixtures/onyx_bgp_show.cfg
test/units/modules/network/onyx/test_onyx_bgp.py
ADDITIONAL INFORMATION