FIX: Remove eth_avf and portX_vifY interfaces from topo in TC/TS tear down 99/21299/5
authorJan Gelety <jgelety@cisco.com>
Tue, 13 Aug 2019 20:57:07 +0000 (22:57 +0200)
committerJan Gelety <jgelety@cisco.com>
Wed, 14 Aug 2019 02:54:43 +0000 (04:54 +0200)
Change-Id: I90901c4ab14dfa76dc3e1f786a4b986479e0ba47
Signed-off-by: Jan Gelety <jgelety@cisco.com>
19 files changed:
resources/libraries/python/InterfaceUtil.py
resources/libraries/python/topology.py
resources/libraries/robot/shared/suite_teardown.robot
tests/vpp/device/l2bd/eth2p-avf-dot1qip4-l2bdbasemaclrn-dev.robot
tests/vpp/perf/ip4/10ge2p1x710-avf-eth-ip4base-ndrpdr.robot
tests/vpp/perf/ip4/10ge2p1x710-avf-ethip4-ip4scale200k-ndrpdr.robot
tests/vpp/perf/ip4/10ge2p1x710-avf-ethip4-ip4scale20k-ndrpdr.robot
tests/vpp/perf/ip4/10ge2p1x710-avf-ethip4-ip4scale2m-ndrpdr.robot
tests/vpp/perf/ip4/2n1l-10ge2p1x710-avf-ethip4-ip4base-ndrpdr.robot
tests/vpp/perf/ip4/2n1l-10ge2p1x710-avf-ethip4-ip4scale200k-ndrpdr.robot
tests/vpp/perf/ip4/2n1l-10ge2p1x710-avf-ethip4-ip4scale20k-ndrpdr.robot
tests/vpp/perf/ip4/2n1l-10ge2p1x710-avf-ethip4-ip4scale2m-ndrpdr.robot
tests/vpp/perf/l2/10ge2p1x710-avf-eth-l2bdbasemaclrn-ndrpdr.robot
tests/vpp/perf/l2/10ge2p1x710-avf-eth-l2patch-ndrpdr.robot
tests/vpp/perf/l2/10ge2p1x710-avf-eth-l2xcbase-ndrpdr.robot
tests/vpp/perf/l2/2n1l-10ge2p1x710-avf-dot1q-l2bdbasemaclrn-gbp-ndrpdr.robot
tests/vpp/perf/l2/2n1l-10ge2p1x710-avf-eth-l2bdbasemaclrn-ndrpdr.robot
tests/vpp/perf/l2/2n1l-10ge2p1x710-avf-eth-l2patch-ndrpdr.robot
tests/vpp/perf/l2/2n1l-10ge2p1x710-avf-eth-l2xcbase-ndrpdr.robot

index 1a97130..982a345 100644 (file)
@@ -1455,12 +1455,12 @@ class InterfaceUtil(object):
 
     @staticmethod
     def init_avf_interface(node, ifc_key, numvfs=1, osi_layer='L2'):
-        """Init PCI device by creating VFs and bind them to vfio-pci for AVF
+        """Init PCI device by creating VIFs and bind them to vfio-pci for AVF
         driver testing on DUT.
 
         :param node: DUT node.
         :param ifc_key: Interface key from topology file.
-        :param numvfs: Number of VFs to initialize, 0 - disable the VFs.
+        :param numvfs: Number of VIFs to initialize, 0 - disable the VIFs.
         :param osi_layer: OSI Layer type to initialize TG with.
             Default value "L2" sets linux interface spoof off.
         :type node: dict
@@ -1518,7 +1518,7 @@ class InterfaceUtil(object):
             DUTSetup.pci_vf_driver_bind(node, pf_pci_addr, vf_id, uio_driver)
 
             # Add newly created ports into topology file
