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_validate_state): ANTA Refactor BGP test to now also support direct neighbors and IPv6 AFI #3572

Merged
merged 10 commits into from Feb 9, 2024

Conversation

carl-baillargeon
Copy link
Contributor

Change Summary

Refactor AvdTestBGP to now also support direct neighbors and IPv6 in addition of peer groups.

Related Issue(s)

N/A

Component(s) name

arista.avd.eos_validate_state ANTA

Proposed changes

AvdTestBGP will now test all activated neighbors and peer groups in the IPv4, IPv6 and EVPN address families of VRF default. This will NOT test BGP neighbors configured in a VRF (coming soon in a separate PR).

How to test

You can add custom structured_config for BGP under specific nodes:

l3leaf:  
  node_groups:
    - group: DC2_BL
      mlag: false
      structured_config:
        router_bgp:
          address_family_ipv4:
            neighbors:
            - ip_address: 123.1.1.10
              activate: true
            - ip_address: 123.1.1.11
              activate: true
          neighbors:
          - ip_address: 123.1.1.10
            remote_as: '1234'
            description: External IPv4 BGP peer
          - ip_address: 123.1.1.11
            remote_as: '1234'
            description: External IPv4 BGP peer
          - ip_address: fd5a:fe45:8831:06c5::a
            remote_as: '12345'
          - ip_address: fd5a:fe45:8831:06c5::b
            remote_as: '12345'
          address_family_ipv6:
            neighbors:
            - ip_address: fd5a:fe45:8831:06c5::a
              activate: true
            - ip_address: fd5a:fe45:8831:06c5::b
              activate: true

eos_validate_state in ANTA mode will now test the additional activated IPv4, IPv6 neighbors.

Also see eos_validate_state molecule scenario.

Checklist

User Checklist

  • N/A

Repository Checklist

  • My code has been rebased from devel before I start
  • I have read the CONTRIBUTING document.
  • My change requires a change to the documentation and documentation have been updated accordingly.
  • I have updated molecule CI testing accordingly. (check the box if not applicable)

Copy link
Member

@carlbuchmann carlbuchmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, IPv4 works as expected in my lab. Need to test IPv6!

@carlbuchmann carlbuchmann requested a review from a team February 1, 2024 02:40
@github-actions github-actions bot added the state: conflict PR with conflict label Feb 2, 2024
Copy link

github-actions bot commented Feb 2, 2024

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

Copy link
Contributor

@MaheshGSLAB MaheshGSLAB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Only one type

@github-actions github-actions bot removed the state: conflict PR with conflict label Feb 2, 2024
Copy link

github-actions bot commented Feb 2, 2024

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

@github-actions github-actions bot added role: eos_validate_state state: CI Updated CI scenario have been updated in the PR labels Feb 2, 2024
Copy link
Member

@carlbuchmann carlbuchmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@carlbuchmann carlbuchmann merged commit af15ee7 into aristanetworks:devel Feb 9, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants