Fix: test all combinations of IPv4/IPv6 addresses

ipspace · Dec 1, 2024 · c7843d0 · c7843d0
1 parent d91c8fa
commit c7843d0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 6 deletions.
diff --git a/netsim/devices/cumulus_nvue.yml b/netsim/devices/cumulus_nvue.yml
@@ -1,11 +1,17 @@
 description: Cumulus VX 5.x configured with NVUE
 interface_name: swp{ifindex}
 lag_interface_name: "bond{lag.ifindex}"
-loopback_interface_name: lo{ifindex if ifindex else ""}     # Can assign multiple IPs to 'lo' interface, ifindex ignored by template
+#
+# Can assign multiple IPs to 'lo' interface, ifindex ignored by template
+loopback_interface_name: lo{ifindex if ifindex else ""}
 mgmt_if: eth0
-mtu: 1500                                                   # Set default MTU for all providers the same
+#
+# Set the same default MTU for all providers
+mtu: 1500
 libvirt:
-  image: CumulusCommunity/cumulus-vx:5.10.0                 # Latest as of November 2024, supports PVRST+ on single vlan-aware bridge
+  #
+  # Latest as of November 2024, supports PVRST+ on single vlan-aware bridge
+  image: CumulusCommunity/cumulus-vx:5.10.0
 virtualbox:
   image: CumulusCommunity/cumulus-vx:5.10.0
 group_vars:

diff --git a/tests/integration/initial/03-unnumbered.yml b/tests/integration/initial/03-unnumbered.yml
@@ -12,8 +12,18 @@ links:
 - r:
     ipv4: True
     ipv6: True
+- r:
+    ipv4: True
+    ipv6: False
+- r:
+    ipv6: True
+    ipv4: False
+- r:
+    ipv4: True            # IPv6 numbered from LAN pool
+- r:
+    ipv6: True            # IPv4 numbered from LAN pool
 
 message: |
-  This test checks that the initial configuration doesn't crash when
-  dealing with the unnumbered interfaces. Use OSPFv2/OSPFv3 integration
-  tests to validate the unnumbered interface functionality.
+  This test checks that the initial configuration doesn't crash when dealing
+  with the all possible combinations of unnumbered interfaces. Use OSPFv2/OSPFv3
+  integration tests to validate the unnumbered interface functionality.