Skip to content

Commit 7d9bec7

Browse files
committed
Fix few bugs and script issues
1 parent 1074253 commit 7d9bec7

File tree

6 files changed

+14
-10
lines changed

6 files changed

+14
-10
lines changed

Godeps/Godeps.json

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

scripts/python/api/objmodel.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -431,10 +431,9 @@ def setFabricMode(mode):
431431
"vlans": "1-4094",
432432
"vxlans": "1-10000",
433433
})
434-
response = httpPost(postUrl, jdata)
434+
response = http.httpPost(postUrl, jdata)
435435

436436
# Check for error
437437
if response == "Error":
438438
print response
439439
errorExit("setFabricMode failed")
440-

scripts/python/api/tbed.py

+6-4
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,18 @@ def __init__(self, addrList, username='vagrant', password='vagrant', binpath='/o
3333

3434
# Start netplugin on all nodes
3535
for node in self.nodes:
36-
print "Starting netplugin on " + node.addr
36+
print "Starting netplugin on " + node.hostname
3737
node.startNetplugin()
3838

3939
# Wait few seconds before starting netmaster
4040
time.sleep(3)
4141

4242
# Start netmaster in the end
43-
for node in self.nodes:
44-
print "Starting netmaster"
45-
node.startNetmaster()
43+
for nidx, node in enumerate(self.nodes):
44+
# Start netmaster only on first three nodes
45+
if nidx < 3:
46+
print "Starting netmaster on " + node.hostname
47+
node.startNetmaster()
4648

4749
# Cleanup a testbed once test is done
4850
def cleanup(self):

scripts/python/runTestCase.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import testcases.tcPolicy
1111
import testcases.tcNetwork
1212
import testcases.tcTrigger
13+
import testcases.tcAci
1314
import argparse
1415
import os
1516
import exceptions
@@ -59,7 +60,7 @@
5960
testbed.chekForNetpluginErrors()
6061

6162
# Run single test case
62-
testcases.tcTrigger.netpluginDisconnectTest(testbed, numCntr, numIteration, encap="vxlan")
63+
testcases.tcTrigger.netmasterSwitchoverTest(testbed, numCntr, numIteration)
6364

6465
# Cleanup testbed
6566
testbed.cleanup()
@@ -75,7 +76,7 @@
7576
api.tutils.log("Tests took " + str(elapsedTime) + " seconds")
7677

7778
api.tutils.info("testcase passed")
78-
os._exit(1)
79+
os._exit(0)
7980

8081
except exceptions.KeyboardInterrupt:
8182
print "\n\n Keyboard interrupt.... Exiting\n"

scripts/python/sanity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
api.tutils.log("Tests took " + str(elapsedTime) + " seconds")
110110

111111
api.tutils.info("Sanity passed")
112-
os._exit(1)
112+
os._exit(0)
113113

114114
except exceptions.KeyboardInterrupt:
115115
print "\n\n Keyboard interrupt.... Exiting\n"

scripts/python/testcases/tcTrigger.py

+2
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,8 @@ def netmasterSwitchoverTest(testbed, numContainer, numIter, encap="vxlan"):
325325
tenant = api.objmodel.tenant('default')
326326
network = tenant.newNetwork('private', pktTag=1001, subnet="10.1.0.0/16", gateway="10.1.1.254", encap=encap)
327327

328+
# Always run even number of iterations so that mastership comes back to original node
329+
numIter = numIter + (numIter % 2)
328330
for iter in range(numIter):
329331
# Start the containers
330332
containers = testbed.runContainers(numContainer)

0 commit comments

Comments
 (0)