Skip to content

Commit 63942f6

Browse files
Fixed memtier when DB is password protected (#372)
1 parent d748195 commit 63942f6

File tree

4 files changed

+13
-6
lines changed

4 files changed

+13
-6
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[tool.poetry]
22
name = "redisbench-admin"
3-
version = "0.9.18"
3+
version = "0.9.19"
44
description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
55
authors = ["filipecosta90 <[email protected]>","Redis Performance Group <[email protected]>"]
66
readme = "README.md"

redisbench_admin/run/args.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,13 @@
3838
MAX_PROFILERS_PER_TYPE = int(os.getenv("MAX_PROFILERS", 1))
3939
PROFILE_FREQ = os.getenv("PROFILE_FREQ", PROFILE_FREQ_DEFAULT)
4040
KEEP_ENV = bool(os.getenv("KEEP_ENV", False))
41+
ALLOWED_TOOLS_DEFAULT = (
42+
"memtier_benchmark,redis-benchmark,redisgraph-benchmark-go,ycsb,go-ycsb,"
43+
+ "tsbs_run_queries_redistimeseries,tsbs_load_redistimeseries,"
44+
+ "ftsb_redisearch,"
45+
+ "aibench_run_inference_redisai_vision,ann-benchmarks",
46+
)
47+
ALLOWED_BENCH_TOOLS = os.getenv("ALLOWED_BENCH_TOOLS", ALLOWED_TOOLS_DEFAULT)
4148

4249

4350
def common_run_args(parser):
@@ -65,10 +72,7 @@ def common_run_args(parser):
6572
parser.add_argument(
6673
"--allowed-tools",
6774
type=str,
68-
default="memtier_benchmark,redis-benchmark,redisgraph-benchmark-go,ycsb,go-ycsb,"
69-
+ "tsbs_run_queries_redistimeseries,tsbs_load_redistimeseries,"
70-
+ "ftsb_redisearch,"
71-
+ "aibench_run_inference_redisai_vision,ann-benchmarks",
75+
default=ALLOWED_BENCH_TOOLS,
7276
help="comma separated list of allowed tools for this module. By default all the supported are allowed.",
7377
)
7478
parser.add_argument(

redisbench_admin/run/common.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,7 @@ def prepare_benchmark_parameters_specif_tooling(
223223
entry,
224224
cluster_api_enabled,
225225
remote_results_file,
226+
redis_password,
226227
)
227228
if "ann" in benchmark_tool:
228229
ann_path = ANN_MULTIRUN_PATH

redisbench_admin/run/memtier_benchmark/memtier_benchmark.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,14 @@ def prepare_memtier_benchmark_command(
1818
benchmark_config: object,
1919
cluster_api_enabled: bool,
2020
result_file: str,
21+
redis_pass=None,
2122
):
2223
command_arr = [executable_path]
2324
command_arr.extend(["-s", "{}".format(server_private_ip)])
2425
command_arr.extend(["-p", "{}".format(server_plaintext_port)])
2526
command_arr.extend(["--hide-histogram"])
26-
27+
if redis_pass is not None:
28+
command_arr.extend(["-a", "{}".format(redis_pass)])
2729
if cluster_api_enabled:
2830
command_arr.extend(["--cluster-mode"])
2931
if "parameters" in benchmark_config:

0 commit comments

Comments
 (0)