-
Notifications
You must be signed in to change notification settings - Fork 234
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_designs): Optional dedicated MLAG peer group for VRFs #4881
Feat(eos_designs): Optional dedicated MLAG peer group for VRFs #4881
Conversation
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-4881
# Activate the virtual environment
source test-avd-pr-4881/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/Vibhu-gslab/avd.git@mlag_peer_group_4826#subdirectory=python-avd" --force
# Point Ansible collections path to the Python virtual environment
export ANSIBLE_COLLECTIONS_PATH=$VIRTUAL_ENV/ansible_collections
# Install Ansible collection
ansible-galaxy collection install git+https://github.com/Vibhu-gslab/avd.git#/ansible_collections/arista/avd/,mlag_peer_group_4826 --force
# Optional: Install AVD examples
cd test-avd-pr-4881
ansible-playbook arista.avd.install_examples |
1748a4c
to
ca1ab75
Compare
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/mlag/__init__.py
Outdated
Show resolved
Hide resolved
072841d
to
4c2087d
Compare
...lections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/BGP_PEER_GROUPS.yml
Outdated
Show resolved
Hide resolved
...ollections/arista/avd/molecule/eos_designs_unit_tests/intended/configs/bgp-peer-groups-1.cfg
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/overlay/router_bgp.py
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/overlay/router_bgp.py
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/network_services/router_bgp.py
Outdated
Show resolved
Hide resolved
bdfb890
to
af8454c
Compare
...lections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/BGP_PEER_GROUPS.yml
Outdated
Show resolved
Hide resolved
...e_collections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/DC1_FABRIC.yml
Show resolved
Hide resolved
...ons/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/EVPN_MULTICAST_TESTS.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/schema/schema_fragments/bgp_peer_groups.schema.yml
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/mlag/__init__.py
Outdated
Show resolved
Hide resolved
2377bf4
to
02bfe61
Compare
python-avd/pyavd/_eos_designs/structured_config/mlag/__init__.py
Outdated
Show resolved
Hide resolved
python-avd/pyavd/_eos_designs/structured_config/network_services/router_bgp.py
Outdated
Show resolved
Hide resolved
for more information, see https://pre-commit.ci
…_groups.schema.yml Co-authored-by: Guillaume Mulocher <[email protected]>
for more information, see https://pre-commit.ci
…_groups.schema.yml Co-authored-by: Claus Holbech <[email protected]>
Co-authored-by: Claus Holbech <[email protected]>
for more information, see https://pre-commit.ci
…_groups.schema.yml Co-authored-by: laxmikantchintakindi <[email protected]>
for more information, see https://pre-commit.ci
68ddcd0
to
2a4b876
Compare
@ksantos01 can you test the PR and see if that works for you? |
@ksantos01 do you have time to test this PR this week? Then we can get it into 5.2 releasing on Friday. |
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.
I have tested it and it works for my use case
Quality Gate passedIssues Measures |
Change Summary
Currently MLAG-IPv4-UNDERLAY-PEER is used for the iBGP session between MLAG pairs in the underlay and overlay. This approach can be misleading due to the use of UNDERLAY word in the BGP peer group name.
The following is an example of the configuration that AVD generates today
Related Issue(s)
Fixes #4826
Fixes #4916
Component(s) name
arista.avd.eos_designs
Proposed changes
Plan for this issue to be non-breaking:
1/ introduce a new knob to use_separate_mlag_peer_group_for_overlay: with a default of false.
2/ add a peer_group in bgp_peer_groups for MLAG overlay to be used when the toggle from 1/ is true
How to test
Molecule testing
Checklist
Repository Checklist