Skip to content

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.

1 participant