Replace bmca to dynamic in connected endpoints role

aristanetworks · Nov 15, 2024 · ab63faf · ab63faf
1 parent 4b1fa02
commit ab63faf
Show file tree

Hide file tree

Showing 9 changed files with 25 additions and 15 deletions.
diff --git a/...llections/arista/avd/molecule/eos_designs_unit_tests/intended/configs/ptp-tests-leaf1.cfg b/...llections/arista/avd/molecule/eos_designs_unit_tests/intended/configs/ptp-tests-leaf1.cfg
@@ -193,7 +193,7 @@ interface Ethernet12
    channel-group 12 mode active
 !
 interface Ethernet13
-   description SERVER_bmca-endpoint_eth1
+   description SERVER_dynamic-endpoint_eth1
    no shutdown
    switchport access vlan 11
    switchport mode access

diff --git a/...rista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/ptp-tests-leaf1.yml b/...rista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/ptp-tests-leaf1.yml
@@ -359,11 +359,11 @@ ethernet_interfaces:
     id: 14
     mode: active
 - name: Ethernet13
-  peer: bmca-endpoint
+  peer: dynamic-endpoint
   peer_interface: eth1
   peer_type: server
-  port_profile: PTP-profile-BMCA
-  description: SERVER_bmca-endpoint_eth1
+  port_profile: PTP-profile-dynamic
+  description: SERVER_dynamic-endpoint_eth1
   shutdown: false
   switchport:
     enabled: true

diff --git a/...collections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/PTP_TESTS.yml b/...collections/arista/avd/molecule/eos_designs_unit_tests/inventory/group_vars/PTP_TESTS.yml
@@ -61,20 +61,20 @@ tenants:
                 ip_address: 172.17.11.1/24
 
 port_profiles:
-  - profile: PTP-profile-BMCA
+  - profile: PTP-profile-dynamic
     vlans: 11
     mode: access
     ptp:
       enabled: true
-      endpoint_role: bmca
+      endpoint_role: dynamic
 
 servers:
-  - name: bmca-endpoint
+  - name: dynamic-endpoint
     adapters:
       - endpoint_ports: [ eth1 ]
         switch_ports: [ Ethernet13 ]
         switches: [ ptp-tests-leaf1 ]
-        profile: PTP-profile-BMCA
+        profile: PTP-profile-dynamic
 
   - name: video-endpoint
     adapters:

diff --git a/...ble_collections/arista/avd/roles/eos_designs/docs/tables/connected-endpoints.md b/...ble_collections/arista/avd/roles/eos_designs/docs/tables/connected-endpoints.md
diff --git a/ansible_collections/arista/avd/roles/eos_designs/docs/tables/network-ports.md b/ansible_collections/arista/avd/roles/eos_designs/docs/tables/network-ports.md
diff --git a/ansible_collections/arista/avd/roles/eos_designs/docs/tables/port-profiles.md b/ansible_collections/arista/avd/roles/eos_designs/docs/tables/port-profiles.md
diff --git a/python-avd/pyavd/_eos_designs/schema/eos_designs.schema.yml b/python-avd/pyavd/_eos_designs/schema/eos_designs.schema.yml
diff --git a/python-avd/pyavd/_eos_designs/schema/schema_fragments/defs_adapter_config.schema.yml b/python-avd/pyavd/_eos_designs/schema/schema_fragments/defs_adapter_config.schema.yml
@@ -135,11 +135,13 @@ $defs:
             default: false
           endpoint_role:
             type: str
+            description: Set `dynamic` instead of `bmca`(deprecated).
             default: "follower"
             valid_values:
               - "bmca"
               - "default"
               - "follower"
+              - "dynamic"
           profile:
             type: str
             default: "aes67-r16-2016"

diff --git a/python-avd/pyavd/_eos_designs/structured_config/connected_endpoints/utils.py b/python-avd/pyavd/_eos_designs/structured_config/connected_endpoints/utils.py
@@ -213,7 +213,7 @@ def _get_adapter_ptp(self: AvdStructuredConfigConnectedEndpoints, adapter: dict)
 
         ptp_config["enable"] = True
 
-        if get(adapter, "ptp.endpoint_role") != "bmca":
+        if get(adapter, "ptp.endpoint_role") != "dynamic":
             ptp_config["role"] = "master"
 
         ptp_config.pop("profile", None)