Fixed allowed setups check (#402)

redis-performance · Mar 8, 2023 · 6cb27ba · 6cb27ba
1 parent 4a5d779
commit 6cb27ba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redisbench-admin"
-version = "0.10.2"
+version = "0.10.3"
 description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
 authors = ["filipecosta90 <filipecosta.90@gmail.com>","Redis Performance Group <performance@redis.com>"]
 readme = "README.md"

diff --git a/redisbench_admin/run_local/run_local.py b/redisbench_admin/run_local/run_local.py
@@ -158,7 +158,12 @@ def run_local_command_logic(args, project_name, project_version):
                             shard_count,
                         ) = get_setup_type_and_primaries_count(setup_settings)
                         if args.allowed_setups != "":
-                            allowed_setups = args.allowed_setups.split()
+                            allowed_setups = args.allowed_setups.split(",")
+                            logging.info(
+                                "Checking if setup named {} of topology type {}. Total primaries: {} is in the allowed list of setups {}".format(
+                                    setup_name, setup_type, shard_count, allowed_setups
+                                )
+                            )
                             if setup_name not in allowed_setups:
                                 logging.warning(
                                     "SKIPPING setup named {} of topology type {}.".format(