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
Feat(eos_cli_config_gen): Support of next_hop_unchanged under EVPN address family #3232
Conversation
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.
Overall LGTM - just one comment regarding adding a new host to molecule only for one line. @ClausHolbechArista is it ok here to put it in router-bgp-evpn
even if the config "might be wrong"
ansible_collections/arista/avd/molecule/eos_cli_config_gen/inventory/hosts.ini
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/router-bgp.j2
Outdated
Show resolved
Hide resolved
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. Note my comment on checking the order on EOS. We try to follow the ordering on EOS for anything new.
Change Summary
Permit the explicit configuration of next-hop-unchanged knob under address-family evpn
Related Issue(s)
Fixes #3102
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
1/ Modified schema fragment "router_bgp" and template "router-bgp" accordingly
2/ Modified device documentation template accordingly
3/ Added a molecule test (new host "router-bgp-evpn-route-server") as the knob is specific to route server role
How to test
The knob is present under address-family evpn in the generated intended configuration
Checklist
User Checklist
Repository Checklist