aristanetworks / avd

Arista Validated Designs
https://avd.arista.com
Apache License 2.0
290 stars 209 forks source link

Allow for same mlag domain on multiple mlag pairs #2715

Closed ClausHolbechArista closed 1 year ago

ClausHolbechArista commented 1 year ago

Enhancement summary

The schema for node_groups does not allow the same group to be used on multiple MLAG pairs. Either relax this requirement or implement another mlag_domain knob.

Which component of AVD is impacted

eos_designs

Use case example

.

Describe the solution you would like

.

Describe alternatives you have considered

No response

Additional context

No response

Contributing Guide

gmuloc commented 1 year ago

so the current blocking point is that we can't have multiple time the same group name for a given node_type? (we have under 5clos molecule the same group name across multiple node type and it works afaik)

        node_groups:
          type: list
          convert_types:
            - dict
          primary_key: group
          items:
            "$ref": "#/keys/node_type/keys/defaults"
            type: dict
            keys:
              group:
                type: str
                required: true
                description: The Node Group Name is used for MLAG domain.
              nodes:
                "$ref": "#/keys/node_type/keys/nodes"

Could you please share a snippet of the use case here? I feel like having multiple times the same group name under a give type would confuse our logic if some parameters are different across node types and would rather prefer to have an extra knob for mlag_domain that defaults to the group name.

ClausHolbechArista commented 1 year ago

Use case is brownfield where everything is deployed with a single static mlag domain. They have no use of the groupings, but it is the only way we allow to configure mlag. With the list data models it is possible to define the same group everywhere and eos_designs works as expected. The only issue is the schema warning.