From 7b24bdc34dcded763c67daf72c024d7a6f0991e3 Mon Sep 17 00:00:00 2001 From: fcosta_oliveira Date: Wed, 24 Sep 2025 09:33:05 +0100 Subject: [PATCH] Removed arbitrary command escape on cluster mode and memtier --- .../__runner__/runner.py | 21 ------------------- 1 file changed, 21 deletions(-) diff --git a/redis_benchmarks_specification/__runner__/runner.py b/redis_benchmarks_specification/__runner__/runner.py index ad3fca73..2b15590b 100644 --- a/redis_benchmarks_specification/__runner__/runner.py +++ b/redis_benchmarks_specification/__runner__/runner.py @@ -2210,21 +2210,6 @@ def delete_temporary_files( ) arbitrary_command = False - if ( - arbitrary_command - and oss_cluster_api_enabled - and "memtier" in benchmark_tool - ): - logging.warning( - "Forcing skip this test given there is an arbitrary commmand and memtier usage. Check https://github.com/RedisLabs/memtier_benchmark/pull/117 ." - ) - delete_temporary_files( - temporary_dir_client=temporary_dir_client, - full_result_path=None, - benchmark_tool_global=benchmark_tool_global, - ) - continue - # Check if we have multiple client configurations client_configs = extract_client_configs(benchmark_config) is_multiple_clients = len(client_configs) > 1 @@ -3375,12 +3360,6 @@ def data_prepopulation_step( 1, unix_socket, ) - if arbitrary_command is True and oss_cluster_api_enabled: - logging.warning( - "Skipping this test given it implies arbitrary command on an cluster setup. Not supported on memtier: https://github.com/RedisLabs/memtier_benchmark/pull/117" - ) - result = False - return result # run the benchmark preload_start_time = datetime.datetime.now()