NFVBENCH-102 NFVBench won't work with external chain
[nfvbench.git] / nfvbench / chain_managers.py
index 033eb7a..5882913 100644 (file)
 #    License for the specific language governing permissions and limitations
 #    under the License.
 #
+import time
 
 from log import LOG
 from network import Network
 from packet_analyzer import PacketAnalyzer
 from specs import ChainType
 from stats_collector import IntervalCollector
-import time
 
 
 class StageManager(object):
+    """A class to stage resources in the systenm under test."""
 
     def __init__(self, config, cred, factory):
         self.config = config
@@ -55,7 +56,7 @@ class StageManager(object):
         return self.client.ports
 
     def get_compute_nodes(self):
-        return self.client.compute_nodes
+        return self.client.compute_nodes if self.client else {}
 
     def set_vm_macs(self):
         if self.client and self.config.service_chain != ChainType.EXT:
@@ -66,7 +67,8 @@ class StageManager(object):
             self.client.dispose()
 
 
-class StatsManager(object):
+class PVPStatsManager(object):
+    """A class to generate traffic and extract results for PVP chains."""
 
     def __init__(self, config, clients, specs, factory, vlans, notifier=None):
         self.config = config
@@ -79,7 +81,10 @@ class StatsManager(object):
         self._setup()
 
     def set_vlan_tag(self, device, vlan):
-        self.worker.set_vlan_tag(device, vlan)
+        if self.worker:
+            self.worker.set_vlan_tag(device, vlan)
+        else:
+            device.set_vlan_tag(vlan)
 
     def _setup(self):
         WORKER_CLASS = self.factory.get_chain_worker(self.specs.openstack.encaps,
@@ -88,18 +93,31 @@ class StatsManager(object):
         try:
             self.worker.set_vlans(self.vlans)
             self._config_interfaces()
-        except Exception as exc:
+        except Exception:
             # since the wrorker is up and running, we need to close it
             # in case of exception
             self.close()
-            raise exc
+            raise
 
     def _get_data(self):
-        return self.worker.get_data()
+        return self.worker.get_data() if self.worker else {}
+
+    def _get_network(self, traffic_port, stats, reverse=False):
+        """Get the Network object corresponding to a given TG port.
 
-    def _get_network(self, traffic_port, index=None, reverse=False):
-        interfaces = [self.clients['traffic'].get_interface(traffic_port)]
-        interfaces.extend(self.worker.get_network_interfaces(index))
+        :param traffic_port: must be either 0 or 1
+        :param stats: TG stats for given traffic port
+        :param reverse: specifies if the interface list for this network
+        should go from TG to loopback point (reverse=false) or
+        from loopback point to TG (reverse=true)
+        """
+        # build the interface list in fwd direction (TG To loopback point)
+        interfaces = [self.clients['traffic'].get_interface(traffic_port, stats)]
+        if self.worker:
+            # if available,
+            # interfaces for workers must be aligned on the TG port number
+            interfaces.extend(self.worker.get_network_interfaces(traffic_port))
+        # let Network reverse the interface order if needed
         return Network(interfaces, reverse)
 
     def _config_interfaces(self):
@@ -110,7 +128,7 @@ class StatsManager(object):
 
     def _generate_traffic(self):
         if self.config.no_traffic:
-            return
+            return {}
 
         self.interval_collector = IntervalCollector(time.time())
         self.interval_collector.attach_notifier(self.notifier)
@@ -126,13 +144,12 @@ class StatsManager(object):
         return self.interval_collector.get() if self.interval_collector else []
 
     def get_version(self):
-        return self.worker.get_version()
+        return self.worker.get_version() if self.worker else {}
 
     def run(self):
-        """
-        Run analysis in both direction and return the analysis
-        """
-        self.worker.run()
+        """Run analysis in both direction and return the analysis."""
+        if self.worker:
+            self.worker.run()
 
         stats = self._generate_traffic()
         result = {
@@ -141,22 +158,24 @@ class StatsManager(object):
             'stats': stats
         }
 
+        # fetch latest stats from traffic gen
+        stats = self.clients['traffic'].get_stats()
         LOG.info('Requesting packet analysis on the forward direction...')
         result['packet_analysis']['direction-forward'] = \
-            self.get_analysis([self._get_network(0, 0),
-                               self._get_network(0, 1, reverse=True)])
+            self.get_analysis([self._get_network(0, stats),
+                               self._get_network(1, stats, reverse=True)])
         LOG.info('Packet analysis on the forward direction completed')
 
         LOG.info('Requesting packet analysis on the reverse direction...')
         result['packet_analysis']['direction-reverse'] = \
-            self.get_analysis([self._get_network(1, 1),
-                               self._get_network(1, 0, reverse=True)])
+            self.get_analysis([self._get_network(1, stats),
+                               self._get_network(0, stats, reverse=True)])
 
         LOG.info('Packet analysis on the reverse direction completed')
         return result
 
     def get_compute_nodes_bios(self):
-        return self.worker.get_compute_nodes_bios()
+        return self.worker.get_compute_nodes_bios() if self.worker else {}
 
     @staticmethod
     def get_analysis(nets):
@@ -175,24 +194,18 @@ class StatsManager(object):
         return packet_analyzer.get_analysis()
 
     def close(self):
-        self.worker.close()
-
-
-class PVPStatsManager(StatsManager):
-
-    def __init__(self, config, clients, specs, factory, vlans, notifier=None):
-        StatsManager.__init__(self, config, clients, specs, factory, vlans, notifier)
+        if self.worker:
+            self.worker.close()
 
 
-class PVVPStatsManager(StatsManager):
+class PVVPStatsManager(PVPStatsManager):
+    """A Class to generate traffic and extract results for PVVP chains."""
 
     def __init__(self, config, clients, specs, factory, vlans, notifier=None):
-        StatsManager.__init__(self, config, clients, specs, factory, vlans, notifier)
+        PVPStatsManager.__init__(self, config, clients, specs, factory, vlans, notifier)
 
     def run(self):
-        """
-        Run analysis in both direction and return the analysis
-        """
+        """Run analysis in both direction and return the analysis."""
         fwd_v2v_net, rev_v2v_net = self.worker.run()
 
         stats = self._generate_traffic()
@@ -201,16 +214,17 @@ class PVVPStatsManager(StatsManager):
             'packet_analysis': {},
             'stats': stats
         }
