aristanetworks / avd

Arista Validated Designs
https://avd.arista.com
Apache License 2.0
284 stars 202 forks source link

Feat(eos_cli_config_gen): Support of BGP default-originate per VRF #3941 #4122

Closed mmaaloul closed 3 months ago

mmaaloul commented 3 months ago

Change Summary

Adding the support of default route originate per VRF for address families EVPN and VPN-IPV4/VPN-IPV6

Related Issue(s)

Fixes #3941

Component(s) name

arista.avd.eos_cli_config_gen

Proposed changes

Changes under schema fragment router_bgp:

How to test

Tested locally for 1 address family (evpn) using the 4 different flavours (default, using always, using route-map and using rcf)

Checklist

Repository Checklist

github-actions[bot] commented 3 months ago

Review docs on Read the Docs

To test this pull request:

# Create virtual environment for this testing below the current directory
python -m venv test-avd-pr-4122
# Activate the virtual environment
source test-avd-pr-4122/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/mmaaloul/ansible-avd.git@fix_issue_3941#subdirectory=python-avd" --force
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/mmaaloul/ansible-avd.git#/ansible_collections/arista/avd/,fix_issue_3941 --force
# Optional: Install AVD examples
cd test-avd-pr-4122
ansible-playbook arista.avd.install_examples
github-actions[bot] commented 3 months ago

This pull request has conflicts, please resolve those before we can evaluate the pull request.

ClausHolbechArista commented 3 months ago

We have moved a lot of things around in the code base these last few days, so I will rebase this one for you.

github-actions[bot] commented 3 months ago

Conflicts have been resolved. A maintainer will review the pull request shortly.