Skip to content
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

Fix(eos_designs): eBGP rfc5549 creates invalid configuration for MLAG scenarios #2950

Merged

Conversation

carlbuchmann
Copy link
Member

@carlbuchmann carlbuchmann commented Jun 1, 2023

Change Summary

Fix issues in the eBGP rfc5549 design with MLAG configuration scenario as described in #2906

Related Issue(s)

Fixes #2906

Component(s) name

arista.avd.eos_designs
arista.avd.eos_cli_config_gen

Proposed changes

If underlay_rfc5549 is True and overlay_mlag_rfc5549 is True configure:

ipv6 unicast-routing vrf <vrf_name>

If underlay_rfc5549 is True and overlay_mlag_rfc5549 is False configure:

router_bgp <asn>
   vrf <vrf_name>
      address-family ipv4
         no neighbor <mlag-peer-ip> next-hop address-family ipv6

How to test

Run Molecule scenario: molecule converge -s evpn_underlay_rfc5549_overlay_ebgp

Checklist

User Checklist

  • Update test case for evpn_underlay_rfc5549_overlay_ebgp
  • Fix underlay_rfc5549 is True and overlay_mlag_rfc5549 is True
  • Add eos_cli_config_gen knob for router_bgp.vrfs.address_family_ipv4."no neighbor <ip_address> next-hop address-family ipv6"
  • Fix If underlay_rfc5549 is True and overlay_mlag_rfc5549 is False

@github-actions github-actions bot added role: eos_designs issue related to eos_designs role state: CI Updated CI scenario have been updated in the PR labels Jun 1, 2023
@github-actions github-actions bot added role: eos_cli_config_gen issue related to eos_cli_config_gen role state: Documentation role Updated labels Jun 2, 2023
@ClausHolbechArista ClausHolbechArista marked this pull request as ready for review June 2, 2023 09:56
@ClausHolbechArista ClausHolbechArista requested a review from a team as a code owner June 2, 2023 09:56
@ClausHolbechArista
Copy link
Contributor

Tested in lab where the issue was recreated, and these fixes work for both cases.

@ClausHolbechArista ClausHolbechArista requested a review from a team June 2, 2023 10:30
@carlbuchmann
Copy link
Member Author

@ClausHolbechArista thank you - LGTM, added unit tests for eos_cli_config_gen

@carlbuchmann carlbuchmann added this to the v4.0.0-rc3 milestone Jun 2, 2023
@onurgashi
Copy link
Contributor

LGTM! Thanks for fixing this.

@carlbuchmann carlbuchmann merged commit 2ff7eea into aristanetworks:devel Jun 2, 2023
35 checks passed
@carlbuchmann carlbuchmann added the cherry-pick-for-3.8.x PR to be cherry-picked to releases/v3.8.x brach later label Jun 2, 2023
Shivani-chourasiya pushed a commit to Shivani-chourasiya/ansible-avd that referenced this pull request Jul 21, 2023
… scenarios (aristanetworks#2950)

Co-authored-by: Claus Holbech <holbech@arista.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn: Fix(eos_designs) role: eos_cli_config_gen issue related to eos_cli_config_gen role role: eos_designs issue related to eos_designs role state: CI Updated CI scenario have been updated in the PR state: Documentation role Updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IBGP neighbors could be established for MLAG peers in each VRF when we set ‘overlay_mlag_rfc5549: true’
3 participants