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(eos_designs): Build AVD topology from CloudVision I&T Studio data #3223

Conversation

ClausHolbechArista
Copy link
Contributor

@ClausHolbechArista ClausHolbechArista commented Oct 10, 2023

Change Summary

Build AVD topology from CloudVision I&T Studio data

Component(s) name

arista.avd.eos_designs

Proposed changes

At first to be used with the studios integration, where the topology studio inputs resemble this cv_topology data model.
Depends on default_interfaces to determine the correct interface role.


CloudVision Topology.
Used to generate AVD configurations directly from the given topology if use_cv_topology is set to true.
Neighbor hostnames must match the inventory hostnames of the AVD inventory to be taken into consideration.
Requires default_interfaces to be set, to detect the proper interface roles automatically.
Currently providing the following configurations based on the given CloudVision topology and default_interfaces:

  • uplink_switches.
  • uplink_interfaces
  • uplink_switch_interfaces
  • mlag_interfaces
  • # mlag_peer (future)
  • platform
  • mgmt_interface
    NOTES:
  • Any derived configuration can be overridden by setting the key manually.
    Even keys set under node type defaults will take precedence over these derived configurations.
  • when using parallel links between the same devices for L3 uplinks it is important to set
    max_uplink_switches and max_parallel_uplinks to ensure consistent IP addressing.
cv_topology:
  - hostname: <str>
    platform: <str>
    interfaces:
      - name: <str>
        neighbor: <str>
        neighbor_interface: <str>
use_cv_topology: <bool>

How to test

Updated an existing molecule scenario to ensure that the same config is generated without the manual inputs.

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)

@github-actions github-actions bot added state: CI Updated CI scenario have been updated in the PR state: Documentation role Updated role: eos_designs issue related to eos_designs role labels Oct 10, 2023
@ClausHolbechArista ClausHolbechArista changed the title Feat(eos_designs): Build topology from LLDP derived topology data Feat(eos_designs): Build AVD topology from CloudVision I&T Studio data Oct 12, 2023
…riables.md

Co-authored-by: Carl Buchmann <carl.buchmann@arista.com>
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 8e3cfce into aristanetworks:devel Oct 13, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rn: Feat(eos_designs) 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.

None yet

2 participants