ArubaCX: fix MTU plus minor stuff (#979)

ipspace · Dec 18, 2023 · 3e60b1a · 3e60b1a
1 parent fe54c52
commit 3e60b1a
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 3 deletions.
diff --git a/docs/caveats.md b/docs/caveats.md
@@ -199,7 +199,7 @@ vSRX container built with *vrnetlab* uses **flow based forwarding**. You have tw
 
 ### VXLAN and EVPN Caveats
 
-* The VXLAN dataplane (at least, on the virtual version) seems not supporting VNI greater than 65535. If you set an higher value, an overflow will occur, and you may have overlapping VNIs. The workaround for this is to set, i.e., `defaults.vxlan.start_vni: 20000` (especially on multi-vendor topologies).
+* The VXLAN dataplane (at least, on the virtual version) seems not supporting VNI greater than 65535. If you set an higher value, an overflow will occur, and you may have overlapping VNIs. The workaround for this is to set, i.e., `defaults.vxlan.start_vni: 20000` and `defaults.evpn.start_transit_vni: 10000` (especially on multi-vendor topologies).
 * EVPN Symmetric IRB is supported only from the Aruba AOS-CX Virtual version *10.13*. Additionally:
   * CPU generated traffic does not get encapsulated in Symmetric IRB on AOS-CX Simulator.
   * Active-Gateway MAC Addresses shall be the same across all VTEPs in AOS-CX Simulator.
diff --git a/docs/platforms.md b/docs/platforms.md
@@ -131,7 +131,7 @@ Ansible playbooks included with **netlab** can deploy and collect device configu
 | Operating system      | Deploy configuration | Collect configuration |
 | --------------------- | :------------------: | :-------------------: |
 | Arista EOS            |          ✅           |           ✅           |
-| Aruba AOS-CX          |          ✅           |           ❌           |
+| Aruba AOS-CX          |          ✅           |           ✅           |
 | Cisco ASAv            |          ✅           |           ✅           |
 | Cisco IOS / IOS XE    |          ✅           |           ✅           |
 | Cisco IOS XRv         |          ✅           |           ✅           |

diff --git a/netsim/ansible/tasks/deploy-config/arubacx.yml b/netsim/ansible/tasks/deploy-config/arubacx.yml
@@ -10,6 +10,12 @@
     src: "{{ config_template }}"
     dest: "{{ tempfile_1.path }}"
 
+# This is required when running "netlab initial" multiple times...
+- name: "aoscx_config: set session auto-confirm for {{ netsim_action }}"
+  arubanetworks.aoscx.aoscx_config:
+    lines:
+    - "auto-confirm"
+
 - name: "aoscx_config: deploying {{ netsim_action }} from {{ config_template }}"
   arubanetworks.aoscx.aoscx_config:
     match: "none"

diff --git a/netsim/ansible/tasks/fetch-config/arubacx.yml b/netsim/ansible/tasks/fetch-config/arubacx.yml
@@ -0,0 +1,11 @@
+# fetch ArubaCX configuration
+#
+---
+- arubanetworks.aoscx.aoscx_command:
+    commands:
+    - no page
+    - show running-config
+  register: conf_output
+
+- set_fact:
+    ansible_net_config: "{{ conf_output.stdout | default([]) | join('') }}\n"
diff --git a/netsim/ansible/templates/initial/arubacx.j2 b/netsim/ansible/templates/initial/arubacx.j2
@@ -29,12 +29,14 @@ interface {{ mgmt.ifname|default('mgmt') }}
 interface {{ l.ifname }}
 {% if l.virtual_interface is not defined %}
     shutdown
+{# Set the maximun allowed L2 MTU here. Fine tuning of L3 MTU will be done with ip mtu on L3 interfaces only #}
+    mtu 9198
 {% endif %}
 {% if l.vrf is defined %}
     vrf attach {{ l.vrf }}
 {% endif %}
 {% if l.mtu is defined %}
-    mtu {{ l.mtu }}
+    ip mtu {{ l.mtu }}
 {% endif %}
 {% if l.name is defined %}
     description {{ l.name }}{{ " ["+l.role+"]" if l.role is defined else "" }}