-
-        fwd_nets = [self._get_network(0, 0)]
+        # fetch latest stats from traffic gen
+        stats = self.clients['traffic'].get_stats()
+        fwd_nets = [self._get_network(0, stats)]
         if fwd_v2v_net:
             fwd_nets.append(fwd_v2v_net)
-        fwd_nets.append(self._get_network(0, 1, reverse=True))
+        fwd_nets.append(self._get_network(1, stats, reverse=True))
 
-        rev_nets = [self._get_network(1, 1)]
+        rev_nets = [self._get_network(1, stats)]
         if rev_v2v_net:
             rev_nets.append(rev_v2v_net)
-        rev_nets.append(self._get_network(1, 0, reverse=True))
+        rev_nets.append(self._get_network(0, stats, reverse=True))
 
         LOG.info('Requesting packet analysis on the forward direction...')
         result['packet_analysis']['direction-forward'] = self.get_analysis(fwd_nets)
@@ -223,15 +237,20 @@ class PVVPStatsManager(StatsManager):
         return result
 
 
-class EXTStatsManager(StatsManager):
+class EXTStatsManager(PVPStatsManager):
+    """A Class to generate traffic and extract results for EXT chains."""
+
     def __init__(self, config, clients, specs, factory, vlans, notifier=None):
-        StatsManager.__init__(self, config, clients, specs, factory, vlans, notifier)
+        PVPStatsManager.__init__(self, config, clients, specs, factory, vlans, notifier)
 
     def _setup(self):
-        WORKER_CLASS = self.factory.get_chain_worker(self.specs.openstack.encaps,
-                                                     self.config.service_chain)
-        self.worker = WORKER_CLASS(self.config, self.clients, self.specs)
-        self.worker.set_vlans(self.vlans)
+        if self.specs.openstack:
+            WORKER_CLASS = self.factory.get_chain_worker(self.specs.openstack.encaps,
+                                                         self.config.service_chain)
+            self.worker = WORKER_CLASS(self.config, self.clients, self.specs)
+            self.worker.set_vlans(self.vlans)
 
-        if not self.config.no_int_config:
-            self._config_interfaces()
+            if not self.config.no_int_config:
+                self._config_interfaces()
+        else:
+            self.worker = None