Damien/fabric-orchestrator
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

[Schema] Remove duplicate InterfaceVxlan — keep VTEP only #44

New Issue
Closed
opened 2026-02-09 18:09:03 +00:00 by Damien · 1 comment
Damien commented 2026-02-09 18:09:03 +00:00
Owner
Copy Link

🔴 Critique — Duplication conceptuelle

Problème

InfraInterfaceVxlan (base.yml) et InfraVTEP (vlan_vxlan.yml) modélisent le même concept : les deux ont un device parent, un source_interface loopback, et un udp_port.

InfraVTEP est plus riche :

  • vlan_vni_mappings (Component)
  • learn_restrict (Dropdown)
  • source_address (IPHost)

Correction

  1. Supprimer InfraInterfaceVxlan de base.yml
  2. Conserver InfraVTEP dans vlan_vxlan.yml comme unique point d'entrée VXLAN
  3. Mettre à jour toute référence dans les autres schémas et le README
  4. Vérifier que le generic InfraInterface et la relation InfraDevice.interfaces ne cassent pas (InterfaceVxlan était un child de InfraInterface)

Fichiers

  • schemas/base.yml — supprimer InterfaceVxlan
  • schemas/vlan_vxlan.yml — vérifier InfraVTEP complet
  • schemas/README.md — mettre à jour le diagramme ER

Validation

  • infrahubctl schema check schemas/
  • Vérifier qu'aucun InfraInterfaceVxlan n'est référencé ailleurs

Related

## 🔴 Critique — Duplication conceptuelle ### Problème `InfraInterfaceVxlan` (base.yml) et `InfraVTEP` (vlan_vxlan.yml) modélisent le même concept : les deux ont un `device` parent, un `source_interface` loopback, et un `udp_port`. `InfraVTEP` est plus riche : - `vlan_vni_mappings` (Component) - `learn_restrict` (Dropdown) - `source_address` (IPHost) ### Correction 1. **Supprimer** `InfraInterfaceVxlan` de `base.yml` 2. **Conserver** `InfraVTEP` dans `vlan_vxlan.yml` comme unique point d'entrée VXLAN 3. Mettre à jour toute référence dans les autres schémas et le README 4. Vérifier que le generic `InfraInterface` et la relation `InfraDevice.interfaces` ne cassent pas (InterfaceVxlan était un child de InfraInterface) ### Fichiers - `schemas/base.yml` — supprimer InterfaceVxlan - `schemas/vlan_vxlan.yml` — vérifier InfraVTEP complet - `schemas/README.md` — mettre à jour le diagramme ER ### Validation - `infrahubctl schema check schemas/` - Vérifier qu'aucun `InfraInterfaceVxlan` n'est référencé ailleurs ## Related - Sub-task of: #41 - **⚡ Ordre d'exécution : 2/7** - Depends on: #43
Damien added the phase-2-minimal-reconciler label 2026-02-09 18:10:22 +00:00
Damien added this to the Fabric Orchestrator project 2026-02-09 19:03:29 +00:00
Damien moved this to In Progress in Fabric Orchestrator on 2026-02-09 19:03:44 +00:00
Damien added reference feature/41-infrahub-schema 2026-02-13 10:48:43 +00:00
Damien commented 2026-02-13 14:09:04 +00:00
Author
Owner
Copy Link

Implémenté ✅

Commit: 719ecfa

Changements

  • Supprimé InfraInterfaceVxlan de schemas/base.yml
  • Conservé InfraVTEP dans schemas/vlan_vxlan.yml comme unique modèle VXLAN

InfraVTEP couvre tous les attributs nécessaires : device (Parent), source_interface, source_address, udp_port, learn_restrict, vlan_vni_mappings (Component).

Validation

Schema check passé. Aucune référence cassée à InfraInterfaceVxlan.

## Implémenté ✅ **Commit:** [`719ecfa`](https://gitea.arnodo.fr/Damien/fabric-orchestrator/commit/719ecfafaaddfc29be66d336de3edcb9d8efce01) ### Changements - **Supprimé** `InfraInterfaceVxlan` de `schemas/base.yml` - **Conservé** `InfraVTEP` dans `schemas/vlan_vxlan.yml` comme unique modèle VXLAN `InfraVTEP` couvre tous les attributs nécessaires : `device` (Parent), `source_interface`, `source_address`, `udp_port`, `learn_restrict`, `vlan_vni_mappings` (Component). ### Validation Schema check passé. Aucune référence cassée à `InfraInterfaceVxlan`.
Damien moved this to Done in Fabric Orchestrator on 2026-02-25 13:59:06 +00:00
Sign in to join this conversation.
feature/41-infrahub-schema
Branches Tags
Labels
Clear labels
documentation
Documentation tasks
 phase-1-yang-discovery
Phase 1: YANG Path Discovery and Validation
 phase-2-minimal-reconciler
Phase 2: Minimal Reconciler (VLANs, VNIs)
 phase-3-full-fabric
Phase 3: Full Fabric Coverage (BGP, MLAG, VRFs)
 phase-4-event-driven
Phase 4: Event-Driven Automation and CLI
 research
Research and investigation tasks
No Label phase-2-minimal-reconciler
Milestone
No items
No Milestone
Projects
Clear projects
No project Fabric Orchestrator
Assignees
Clear assignees
Damien
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Damien/fabric-orchestrator#44
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?