From: Luis Gomez <ecelgp@gmail.com>
Date: Thu, 4 Sep 2014 21:17:17 +0000 (+0000)
Subject: Merge "Renamed populate.py to crud.py Fixed method call name change Signed-off-by... 
X-Git-Tag: release/helium~35
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=d19bee61d7690144f005d46a8bcb58c590365952;hp=2151e91fd93e00e18f2ac5d3dd833989431323c7;p=integration%2Ftest.git

Merge "Renamed populate.py to crud.py Fixed method call name change Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>"
---

diff --git a/test/csit/suites/l2switch/030__LoopRemoval_OF13/__init__.txt b/test/csit/suites/l2switch/030__LoopRemoval_OF13/__init__.txt
index d4d6eae73a..645ae2f672 100644
--- a/test/csit/suites/l2switch/030__LoopRemoval_OF13/__init__.txt
+++ b/test/csit/suites/l2switch/030__LoopRemoval_OF13/__init__.txt
@@ -5,7 +5,7 @@ Suite Teardown    Stop Suite
 Library     SSHLibrary
 
 *** Variables ***
-${start}=  sudo mn --controller=remote,ip=${CONTROLLER} --custom ${USER_HOME}/integration/test/csit/suites/l2switch/topologies/loop.py --topo loop --switch ovsk,protocols=OpenFlow13
+${start}=  sudo mn --controller=remote,ip=${CONTROLLER} --custom customtopo.py --topo ring --switch ovsk,protocols=OpenFlow13
 
 ** Keywords ***
 Start Suite
@@ -15,6 +15,7 @@ Start Suite
     Write    sudo ovs-vsctl set-manager ptcp:6644
     Write    sudo mn -c
     Sleep    2
+    Put File    ${USER_HOME}/integration/test/csit/suites/l2switch/topologies/customtopo.py
     Write    ${start}
     Sleep    30
     Read
diff --git a/test/csit/suites/l2switch/topologies/customtopo.py b/test/csit/suites/l2switch/topologies/customtopo.py
new file mode 100755
index 0000000000..53327ad8f3
--- /dev/null
+++ b/test/csit/suites/l2switch/topologies/customtopo.py
@@ -0,0 +1,54 @@
+# !/usr/bin/python
+
+# usage: sudo mn --controller=remote,ip=<controller_ip> --switch=ovsk,protocols=OpenFlow13 --custom <path to customtopo.py> --topo ring ...
+
+from mininet.topo import Topo
+
+
+def add_hosts_to_switch(self, switch, hosts, start_host_suffix):
+    host_suffix = start_host_suffix
+    for _ in range(hosts):
+        host = self.addHost("h%s" % host_suffix)
+        self.addLink(switch, host)
+        host_suffix += 1
+
+
+class RingTopo(Topo):
+    def __init__(self, switches=3, hosts_per_switch=1, **opts):
+        Topo.__init__(self, **opts)
+        host_suffix = 1
+        switch = self.addSwitch('s%s' % 1)
+        first_switch = switch
+        for i in range(1, switches):
+            # add hosts to switch
+            add_hosts_to_switch(self, switch, hosts_per_switch, host_suffix)
+            host_suffix += hosts_per_switch
+
+            new_switch = self.addSwitch('s%s' % (i + 1))
+            self.addLink(new_switch, switch)
+            switch = new_switch
+
+        add_hosts_to_switch(self, switch, hosts_per_switch, host_suffix)
+        self.addLink(switch, first_switch)
+
+
+class MeshTopo(Topo):
+    def __init__(self, switches=3, hosts_per_switch=1, **opts):
+        Topo.__init__(self, **opts)
+        created_switches = []
+        host_suffix = 1
+        for i in range(switches):
+            new_switch = self.addSwitch('s%s' % (i + 1))
+
+            # add hosts to new switch
+            add_hosts_to_switch(self, new_switch, hosts_per_switch, host_suffix)
+            host_suffix += hosts_per_switch
+
+            for switch in created_switches:
+                self.addLink(new_switch, switch)
+
+            created_switches.append(new_switch)
+
+
+topos = {'ring': RingTopo,
+         'mesh': MeshTopo}
diff --git a/test/csit/suites/l2switch/topologies/loop.py b/test/csit/suites/l2switch/topologies/loop.py
deleted file mode 100644
index 45cea0a370..0000000000
--- a/test/csit/suites/l2switch/topologies/loop.py
+++ /dev/null
@@ -1,28 +0,0 @@
-#!/usr/bin/python
-
-# usage: sudo mn --controller=remote,ip=<controller_ip> --switch=ovsk,protocols=OpenFlow13 --custom <path to createtopo.py> --topo loop ...
-
-from mininet.topo import Topo
-from mininet.net import Mininet
-from mininet.node import RemoteController
-from mininet.cli import CLI
-
-class LoopTopo(Topo):
-        def __init__(self, switches = 3, hosts_per = 1, **opts):
-            Topo.__init__(self, **opts)
-            sws = []
-            hnum = 0
-            for i in range(switches):
-                sw = self.addSwitch('s%s' % (i+ 1))
-
-                for _ in range(hosts_per):
-                    hnum += 1
-                    host = self.addHost('h%s' % hnum)
-                    self.addLink(sw, host)
-
-                for rhs in sws:
-                    self.addLink(sw, rhs)
-
-                sws.append(sw)
-
-topos = { 'loop': LoopTopo }
diff --git a/test/csit/variables/xmls/f19.xml b/test/csit/variables/xmls/f19.xml
index 177917eaff..789b9f9bd6 100644
--- a/test/csit/variables/xmls/f19.xml
+++ b/test/csit/variables/xmls/f19.xml
@@ -19,7 +19,7 @@
                     <output-action>
                         <output-node-connector>INPORT</output-node-connector>
                         <max-length>60</max-length>
-                    </output-action>>
+                    </output-action>
                 </action>
             </apply-actions>
         </instruction>
diff --git a/test/csit/variables/xmls/f20.xml b/test/csit/variables/xmls/f20.xml
index 13c9e6fde5..6e200d872b 100644
--- a/test/csit/variables/xmls/f20.xml
+++ b/test/csit/variables/xmls/f20.xml
@@ -18,7 +18,7 @@
                     <order>0</order>
                     <output-action>
                         <output-node-connector>LOCAL</output-node-connector>
-                    </output-action>>
+                    </output-action>
                 </action>
             </apply-actions>
         </instruction>