Skip to content

Commit 8dc18b6

Browse files
Remove debug statements from topology filter implementation
1 parent 81f4a42 commit 8dc18b6

File tree

1 file changed

+6
-27
lines changed
  • redis_benchmarks_specification/__runner__

1 file changed

+6
-27
lines changed

redis_benchmarks_specification/__runner__/runner.py

Lines changed: 6 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -97,19 +97,7 @@ def main():
9797
)
9898
args = parser.parse_args()
9999

100-
# Debug output
101-
print("DEBUG: Starting main function")
102-
print(f"DEBUG: Topology filter: {args.topology}")
103-
104-
# Redirect stdout to a file for debugging
105-
import sys
106-
original_stdout = sys.stdout
107-
with open('debug_output.txt', 'w') as f:
108-
sys.stdout = f
109-
print("DEBUG: Redirected stdout to file")
110-
print(f"DEBUG: Topology filter: {args.topology}")
111-
run_client_runner_logic(args, project_name, project_name_suffix, project_version)
112-
sys.stdout = original_stdout
100+
run_client_runner_logic(args, project_name, project_name_suffix, project_version)
113101

114102

115103
def run_client_runner_logic(args, project_name, project_name_suffix, project_version):
@@ -395,11 +383,6 @@ def delete_temporary_files(
395383
shutil.rmtree(temporary_dir_client, ignore_errors=True)
396384
logging.info(f"Removing temporary client dir {temporary_dir_client}")
397385

398-
# Debug output
399-
print(f"DEBUG: Starting process_self_contained_coordinator_stream")
400-
print(f"DEBUG: Topology filter: {args.topology}")
401-
print(f"DEBUG: Test files: {testsuite_spec_files}")
402-
403386
overall_result = True
404387
results_matrix = []
405388
total_test_suite_runs = 0
@@ -435,17 +418,13 @@ def delete_temporary_files(
435418
)
436419
)
437420

438-
print(f"DEBUG: Test file: {test_file}, Test name: {test_name}")
439-
print(f"DEBUG: Available topologies: {benchmark_config['redis-topologies']}")
440-
441421
for topology_spec_name in benchmark_config["redis-topologies"]:
442422
# Filter by topology if specified
443-
if args.topology and args.topology != "":
444-
if topology_spec_name != args.topology:
445-
print(f"DEBUG: Skipping topology {topology_spec_name} as it doesn't match the requested topology {args.topology}")
446-
continue
447-
else:
448-
print(f"DEBUG: Running topology {topology_spec_name} as it matches the requested topology {args.topology}")
423+
if args.topology and args.topology != "" and topology_spec_name != args.topology:
424+
logging.info(
425+
f"Skipping topology {topology_spec_name} as it doesn't match the requested topology {args.topology}"
426+
)
427+
continue
449428

450429
test_result = False
451430
benchmark_tool_global = ""

0 commit comments

Comments
 (0)