-            vf_ifc_name = '{pf_if_key}_vf'.format(pf_if_key=ifc_key)
+            vf_ifc_name = '{pf_if_key}_vif'.format(pf_if_key=ifc_key)
             vf_pci_addr = DUTSetup.get_virtfn_pci_addr(node, pf_pci_addr, vf_id)
             vf_ifc_key = Topology.add_new_port(node, vf_ifc_name)
             Topology.update_interface_name(node, vf_ifc_key,
index 698c97b..1e5ce4b 100644 (file)
@@ -13,6 +13,8 @@
 
 """Defines nodes and topology structure."""
 
+import re
+
 from collections import Counter
 
 from yaml import load
@@ -59,6 +61,7 @@ class NodeSubTypeTG(object):
     # IxNetwork
     IXNET = 'IXNET'
 
+
 DICT__nodes = load_topo_from_yaml()
 
 
@@ -139,13 +142,39 @@ class Topology(object):
         """
         port_types = ('subinterface', 'vlan_subif', 'memif', 'tap', 'vhost',
                       'loopback', 'gre_tunnel', 'vxlan_tunnel', 'eth_bond',
-                      'avf')
+                      'eth_avf')
 
         for node_data in nodes.values():
             if node_data['type'] == NodeType.DUT:
                 for ptype in port_types:
                     Topology.remove_all_ports(node_data, ptype)
 
+    @staticmethod
+    def remove_all_vif_ports(node):
+        """Remove all Virtual Interfaces on DUT node.
+
+        :param node: Node to remove VIF ports on.
+        :type node: dict
+        :returns: Nothing
+        """
+        reg_ex = re.compile(r'port\d+_vif\d+')
+        for if_key in list(node['interfaces']):
+            if re.match(reg_ex, if_key):
+                node['interfaces'].pop(if_key)
+
+    @staticmethod
+    def remove_all_added_vif_ports_on_all_duts_from_topology(nodes):
+        """Remove all added Virtual Interfaces on all DUT nodes in
+        the topology.
+
+        :param nodes: Nodes in the topology.
+        :type nodes: dict
+        :returns: Nothing
+        """
+        for node_data in nodes.values():
+            if node_data['type'] == NodeType.DUT:
+                Topology.remove_all_vif_ports(node_data)
+
     @staticmethod
     def update_interface_sw_if_index(node, iface_key, sw_if_index):
         """Update sw_if_index on the interface from the node.
index e5db4f0..a6c021e 100644 (file)
@@ -45,3 +45,9 @@
 | | :FOR | ${dut} | IN | @{duts}
 | | | Cleanup DPDK Environment
 | | | ... | ${nodes['${dut}']} | ${${dut}_if1} | ${${dut}_if2}
+
+| Additional Suite Tear Down Action For vifs
+| | [Documentation]
+| | ... | Additional teardown for suites which uses Virtual Interfaces.
+| | ...
+| | Remove All Added VIF Ports On All DUTs From Topology | ${nodes}
index b75b89c..9d79d80 100644 (file)
@@ -18,6 +18,7 @@
 | ... | NIC_Virtual | ETH | IP4FWD | BASE | DOT1Q | IP4BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | avf | scapy
+| Suite Teardown | Tear down suite | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | packet_trace
 | ...
index 7be6e27..bd92299 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | BASE | IP4BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 658e2a3..ae3297f 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_200K | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 7513272..c0cfe85 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_20K | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index c111b6a..8264fdd 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_2M | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index b38437a..cc3a5f7 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | BASE | IP4BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index c5dd1d3..bbdcf11 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_200k | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 35aefc0..47ce853 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_20k | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 066f145..6d4f7d1 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | IP4FWD | SCALE | FIB_2M | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index c49a478..f94caf3 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2BDMACLRN | BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 52c8edf..0e40fce 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2PATCH | BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 558694a..af329fb 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2XCFWD | BASE | L2XCBASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index bcf5764..b582bca 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | DOT1Q | L2BDMACLRN | BASE | DRV_AVF | GBP
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 4865f69..9236108 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2BDMACLRN | BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index 165e816..a208e5b 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2PATCH | BASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...
index ac55264..8d8594a 100644 (file)
@@ -18,7 +18,7 @@
 | ... | NIC_Intel-X710 | ETH | L2XCFWD | BASE | L2XCBASE | DRV_AVF
 | ...
 | Suite Setup | Setup suite single link | performance_avf
-| Suite Teardown | Tear down suite | performance
+| Suite Teardown | Tear down suite | performance | vifs
 | Test Setup | Setup test
 | Test Teardown | Tear down test | performance
 | ...

©2016 FD.io a Linux Foundation Collaborative Project. All Rights Reserved.
Linux Foundation is a registered trademark of The Linux Foundation. Linux is a registered trademark of Linus Torvalds.
Please see our privacy policy and terms of use.