Merge pull request #557 from onurgashi/issue534

Storm control in l3ls_evpn the unit as optional
aristanetworks · Dec 31, 2020 · e3df86a · e3df86a
2 parents 3b0bf88 + ede475e
commit e3df86a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/ansible_collections/arista/avd/roles/eos_l3ls_evpn/README.md b/ansible_collections/arista/avd/roles/eos_l3ls_evpn/README.md
@@ -1242,16 +1242,16 @@ port_profiles:
     storm_control:
       all:
         level: < Configure maximum storm-control level >
-        unit: < percent | pps >
+        unit: < percent | pps > | Optional var and is hardware dependant - default is percent)
       broadcast:
         level: < Configure maximum storm-control level >
-        unit: < percent | pps >
+        unit: < percent | pps > | Optional var and is hardware dependant - default is percent)
       multicast:
         level: < Configure maximum storm-control level >
-        unit: < percent | pps >
+        unit: < percent | pps > | Optional var and is hardware dependant - default is percent)
       unknown_unicast:
         level: < Configure maximum storm-control level >
-        unit: < percent | pps >
+        unit: < percent | pps > | Optional var and is hardware dependant - default is percent)
 
 # Dictionary of servers, a device attaching to a L2 switched port(s)
 servers:

diff --git a/...collections/arista/avd/roles/eos_l3ls_evpn/templates/edge_ports/leaf-server-interfaces.j2 b/...collections/arista/avd/roles/eos_l3ls_evpn/templates/edge_ports/leaf-server-interfaces.j2
@@ -28,7 +28,9 @@
 {%                     for section in port_profiles[adapter.profile].storm_control %}
       {{ section }}:
         level: {{ port_profiles[adapter.profile].storm_control[section].level }}
+{%                         if port_profiles[adapter.profile].storm_control[section].unit is defined and port_profiles[adapter.profile].storm_control[section].unit is not none %}
         unit: {{ port_profiles[adapter.profile].storm_control[section].unit }}
+{%                         endif %}
 {%                     endfor %}
 {%                 endif %}
 {%                 if port_profiles[adapter.profile].native_vlan is defined %}