[PATCH proxmox-ve-rs v3 01/13] frr: add fabric properties to ISIS types and rename domain

[Gabriel Goller <g.goller@proxmox.com>]

The ISIS types in proxmox-frr are used by pve-network to generate FRR
configuration for ISIS controllers. To share these types between ISIS
fabric and controller implementations, add properties required by fabrics
and rename the "domain" field to "fabric_id" for consistency with fabric
terminology. This rename is also applied in pve-network.

Signed-off-by: Gabriel Goller <g.goller@proxmox.com>
---
 proxmox-frr-templates/templates/isisd.jinja | 23 +++++++++++++++---
 proxmox-frr/src/ser/isis.rs                 | 27 ++++++++++++++++++++-
 proxmox-frr/src/ser/mod.rs                  | 11 +++++++++
 3 files changed, 56 insertions(+), 5 deletions(-)

diff --git a/proxmox-frr-templates/templates/isisd.jinja b/proxmox-frr-templates/templates/isisd.jinja
index e3c9292d575c..a6dd748163b8 100644
--- a/proxmox-frr-templates/templates/isisd.jinja
+++ b/proxmox-frr-templates/templates/isisd.jinja
@@ -21,11 +21,26 @@ exit
 {% endfor %}
 {% for interface_name, interface_config in isis.interfaces|items %}
 {% call interface(interface_name, interface_config.addresses_v4, interface_config.addresses_v6) %}
-{% if interface_config.domain and interface_config.is_ipv4 %}
- ip router isis {{ interface_config.domain }}
+{% if interface_config.fabric_id and interface_config.is_ipv4 %}
+ ip router isis {{ interface_config.fabric_id }}
 {% endif %}
-{% if interface_config.domain and interface_config.is_ipv6 %}
- ipv6 router isis {{ interface_config.domain }}
+{% if interface_config.fabric_id and interface_config.is_ipv6 %}
+ ipv6 router isis {{ interface_config.fabric_id }}
+{% endif %}
+{% if interface_config.passive %}
+ isis passive
+{% endif %}
+{% if interface_config.hello_interval %}
+ isis hello-interval {{ interface_config.hello_interval }}
+{% endif %}
+{% if interface_config.csnp_interval %}
+ isis csnp-interval {{ interface_config.csnp_interval }}
+{% endif %}
+{% if interface_config.hello_multiplier %}
+ isis hello-multiplier {{ interface_config.hello_multiplier }}
+{% endif %}
+{% if interface_config.point_to_point %}
+ isis network point-to-point
 {% endif %}
 {% for line in interface_config.custom_frr_config %}
 {{ line }}
diff --git a/proxmox-frr/src/ser/isis.rs b/proxmox-frr/src/ser/isis.rs
index 211c5b21e9e1..6ac30e714113 100644
--- a/proxmox-frr/src/ser/isis.rs
+++ b/proxmox-frr/src/ser/isis.rs
@@ -9,6 +9,18 @@ use crate::ser::FrrWord;
 #[derive(Clone, Debug, PartialEq, Eq, Hash, PartialOrd, Ord, Serialize, Deserialize)]
 pub struct IsisRouterName(FrrWord);
 
+impl From<FrrWord> for IsisRouterName {
+    fn from(value: FrrWord) -> Self {
+        Self(value)
+    }
+}
+
+impl IsisRouterName {
+    pub fn new(name: FrrWord) -> Self {
+        Self(name)
+    }
+}
+
 #[derive(Clone, Debug, PartialEq, Eq, Hash, PartialOrd, Ord, Serialize, Deserialize)]
 pub enum IsisLevel {
     #[serde(rename = "level-1")]
@@ -37,11 +49,24 @@ pub struct IsisRouter {
 
 #[derive(Clone, Debug, PartialEq, Eq, Hash, PartialOrd, Ord, Serialize, Deserialize)]
 pub struct IsisInterface {
-    pub domain: IsisRouterName,
+    pub fabric_id: IsisRouterName,
     #[serde(deserialize_with = "proxmox_serde::perl::deserialize_bool")]
     pub is_ipv4: bool,
     #[serde(deserialize_with = "proxmox_serde::perl::deserialize_bool")]
     pub is_ipv6: bool,
+    #[serde(default, deserialize_with = "proxmox_serde::perl::deserialize_bool")]
+    pub passive: Option<bool>,
+    // Note: openfabric is very similar to isis, so we can use the same properties here
+    #[serde(default)]
+    pub hello_interval: Option<proxmox_sdn_types::openfabric::HelloInterval>,
+    #[serde(default)]
+    pub csnp_interval: Option<proxmox_sdn_types::openfabric::CsnpInterval>,
+    #[serde(default)]
+    pub hello_multiplier: Option<proxmox_sdn_types::openfabric::HelloMultiplier>,
+
+    #[serde(default, deserialize_with = "proxmox_serde::perl::deserialize_bool")]
+    pub point_to_point: bool,
+
     #[serde(default)]
     pub custom_frr_config: Vec<String>,
 }
diff --git a/proxmox-frr/src/ser/mod.rs b/proxmox-frr/src/ser/mod.rs
index b651121b4a82..290e14042491 100644
--- a/proxmox-frr/src/ser/mod.rs
+++ b/proxmox-frr/src/ser/mod.rs
@@ -160,6 +160,16 @@ impl From<openfabric::OpenfabricInterface> for Interface<openfabric::OpenfabricI
     }
 }
 
+impl From<isis::IsisInterface> for Interface<isis::IsisInterface> {
+    fn from(value: isis::IsisInterface) -> Self {
+        Interface {
+            addresses_v4: Vec::new(),
+            addresses_v6: Vec::new(),
+            properties: value,
+        }
+    }
+}
+
 impl From<ospf::OspfInterface> for Interface<ospf::OspfInterface> {
     fn from(value: ospf::OspfInterface) -> Self {
         Interface {
@@ -191,6 +201,7 @@ pub struct IpRoute {
 pub enum FrrProtocol {
     Ospf,
     Openfabric,
+    Isis,
     Bgp,
 }
 
-- 
2.47.3