Permalink
Browse files

[perf ref] get num_nodes and num_clients in TestInput

Move the burden from Jenkins to TestInput.
We don't have to pass in num_nodes and num_clients during do_cluster

Change-Id: I28a03762377034d3db7766bfe87cf6a169504379
Reviewed-on: http://review.couchbase.org/15196
Reviewed-by: Steve Yen <steve.yen@gmail.com>
Reviewed-by: Farshid Ghods <farshid@couchbase.com>
Tested-by: Ronnie Sun <ronniedada@gmail.com>
  • Loading branch information...
1 parent ebca703 commit 32c29a8c879c84242dfd5ef7a3bdc6171405d3cc @ronniedada ronniedada committed with ronniedada Apr 25, 2012
Showing with 6 additions and 0 deletions.
  1. +6 −0 TestInput.py
View
@@ -141,6 +141,12 @@ def get_test_input(argv):
else:
input = TestInputParser.parse_from_command_line(argv)
input.test_params = params
+
+ if "num_clients" not in input.test_params.keys() and input.clients: # do not override the command line value
+ input.test_params["num_clients"] = len(input.clients)
+ if "num_nodes" not in input.test_params.keys() and input.servers:
+ input.test_params["num_nodes"] = len(input.servers)
+
return input
@staticmethod

0 comments on commit 32c29a8

Please sign in to comment.