Use 'no switchport' as default on Arista EOS LAG interfaces

Replaces #1889 and #1886, fixes #1884, #1883

netsim/ansible/templates/initial/eos.j2

@@ -33,7 +33,7 @@ interface {{ mgmt.ifname|default('Management1') }}
 !
 {% for l in netlab_interfaces %}
 interface {{ l.ifname }}
-{% if l.virtual_interface is not defined %}
+{% if l.virtual_interface is not defined or l.type in ['lag'] %}
  no switchport
 {% endif %}
 {% if l.vrf is defined %}

netsim/devices/eos.py

@@ -67,16 +67,6 @@ def check_dhcp_clients(node: Box, topology: Box) -> None:
       category=log.IncorrectType,
       module='quirks')
 
-def check_l3_lag(node: Box, topology: Box) -> None:
-  for intf in node.interfaces:
-    if intf.type != 'lag':
-      continue
-    if intf.get('ipv4',False) is not False or intf.get('ipv6',False) is not False:
-      log.error(
-        f"Interface {intf.ifname} on Arista EOS node {node.name} cannot have IPv4/IPv6 addresses",
-        category=log.IncorrectType,
-        module='quirks')
-
 def configure_ceos_attributes(node: Box, topology: Box) -> None:
   serialnumber = node.eos.get('serialnumber',None)
   systemmacaddr = node.eos.get('systemmacaddr',None)
@@ -135,8 +125,6 @@ def device_quirks(self, node: Box, topology: Box) -> None:
       check_shared_mac(node,topology)
     if 'dhcp' in mods:
       check_dhcp_clients(node,topology)
-    if 'lag' in mods:
-      check_l3_lag(node,topology)
     if 'ospf' in mods:
       passive_stub_interfaces(node,topology)
     if 'eos' in node: