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

Update for act #31

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

chetryan
Copy link

@chetryan chetryan commented Mar 27, 2023

Overview

At the moment, the repository offers an example AVD configuration that would configure a L3LS using CVP. However the arbitrary choices of configuration would require changes before being used with the Arista Sample Sample topology that is the default in Arista Cloud Test.

This change changes certain choices to make this repository work with the Sample Topology with little to now changes, making it easier for someone new to both ACT and AVD to get started quickly.

Adds

  • playbooks/dc1-fabric-validate.yml - as an example of the eos_validate_state role

Changes

  • Switch Makefile to use the avd-all-in-one docker image, managed by this project : https://github.com/arista-netdevops-community/avd-all-in-one-container
  • Add custom_structured_config to set the TerminAttr to use the token authentication, which is the default method in the more recent CVP versions
  • Change Management subnet to 192.168.0.0/24
  • Add the default vrf to the management api http-commands to allow eAPI communication even when a MGMT vrf is defined on the devices
  • Set the hash of the cvpadmin password based on the cleartext value supplied in the inventory. This solves the issue where the user had to copy the hash from a configured device.

Removes

  • requirements.txt - the Makefile has been updated to use the requirements.txt specified in the arista.avd collection

Impact if not merged

  • If this pull request is not merged to master, any user making use of this repo to configure a Sample lab would need to make changes to the group_vars to get working configuration on the devices in the ACT lab. This is prone to human errors, especially if the user is not familiar both AVD and ACT.

@chetryan chetryan marked this pull request as draft March 27, 2023 08:35
Switch to using AVD all in one from github container reg
Add makefile commands to match ACT userguide
We now use the requirements specified in the arista.avd collection
@chetryan chetryan marked this pull request as ready for review March 31, 2023 17:55
@chetryan chetryan changed the title Draft: Update for act Update for act Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant