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

Feat(anta_runner): Add ANTA BGP neighbors reachability tests #5188

Merged

Conversation

carl-baillargeon
Copy link
Contributor

Change Summary

Add additional inputs to VerifyReachability to cover BGP neighbors with update source configured.

Also simplify the bgp_neighbors property logic.

Component(s) name

arista.avd.anta_runner

Proposed changes

See Change Summary.

How to test

---
- name: Run ANTA
  hosts: FABRIC
  connection: local
  gather_facts: false
  tasks:
    - name: Run ANTA on EOS devices
      import_role:
        name: arista.avd.anta_runner

ansible-playbook playbook/fabric-validate.yml

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)

Sorry, something went wrong.

Copy link

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-5188
# Activate the virtual environment
source test-avd-pr-5188/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/carl-baillargeon/avd.git@anta/add_reachability_tests#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/carl-baillargeon/avd.git#/ansible_collections/arista/avd/,anta/add_reachability_tests --force
# Optional: Install AVD examples
cd test-avd-pr-5188
ansible-playbook arista.avd.install_examples

@github-actions github-actions bot added state: CI Updated CI scenario have been updated in the PR state: Documentation role Updated labels Mar 19, 2025
Copy link
Contributor

@danieltudares danieltudares left a comment

Choose a reason for hiding this comment

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

Successfully ran the test in the lab. Tested manually shutting down some random interfaces and the report came back with a proper response pointing at failed BGP peering in idle state. After restoring the interfaces the report came back with all BGP tests passing.
LGTM!

Copy link

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

@github-actions github-actions bot added the state: conflict PR with conflict label Mar 21, 2025
@github-actions github-actions bot removed the state: conflict PR with conflict label Mar 21, 2025
Copy link

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

@carl-baillargeon carl-baillargeon changed the title Feat: Add ANTA BGP neighbors reachability tests Feat(anta_runner): Add ANTA BGP neighbors reachability tests Mar 21, 2025
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 a3ef574 into aristanetworks:devel Mar 21, 2025
52 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

4 participants