fixes

Author: petercrocker

generators/common.py

@@ -959,6 +959,9 @@ async def create_oob_connections(
                 },
             )
 
+            # Save the cable first so it exists in the database
+            await cable.save(allow_upsert=True)
+
             # Set the connector relationship on both interfaces
             source_endpoint.connector = cable.id
             target_endpoint.connector = cable.id
@@ -969,9 +972,6 @@ async def create_oob_connections(
             batch.add(
                 task=target_endpoint.save, allow_upsert=True, node=target_endpoint
             )
-            batch.add(
-                task=cable.save, allow_upsert=True, node=cable
-            )
         try:
             async for node, _ in batch.execute():
                 hfid_str = ' -> '.join(node.hfid) if isinstance(node.hfid, list) else str(node.hfid)

generators/generate_dc.py

@@ -182,21 +182,21 @@ async def create_fabric_peering(self) -> None:
                 },
             )
 
+            # Save the cable first so it exists in the database
+            await cable.save(allow_upsert=True)
+
             # Set bidirectional connector relationship
             # This allows queries to traverse: interface → connector → cable → connected_endpoints
             source_endpoint.connector = cable.id
             target_endpoint.connector = cable.id
 
-            # Queue all objects for batch save
+            # Queue interface saves for batch
             batch.add(
                 task=source_endpoint.save, allow_upsert=True, node=source_endpoint
             )
             batch.add(
                 task=target_endpoint.save, allow_upsert=True, node=target_endpoint
             )
-            batch.add(
-                task=cable.save, allow_upsert=True, node=cable
-            )
 
         # Execute batch and log results
         async for node, _ in batch.execute():

objects/dc/dc-arista-s.yml

@@ -4,9 +4,9 @@ kind: Object
 spec:
   kind: TopologyDataCenter
   data:
-    - name: DC-3
+    - name: DC-Arista
       location: Frankfurt
-      description: Frankfurt Data Center
+      description: Arista Data Center with cEOS switches
       strategy: ospf-ibgp
       design: PHYSICAL DC ARISTA S
       emulation: true

objects/dc/dc-cisco-s-border-leafs.yml

@@ -4,9 +4,9 @@ kind: Object
 spec:
   kind: TopologyDataCenter
   data:
-    - name: DC-1
+    - name: DC-Cisco-Border
       location: Katowice
-      description: Katowice Data Center
+      description: Cisco Data Center with border leafs
       strategy: ospf-ibgp
       design: PHYSICAL DC CISCO S WITH BORDER LEAFS
       emulation: true

objects/dc/dc-cisco-s.yml

@@ -4,9 +4,9 @@ kind: Object
 spec:
   kind: TopologyDataCenter
   data:
-    - name: DC-2
+    - name: DC-Cisco
       location: Munich
-      description: Munich Data Center
+      description: Cisco Data Center with Nexus switches
       strategy: ebgp-ibgp
       design: PHYSICAL DC CISCO S
       emulation: true

objects/dc/dc-juniper-s.yml

@@ -4,9 +4,9 @@ kind: Object
 spec:
   kind: TopologyDataCenter
   data:
-    - name: DC-5
+    - name: DC-Juniper
       location: London
-      description: London Data Center with Juniper QFX5220-32CD
+      description: Juniper Data Center with Juniper QFX5220-32CD
       strategy: ospf-ibgp
       design: PHYSICAL DC JUNIPER S
       emulation: true

objects/dc/dc-sonic-border-leafs.yml

@@ -4,9 +4,9 @@ kind: Object
 spec:
   kind: TopologyDataCenter
   data:
-    - name: DC-4
+    - name: DC-SONiC
       location: Barcelona
-      description: Barcelona Data Center 4
+      description: SONiC Data Center with border leafs
       strategy: ebgp-ibgp
       design: PHYSICAL DC SONIC S WITH BORDER LEAFS
       emulation: true