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): Add support for BGP link-state address-family #3211
Feat(eos_cli_config_gen): Add support for BGP link-state address-family #3211
Conversation
...llections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/router_bgp.schema.yml
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/documentation/router-bgp.j2
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.
Only a minor comment on docs. LGTM.
{% if router_bgp.address_family_link_state.path_selection.roles is arista.avd.defined %} | ||
{% set path_selection_roles = [] %} | ||
{% if router_bgp.address_family_link_state.path_selection.roles.producer is arista.avd.defined(true) %} | ||
{% do path_selection_roles.append("- producer") %} |
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.
In other docs we just join the raw strings on <br>
with no -
or *
.
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
40ac706
to
e4076f9
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
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.
Tested and reviewed, LGTM!
Change Summary
Adding support for BGP-LS to prepare Path-Selection feature
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
NOTE FOR REVIEWERS: Please note that in EOS CLI, the role
producer
not shown, instead thepath-selection
entry withoutrole ...
is implicitly enabling the producer role. Feel free to reach out to me for more details. I have tried to be very explicit in the descriptionHow to test
molecule
Checklist
Repository Checklist