-
Notifications
You must be signed in to change notification settings - Fork 230
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 MPLS resolution RIBs #3592
Feat(eos_cli_config_gen): Add support for MPLS resolution RIBs #3592
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
...ections/arista/avd/molecule/eos_cli_config_gen/documentation/devices/queue_monitor_length.md
Outdated
Show resolved
Hide resolved
...lections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/router-bgp-evpn-mpls.yml
Outdated
Show resolved
Hide resolved
643696f
to
ea1e8de
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
ca3b674
to
4166cfc
Compare
4166cfc
to
e1ad879
Compare
...lections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/router-bgp-evpn-mpls.yml
Show resolved
Hide resolved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
…mily-support-for-tunnel-rib
Please rebase and delete the .pickle files when it asks. We have removed those from the repo. |
...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
…s/schema_fragments/router_bgp.schema.yml Co-authored-by: Claus Holbech <[email protected]>
…tes/documentation/router-bgp.j2 Co-authored-by: Claus Holbech <[email protected]>
…tes/eos/router-bgp.j2 Co-authored-by: Claus Holbech <[email protected]>
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
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
Change Summary
Add support for the
next-hop mpls resolution ribs
command.Since the command supports both a Primary-RIB and a Fallback-RIB and both can be quite complex, I propose that we support string input for now.
Related Issue(s)
Fixes #3415
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
How to test
molecule test -s eos_cli_config_gen
Checklist
User Checklist
Repository Checklist