Fix critical bugs in pubsub-sub-bench integration

Bug Fixes:
1. Fixed UnboundLocalError: 'is_multiple_clients' referenced before assignment
   - Removed duplicate/misplaced code block that used is_multiple_clients before definition
   - Ensured proper code flow in process_self_contained_coordinator_stream()

2. Fixed TypeError: argument of type 'NoneType' is not iterable
   - Added null check in delete_temporary_files() for benchmark_tool_global
   - Changed: if 'redis-benchmark' in benchmark_tool_global
   - To: if benchmark_tool_global and 'redis-benchmark' in benchmark_tool_global

Testing:
- All 5 pubsub-sub-bench tests continue to pass
- Successfully tested mixed workload execution:
  * memtier_benchmark doing PUBLISH commands
  * pubsub-sub-bench doing SUBSCRIBE commands
  * Both tools running simultaneously with proper parameter preparation

The runner now correctly handles mixed pub/sub workloads without crashes.

Author: fcostaoliveira

redis_benchmarks_specification/__runner__/runner.py

@@ -641,7 +641,7 @@ def delete_temporary_files(
         if preserve_temporary_client_dirs is True:
             logging.info(f"Preserving temporary client dir {temporary_dir_client}")
         else:
-            if "redis-benchmark" in benchmark_tool_global:
+            if benchmark_tool_global and "redis-benchmark" in benchmark_tool_global:
                 if full_result_path is not None:
                     os.remove(full_result_path)
                     logging.info("Removing temporary JSON file")
@@ -988,76 +988,6 @@ def delete_temporary_files(
                     )
                     arbitrary_command = False
 
-                    # Handle multiple client configurations
-                    if is_multiple_clients:
-                        # Use the new multiple client runner
-                        logging.info("Preparing multiple client configurations")
-                        # We'll handle benchmark preparation inside run_multiple_clients
-                        benchmark_command_str = "multiple_clients"  # Placeholder
-                    else:
-                        # Legacy single client preparation
-                        if "memtier_benchmark" in benchmark_tool:
-                            (
-                                _,
-                                benchmark_command_str,
-                                arbitrary_command,
-                            ) = prepare_memtier_benchmark_parameters(
-                                benchmark_config["clientconfig"],
-                                full_benchmark_path,
-                                port,
-                                host,
-                                password,
-                                local_benchmark_output_filename,
-                                oss_cluster_api_enabled,
-                                tls_enabled,
-                                tls_skip_verify,
-                                test_tls_cert,
-                                test_tls_key,
-                                test_tls_cacert,
-                                resp_version,
-                                override_memtier_test_time,
-                                override_test_runs,
-                                unix_socket,
-                            )
-                        elif "pubsub-sub-bench" in benchmark_tool:
-                            (
-                                _,
-                                benchmark_command_str,
-                                arbitrary_command,
-                            ) = prepare_pubsub_sub_bench_parameters(
-                                benchmark_config["clientconfig"],
-                                full_benchmark_path,
-                                port,
-                                host,
-                                password,
-                                local_benchmark_output_filename,
-                                oss_cluster_api_enabled,
-                                tls_enabled,
-                                tls_skip_verify,
-                                test_tls_cert,
-                                test_tls_key,
-                                test_tls_cacert,
-                                resp_version,
-                                override_memtier_test_time,
-                                unix_socket,
-                                None,  # username - could be extracted from config if needed
-                            )
-                        else:
-                            # prepare the benchmark command for other tools
-                            (
-                                benchmark_command,
-                                benchmark_command_str,
-                            ) = prepare_benchmark_parameters(
-                                benchmark_config,
-                                full_benchmark_path,
-                                port,
-                                host,
-                                local_benchmark_output_filename,
-                                False,
-                                benchmark_tool_workdir,
-                                False,
-                            )
-
                     if (
                         arbitrary_command
                         and oss_cluster_api_enabled

…suites/pubsub-mixed-workload-example.yml …els-128B-40-publishers-5-subscribers.ymlredis_benchmarks_specification/test-suites/pubsub-mixed-workload-example.yml renamed to redis_benchmarks_specification/test-suites/memtier_benchmark-nokeys-pubsub-mixed-100-channels-128B-40-publishers-5-subscribers.yml redis_benchmarks_specification/test-suites/pubsub-mixed-workload-example.yml renamed to redis_benchmarks_specification/test-suites/memtier_benchmark-nokeys-pubsub-mixed-100-channels-128B-40-publishers-5-subscribers.yml

@@ -1,5 +1,5 @@
 version: 0.4
-name: pubsub-mixed-workload-memtier-publish-pubsub-subscribe
+name: memtier_benchmark-nokeys-pubsub-mixed-100-channels-128B-40-publishers-5-subscribers
 description: Mixed workload with memtier publishing messages and pubsub-sub-bench subscribing to channels simultaneously.
 dbconfig:
   configuration-parameters: