Cumulus NVUE: Add support for OSPFv2 timers and priority (#1765)

ipspace · Jan 6, 2025 · 7010bfd · 7010bfd
1 parent f54e4d2
commit 7010bfd
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 2 deletions.
diff --git a/docs/module/ospf.md b/docs/module/ospf.md
@@ -59,7 +59,7 @@ The following table describes per-platform support of individual router-level OS
 | Cisco IOS XE[^18v]       | ✅| ✅| ✅| ✅| ✅|
 | Cisco Nexus OS           | ✅| ✅| ✅| ❌ | ❌ |
 | Cumulus Linux            | ✅| ✅| ✅| ✅| ✅|
-| Cumulus Linux 5.0 (NVUE) | ✅| ✅| ❌ | ❌ | ❌ |
+| Cumulus Linux 5.x (NVUE) | ✅| ✅| ❌ | ❌ | ❌ |
 | Dell OS10 ([❗](caveats-os10)) | | ✅| ✅| ✅| ❌ | ❌ |
 | Fortinet FortiOS         |[❗](caveats-fortios)| ✅ | ❌ | ❌ | ❌ |
 | FRR                      | ✅| ✅| ✅| ✅| ✅|
@@ -119,7 +119,7 @@ The following table documents the common interface-level OSPF features:
 | Cisco IOS XRv            | ✅ | ✅ | ✅ | ✅ |
 | Cisco Nexus OS           | ✅ | ✅ | ✅ | ✅ |
 | Cumulus Linux            | ✅ | ✅ | ✅ | ✅ |
-| Cumulus Linux 5.0 (NVUE) | ✅ | ✅ | ✅ | ✅ |
+| Cumulus Linux 5.x (NVUE) | ✅ | ✅ | ✅ | ✅ |
 | Dell OS10                | ✅ | ✅ | ❌  | ✅ |
 | Fortinet FortiOS         | ✅ | [❗](caveats-fortios) | ❌ | ✅ |
 | FRR                      | ✅ | ✅ | ✅ | ✅ |
@@ -154,6 +154,7 @@ These devices also support optional OSPF interface attributes:
 | Cisco IOS XE[^18v]       | ✅ | ✅ | ✅ | ❌  |
 | Cisco Nexus OS           | ✅ | ✅ | ✅ | ❌  |
 | Cumulus Linux 4.x        | ✅ | ✅ | ✅ | ❌  |
+| Cumulus Linux 5.x (NVUE) | ✅ | ✅ | ❌  | ❌  |
 | Dell OS10                | ✅ | ✅ | ✅ | ❌  |
 | FRR                      | ✅ | ✅ | ✅ | ❌  |
 

diff --git a/netsim/ansible/templates/ospf/cumulus_nvue.j2 b/netsim/ansible/templates/ospf/cumulus_nvue.j2
@@ -38,6 +38,18 @@
 {%       if l.ospf.passive | default(False) %}
             passive: on
 {%       endif %}
+{%       if l.ospf.timers is defined %}
+            timers:
+{%         if l.ospf.timers.dead is defined %}
+              dead-interval: {{ l.ospf.timers.dead }}
+{%         endif %}              
+{%         if l.ospf.timers.hello is defined %}
+              hello-interval: {{ l.ospf.timers.hello }}
+{%         endif %}
+{%       endif %}
+{%       if l.ospf.priority is defined %}
+            priority: {{ l.ospf.priority }}
+{%       endif %}
 {%     endfor %}
 
 {%   else %}

diff --git a/netsim/devices/cumulus_nvue.yml b/netsim/devices/cumulus_nvue.yml
@@ -40,6 +40,8 @@ features:
         backup_ip: loopback.ipv4  # Use loopback.ipv4 as a backup IP address for peerlink
   ospf:
     unnumbered: True
+    timers: True
+    priority: True
   stp:
     supported_protocols: [ stp, rstp, pvrst ]  # PVRST requires release 5.6.0 or higher
     enable_per_port: True