diff --git a/CHANGELOG.md b/CHANGELOG.md index 6c14b31..9687d2c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,6 +15,13 @@ Le devis émet désormais `` et dit pourquoi, à figer apr `underlay.passerelle_sortie` **garde** son sens : c'est l'adresse du pare-feu sur le lien de transit, donc la sortie de l'**underlay**. Deux choses distinctes qu'il ne faut pas confondre. +### Corrigé — la description du lien affichait le marqueur du prochain saut +Régression de la correction ci-dessus : le champ décrivant le **câblage** du lien de transit +réutilisait la variable du **prochain saut**. Les deux étaient identiques jusqu'à la bascule +SDN ; elles ont divergé, et la section 1 annonçait `switch `. Sur ce +lien, le commutateur est bien à `10.0.4.6` — le nœud de sortie n'y figure pas. Le champ décrit +désormais le câblage, indépendamment du routage. + ### Corrigé — une contradiction entre les deux devis, antérieure au SDN La section 0 du devis frontière demandait de router les réseaux d'administration **vers `10.0.4.6`** — le SVI du commutateur lui-même. `make devis-reseau` émet `10.0.4.1`, l'adresse diff --git a/scripts/devis_opnsense.py b/scripts/devis_opnsense.py index 512d8ad..598410a 100644 --- a/scripts/devis_opnsense.py +++ b/scripts/devis_opnsense.py @@ -319,7 +319,10 @@ def construire(tenants: list[tuple[str, str, dict]]) -> dict: "nom": (transit or {}).get("nom"), "vlan": (transit or {}).get("vlan"), "sous_reseau": (transit or {}).get("sous_reseau"), - "svi_switch": saut, + # L'adresse du commutateur SUR LE LIEN, pas le prochain saut des tenants : + # les deux ont diverge avec le SDN, ou le voisin de la frontiere devient un + # noeud de sortie. Ce champ decrit le cablage, pas le routage. + "svi_switch": (transit or {}).get("passerelle"), "adresse_frontiere": (transit or {}).get("passerelle_sortie"), "nom_frontiere": nom_frontiere(transit), } if transit else None,